Any idea when this started becoming flaky?

Looking at the test, the test failure doesn't correlate to the test being executed.

The test which failed is synchronous and is expecting a method to either throw an exception or return normally.

The test failure in the log is a Future callback expecting true and getting false.

[+martin since this may not be solved before I go on vacation]

Looking at the test, the test failure doesn't correlate to the test being executed.
Please elaborate, from the output this correlates fine in what I see.

Looking at the test however, shouldn't we do the following?
Index: xhr_test.dart
===================================================================

--- xhr_test.dart (revision 24572)
+++ xhr_test.dart (working copy)
@@ -199,9 +199,9 @­@
         }
         xhr.send(data);
 

•    return xhr.onLoad.first.then((_) {
  

•    return xhr.onLoad.first.then(expectAsync1((_) {
  

             expect(progressCalled, isTrue);

•    });
  

•    }));
  

         });
       }
     });

cc @mkustermann.

Nope. With the test.dart waits for futures that are returned to pass before declaring a test as PASS. See tests/html/websql_test.dart for another example

Still needs clarification on this statement:
Looking at the test, the test failure doesn't correlate to the test being executed.

cheers,
Rico

Correction- the test failure is indeed the test in question.

The test attempts to use a large payload to make sure the progress events always fire, but appears to not always work.

Set owner to @blois.
Added this to the M6 milestone.
Added Accepted label.

Removed this from the M6 milestone.
Added this to the M7 milestone.

Removed this from the M7 milestone.
Added this to the M8 milestone.
Removed Priority-High label.
Added Priority-Medium label.

Moving to later as the XHR test is unstable for a variety of reasons, I do not think we can stabilize it for M8.

This specific issue appears to be an issue with the test itself rather than the implementation.

Removed this from the M8 milestone.
Added this to the Later milestone.

Removed Area-HTML label.
Added Area-Library, Library-Html labels.

Removed this from the Later milestone.
Added Oldschool-Milestone-Later label.

Removed Oldschool-Milestone-Later label.

Moving flaky test under CodeHealth.

Removed the owner.
Removed Type-Defect label.
Added Type-CodeHealth label.