More status changes for new version of Firefox.

More status changes for new version of Firefox.

R=whesse@google.com

Committed: https://github.com/dart-lang/sdk/commit/6fa31daf2736d3768f968184ff81510b34a56b96

[Lasse Reichstein Nielsen, 2017-04-24 11:53:30]

lrn@google.com changed reviewers:
+ whesse@google.com

[Lasse Reichstein Nielsen, 2017-04-24 11:53:31]

https://codereview.chromium.org/2836003002/diff/1/tests/html/html.status
File tests/html/html.status (left):

https://codereview.chromium.org/2836003002/diff/1/tests/html/html.status#oldc...
tests/html/html.status:283: # All changes should be accompanied by platform
support annotation changes.
@efortuna: Do you know what this platform support annotation is and where to
find it?

[Bill Hesse, 2017-04-24 12:04:14]

https://codereview.chromium.org/2836003002/diff/1/tests/co19/co19-dart2js.status
File tests/co19/co19-dart2js.status (right):

https://codereview.chromium.org/2836003002/diff/1/tests/co19/co19-dart2js.sta...
tests/co19/co19-dart2js.status:8353: [ $runtime == dartium || $compiler ==
dart2js ]
You cannot add new sections to this file.  Find the existing section, and put
the status line into the correct place in alphabetical order.

[Lasse Reichstein Nielsen, 2017-04-24 12:38:34]

ptal

[Bill Hesse, 2017-04-24 13:18:58]

LGTM, with comment about dartium status.

https://codereview.chromium.org/2836003002/diff/20001/tests/co19/co19-dart2js...
File tests/co19/co19-dart2js.status (right):

https://codereview.chromium.org/2836003002/diff/20001/tests/co19/co19-dart2js...
tests/co19/co19-dart2js.status:8342: [ $runtime == dartium || $compiler ==
dart2js ]
Please put this in the co19-dartium.status file.

https://codereview.chromium.org/2836003002/diff/20001/tests/corelib/corelib.s...
File tests/corelib/corelib.status (right):

https://codereview.chromium.org/2836003002/diff/20001/tests/corelib/corelib.s...
tests/corelib/corelib.status:139: [ $compiler == dart2js && $runtime == jsshell
&& $fast_startup == false]
Maybe we should update jsshell! I have added a TODO to my work list.

[Lasse Reichstein Nielsen, 2017-04-24 13:23:42]

https://codereview.chromium.org/2836003002/diff/20001/tests/co19/co19-dart2js...
File tests/co19/co19-dart2js.status (right):

https://codereview.chromium.org/2836003002/diff/20001/tests/co19/co19-dart2js...
tests/co19/co19-dart2js.status:8342: [ $runtime == dartium || $compiler ==
dart2js ]
Split up as one entry in this file and one in co19-dartium.status.

[Lasse Reichstein Nielsen, 2017-04-24 13:24:25]

Description was changed from

==========
More status changes for new version of Firefox.
==========

to

==========
More status changes for new version of Firefox.

R=whesse@google.com

Committed:
https://github.com/dart-lang/sdk/commit/6fa31daf2736d3768f968184ff81510b34a56b96
==========

[Lasse Reichstein Nielsen, 2017-04-24 13:24:26]

Committed patchset #3 (id:40001) manually as
6fa31daf2736d3768f968184ff81510b34a56b96 (presubmit successful).