Closed
Bug 1307332
Opened 8 years ago
Closed 8 years ago
Remove B2G and Mulet annotations from reftest.list
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla52
Tracking | Status | |
---|---|---|
firefox52 | --- | fixed |
People
(Reporter: dholbert, Assigned: aryx)
References
(Blocks 1 open bug)
Details
Attachments
(79 files)
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/generated-content.
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/margin-collapsing.
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/xul-document-load.
58 bytes,
text/x-review-board-request
|
dholbert
:
review+
|
Details |
As noted in bug 1306391, we're removing B2G-specific code from mozilla-central.
We have lots of (sometimes-complex) reftest.list test-failure annotations that are B2G or Mulet-specific. (Mulet is a B2G shim described at https://wiki.mozilla.org/Mulet )
Examples:
> skip-if(B2G||Mulet) fuzzy-if(skiaContent,1,3) needs-focus == input-load.html input-ref.html # B2G timed out waiting for reftest-wait to be removed # Initial mulet triage: parity with B2G/B2G Desktop
https://dxr.mozilla.org/mozilla-central/source/dom/html/reftests/autofocus/reftest.list#2
...and:
> skip-if((B2G&&browserIsRemote)||Mulet) == dynamic-1-remove-to-none-grouped.xul dynamic-1-ref.xul # Initial mulet triage: parity with B2G/B2G Desktop
https://dxr.mozilla.org/mozilla-central/source/layout/reftests/box-ordinal/reftest.list#2
Filing this bug on removing these annotations.
Comment 1•8 years ago
|
||
FYI, I think Aryx is already working on this.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
![]() |
Assignee | |
Comment 81•8 years ago
|
||
The |reftest-stylo.list|s will be reimported by Manish.
Try series which turned the patch queue (together with changes to a test manifests) from orange to green:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3a029b48343e
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6bea97f54490
https://treeherder.mozilla.org/#/jobs?repo=try&revision=b64e3687a87e
Assignee: nobody → aryx.bugmail
Status: NEW → ASSIGNED
Reporter | ||
Comment 82•8 years ago
|
||
mozreview-review |
Comment on attachment 8797444 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/abs-pos.
https://reviewboard.mozilla.org/r/83056/#review81814
Ship It!
Reporter | ||
Comment 83•8 years ago
|
||
mozreview-review |
Comment on attachment 8797444 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/abs-pos.
https://reviewboard.mozilla.org/r/83056/#review81816
Hmm, the "Ship It!" Button used to work for 2-click r+'s (after I unhide it with a customm Stylish stylesheet), but apparently it doesn't grant r+ anymore. :-/ Serves me right for using unsupported hidden functionality.
I guess I'll go with the standard ~4-click r+ workflow here. (woe is me)
Thanks for attacking this, BTW!
Attachment #8797444 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 84•8 years ago
|
||
mozreview-review |
Comment on attachment 8797445 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/async-scrolling.
https://reviewboard.mozilla.org/r/83058/#review81820
Attachment #8797445 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 85•8 years ago
|
||
mozreview-review |
Comment on attachment 8797446 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/backgrounds.
https://reviewboard.mozilla.org/r/83060/#review81826
Attachment #8797446 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 86•8 years ago
|
||
mozreview-review |
Comment on attachment 8797447 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/bidi.
https://reviewboard.mozilla.org/r/83062/#review81830
Attachment #8797447 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 87•8 years ago
|
||
mozreview-review |
Comment on attachment 8797449 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/border-image.
https://reviewboard.mozilla.org/r/83064/#review81832
Attachment #8797449 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 88•8 years ago
|
||
mozreview-review |
Comment on attachment 8797450 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/border-radius.
https://reviewboard.mozilla.org/r/83066/#review81834
Attachment #8797450 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 89•8 years ago
|
||
mozreview-review |
Comment on attachment 8797451 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/box.
https://reviewboard.mozilla.org/r/83068/#review81836
Ship It!
Reporter | ||
Comment 90•8 years ago
|
||
mozreview-review |
Comment on attachment 8797451 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/box.
https://reviewboard.mozilla.org/r/83068/#review81840
Attachment #8797451 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 91•8 years ago
|
||
mozreview-review |
Comment on attachment 8797452 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/box-ordinal.
https://reviewboard.mozilla.org/r/83070/#review81844
Attachment #8797452 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 92•8 years ago
|
||
mozreview-review |
Comment on attachment 8797453 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/box-properties.
https://reviewboard.mozilla.org/r/83072/#review81846
Attachment #8797453 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 93•8 years ago
|
||
mozreview-review |
Comment on attachment 8797454 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/box-shadow.
https://reviewboard.mozilla.org/r/83074/#review81848
Attachment #8797454 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 94•8 years ago
|
||
mozreview-review |
Comment on attachment 8797455 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/bugs.
https://reviewboard.mozilla.org/r/83076/#review81850
Looks like this patch incorporated something busted -- right now it's deleting a ton of files from layout/reftests/bugs, which I don't think is what you intended.
(Direct link to the commit showing all the deletions: https://reviewboard-hg.mozilla.org/gecko/rev/c9532f6e1ab8ddf14583f0fa5cf78dfc9bd4c561 )
Attachment #8797455 -
Flags: review?(dholbert) → review-
Reporter | ||
Comment 95•8 years ago
|
||
mozreview-review |
Comment on attachment 8797456 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/canvas.
https://reviewboard.mozilla.org/r/83078/#review81852
Attachment #8797456 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 96•8 years ago
|
||
mozreview-review |
Comment on attachment 8797457 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/columns.
https://reviewboard.mozilla.org/r/83080/#review81854
Attachment #8797457 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 97•8 years ago
|
||
mozreview-review |
Comment on attachment 8797458 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/counter-style.
https://reviewboard.mozilla.org/r/83082/#review81856
Attachment #8797458 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 98•8 years ago
|
||
mozreview-review |
Comment on attachment 8797459 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-disabled.
https://reviewboard.mozilla.org/r/83084/#review81858
Attachment #8797459 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 99•8 years ago
|
||
mozreview-review |
Comment on attachment 8797460 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-display.
https://reviewboard.mozilla.org/r/83086/#review81860
Attachment #8797460 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 100•8 years ago
|
||
mozreview-review |
Comment on attachment 8797461 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-enabled.
https://reviewboard.mozilla.org/r/83088/#review81862
Attachment #8797461 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 101•8 years ago
|
||
mozreview-review |
Comment on attachment 8797462 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-gradients.
https://reviewboard.mozilla.org/r/83090/#review81864
Attachment #8797462 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 102•8 years ago
|
||
mozreview-review |
Comment on attachment 8797463 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-import.
https://reviewboard.mozilla.org/r/83092/#review81866
Attachment #8797463 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 103•8 years ago
|
||
mozreview-review |
Comment on attachment 8797464 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-invalid.
https://reviewboard.mozilla.org/r/83094/#review81888
Ship It!
Attachment #8797464 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 104•8 years ago
|
||
mozreview-review |
Comment on attachment 8797465 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-mediaqueries.
https://reviewboard.mozilla.org/r/83096/#review81892
Ship It!
Attachment #8797465 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 105•8 years ago
|
||
mozreview-review |
Comment on attachment 8797466 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-selectors.
https://reviewboard.mozilla.org/r/83098/#review81894
Ship It!
Attachment #8797466 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 106•8 years ago
|
||
mozreview-review |
Comment on attachment 8797467 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-submit-invalid.
https://reviewboard.mozilla.org/r/83100/#review81896
Ship It!
Attachment #8797467 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 107•8 years ago
|
||
mozreview-review |
Comment on attachment 8797468 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-ui-invalid.
https://reviewboard.mozilla.org/r/83102/#review81898
Ship It!
Attachment #8797468 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 108•8 years ago
|
||
mozreview-review |
Comment on attachment 8797469 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-ui-valid.
https://reviewboard.mozilla.org/r/83104/#review81900
Ship It!
Attachment #8797469 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 109•8 years ago
|
||
mozreview-review |
Comment on attachment 8797470 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-valid.
https://reviewboard.mozilla.org/r/83106/#review81902
Ship It!
Attachment #8797470 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 110•8 years ago
|
||
mozreview-review |
Comment on attachment 8797471 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/css-valuesandunits.
https://reviewboard.mozilla.org/r/83108/#review81904
Ship It!
Attachment #8797471 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 111•8 years ago
|
||
mozreview-review |
Comment on attachment 8797472 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/cssom.
https://reviewboard.mozilla.org/r/83110/#review81906
Ship It!
Attachment #8797472 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 112•8 years ago
|
||
mozreview-review |
Comment on attachment 8797473 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/dom.
https://reviewboard.mozilla.org/r/83112/#review81908
Ship It!
Attachment #8797473 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 113•8 years ago
|
||
mozreview-review |
Comment on attachment 8797474 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/first-letter.
https://reviewboard.mozilla.org/r/83114/#review81910
r=me with the following issue addressed:
::: layout/reftests/first-letter/reftest.list:67
(Diff revision 1)
> fails-if(winWidget||cocoaWidget) == 617869-1.html 617869-1-ref.html
> == 723509-1.html 723509-1-ref.html
> == 922550-1.html 922550-1-ref.html
> == 958249.html 958249-ref.html
> == font-text-styles.html font-text-styles-ref.html
> -fails-if(gtkWidget&&!Mulet) random-if(winWidget&&!d2d) == font-text-styles-floater.html font-text-styles-floater-ref.html # bug 992846 # Initial mulet triage: parity with B2G/B2G Desktop
> +fails-if(gtkWidget) random-if(winWidget&&!d2d) == font-text-styles-floater.html font-text-styles-floater-ref.html
The "bug 992846" annotation needs to stay here -- don't remove that.
Attachment #8797474 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 114•8 years ago
|
||
mozreview-review |
Comment on attachment 8797475 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/flexbox.
https://reviewboard.mozilla.org/r/83116/#review81912
Ship It!
Attachment #8797475 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 115•8 years ago
|
||
mozreview-review |
Comment on attachment 8797476 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/font-face.
https://reviewboard.mozilla.org/r/83118/#review81914
Ship It!
Attachment #8797476 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 116•8 years ago
|
||
mozreview-review |
Comment on attachment 8797477 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/font-features.
https://reviewboard.mozilla.org/r/83120/#review81916
Ship It!
Attachment #8797477 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 117•8 years ago
|
||
mozreview-review |
Comment on attachment 8797478 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/font-inflation.
https://reviewboard.mozilla.org/r/83122/#review81918
Ship It!
Attachment #8797478 -
Flags: review?(dholbert) → review+
![]() |
Assignee | |
Comment 118•8 years ago
|
||
mozreview-review-reply |
Comment on attachment 8797455 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/bugs.
https://reviewboard.mozilla.org/r/83076/#review81850
That's intended because the tests which used these files only ran for B2G and don't get executed anymore, see https://reviewboard-hg.mozilla.org/gecko/rev/c9532f6e1ab8ddf14583f0fa5cf78dfc9bd4c561#l57.1179 Should they be kept in tree?
Reporter | ||
Comment 119•8 years ago
|
||
mozreview-review-reply |
Comment on attachment 8797455 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/bugs.
https://reviewboard.mozilla.org/r/83076/#review81850
I see, thanks. I just asked over on bug 1177334 (which is where we made these tests B2G-specific) -- let's see what happens over there.
Also: if we *are* removing the tests files here, this changeset's commit message should reflect that. Right now it's "Remove B2G and Mulet annotations from reftest.list: layout/reftests/bugs". That should probably say: "Remove B2G and Mulet annotations from reftest.list (and B2G-specific reftests): layout/reftests/bugs"
Reporter | ||
Comment 120•8 years ago
|
||
mozreview-review |
Comment on attachment 8797479 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/font-matching.
https://reviewboard.mozilla.org/r/83124/#review81920
r=me on this part with the following comment-nit:
::: layout/reftests/font-matching/reftest.list:75
(Diff revision 1)
> fuzzy-if(OSX==1010&&browserIsRemote,1,17) fuzzy-if(Android,4,8) HTTP(..) == stretchmapping-35.html stretchmapping-35-ref.html
> fuzzy-if(OSX==1010,3,5) HTTP(..) == stretchmapping-137.html stretchmapping-137-ref.html
>
> # test for font-stretch using @font-face
> -skip-if(B2G||Mulet) skip-if(Android) HTTP(..) == font-stretch-1.html font-stretch-1-ref.html # bugs 773482, 927602 # Initial mulet triage: parity with B2G/B2G Desktop
> -skip-if(B2G||Mulet) skip-if(Android) HTTP(..) == font-shorthand-stretch-1.html font-stretch-1-ref.html # bugs 773482, 927602 # Initial mulet triage: parity with B2G/B2G Desktop
> +skip-if(Android) HTTP(..) == font-stretch-1.html font-stretch-1-ref.html # 927602
> +skip-if(Android) HTTP(..) == font-shorthand-stretch-1.html font-stretch-1-ref.html # 927602
s/927602/bug 927602/ in the comment at the end of these two lines, please (for font-stretch and font-shorthand-stretch). Include the word "bug".
Attachment #8797479 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 121•8 years ago
|
||
mozreview-review-reply |
Comment on attachment 8797455 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/bugs.
https://reviewboard.mozilla.org/r/83076/#review81850
OK, kats says these tests might want to change to "skip-if(!Android)", though we're not sure that'll work (and it also might initially work but turn out to be flaky, perhaps.
My suggestion: could we leave these "skip-if(!B2G)" tests alone for the purposes of this bug, and spin off a followup-bug to deal with them? It might end up being as easy as a straight s/!B2G/!Android/ search-and-replace, but if there are unforseen compliations or regressions, it'd be nice to have a dedicated place to deal with them, rather than lumping it into a monster bug like this one.
Reporter | ||
Comment 122•8 years ago
|
||
mozreview-review |
Comment on attachment 8797480 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/forms.
https://reviewboard.mozilla.org/r/83126/#review81928
r=me on this part, with the following:
::: layout/reftests/forms/button/reftest.list:12
(Diff revision 1)
> == max-height.html max-height-ref.html
> == min-height.html min-height-ref.html
> == overflow-areas-1.html overflow-areas-1-ref.html
>
> -# The buttons in these tests have some fancy shading applied to their corners
> -# on B2G, despite their "-moz-appearance: none; background: gray", so they
> +# The buttons in these tests have some fancy shading applied to their corners,
> +# despite their "-moz-appearance: none; background: gray", so they
You're removing the "on B2G" qualifier from the comment here.
Instead of just removing it, please replace it with "on Android". (This comment still applies to Android -- in fact, it *only* applies to Android now.)
(Note: We really should have added a mention of Android to the comment here, back when we added Android to the fuzzy annotations, in https://hg.mozilla.org/integration/mozilla-inbound/rev/ac696b8baf85#l5.9 )
Attachment #8797480 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 123•8 years ago
|
||
mozreview-review |
Comment on attachment 8797481 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/generated-content.
https://reviewboard.mozilla.org/r/83128/#review81942
Ship It!
Attachment #8797481 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 124•8 years ago
|
||
mozreview-review |
Comment on attachment 8797482 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/image.
https://reviewboard.mozilla.org/r/83130/#review81946
Ship It!
Attachment #8797482 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 125•8 years ago
|
||
mozreview-review |
Comment on attachment 8797483 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/image-element.
https://reviewboard.mozilla.org/r/83132/#review81950
Ship It!
Attachment #8797483 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 126•8 years ago
|
||
mozreview-review |
Comment on attachment 8797484 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/image-rect.
https://reviewboard.mozilla.org/r/83134/#review81952
Ship It!
Attachment #8797484 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 127•8 years ago
|
||
mozreview-review |
Comment on attachment 8797485 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/image-region.
https://reviewboard.mozilla.org/r/83136/#review81954
Ship It!
Attachment #8797485 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 128•8 years ago
|
||
mozreview-review |
Comment on attachment 8797486 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/indic-shaping.
https://reviewboard.mozilla.org/r/83138/#review81956
Ship It!
Attachment #8797486 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 129•8 years ago
|
||
mozreview-review |
Comment on attachment 8797487 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/invalidation.
https://reviewboard.mozilla.org/r/83140/#review81958
r=me with trailing space fixed:
::: layout/reftests/invalidation/reftest.list:5
(Diff revision 1)
> == table-repaint-a.html table-repaint-a-ref.html
> == table-repaint-b.html table-repaint-b-ref.html
> == table-repaint-c.html table-repaint-c-ref.html
> == table-repaint-d.html table-repaint-d-ref.html
> -skip-if((B2G&&browserIsRemote)||Mulet) == 540247-1.xul 540247-1-ref.xul # bug 974780 # Initial mulet triage: parity with B2G/B2G Desktop
> +== 540247-1.xul 540247-1-ref.xul
Nit: drop whitespace at end of line.
(This probably got missed because there were two spaces before the comment instead of 1, and your regex only spliced out a single space.)
Attachment #8797487 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 130•8 years ago
|
||
mozreview-review |
Comment on attachment 8797488 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/line-breaking.
https://reviewboard.mozilla.org/r/83142/#review81966
Ship It!
Attachment #8797488 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 131•8 years ago
|
||
mozreview-review |
Comment on attachment 8797489 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/margin-collapsing.
https://reviewboard.mozilla.org/r/83144/#review81968
Ship It!
Attachment #8797489 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 132•8 years ago
|
||
mozreview-review |
Comment on attachment 8797490 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/marquee.
https://reviewboard.mozilla.org/r/83146/#review81970
Ship It!
Attachment #8797490 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 133•8 years ago
|
||
mozreview-review |
Comment on attachment 8797491 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/mathml.
https://reviewboard.mozilla.org/r/83148/#review81972
Ship It!
Attachment #8797491 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 134•8 years ago
|
||
mozreview-review |
Comment on attachment 8797492 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/mp4-video.
https://reviewboard.mozilla.org/r/83150/#review81976
Ship It!
Attachment #8797492 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 135•8 years ago
|
||
mozreview-review |
Comment on attachment 8797493 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/object.
https://reviewboard.mozilla.org/r/83152/#review81978
Ship It!
Attachment #8797493 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 136•8 years ago
|
||
mozreview-review |
Comment on attachment 8797494 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/ogg-video.
https://reviewboard.mozilla.org/r/83154/#review81982
Ship It!
Attachment #8797494 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 137•8 years ago
|
||
mozreview-review |
Comment on attachment 8797495 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/outline.
https://reviewboard.mozilla.org/r/83156/#review81984
Ship It!
Attachment #8797495 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 138•8 years ago
|
||
mozreview-review |
Comment on attachment 8797496 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/pagination.
https://reviewboard.mozilla.org/r/83158/#review81988
r=me with the following addressed:
::: layout/reftests/pagination/reftest.list:56
(Diff revision 1)
> == table-page-break-before-right-1.html table-page-break-before-auto-2-ref.html
> == table-page-break-after-always-1.html table-page-break-before-auto-2-ref.html
> == table-page-break-after-left-1.html table-page-break-before-auto-2-ref.html
> == table-page-break-after-right-1.html table-page-break-before-auto-2-ref.html
> -skip-if(B2G||Mulet) == rowgroup-page-break-after-always-1.html table-page-break-before-auto-2-ref.html # bug 773482 # Initial mulet triage: parity with B2G/B2G Desktop
> -skip-if(B2G||Mulet) == row-page-break-after-always-1.html table-page-break-before-auto-2-ref.html # bug 773482 # Initial mulet triage: parity with B2G/B2G Desktop
> +== rowgroup-page-break-after-always-1.html table-page-break-before-auto-2-ref.html # bug 773482
> +== row-page-break-after-always-1.html table-page-break-before-auto-2-ref.html # bug 773482
In this file, you've left behind "# bug 773482" comments (and that bug is "Tracking Bug to enable reftests on B2G", and is associated with the skip-if() annotation that you're removing).
So: please remove all these # bug 773482 mentions here.
Attachment #8797496 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 139•8 years ago
|
||
mozreview-review |
Comment on attachment 8797497 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/percent-overflow-sizing.
https://reviewboard.mozilla.org/r/83160/#review81990
Ship It!
Attachment #8797497 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 140•8 years ago
|
||
mozreview-review |
Comment on attachment 8797498 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/pixel-rounding.
https://reviewboard.mozilla.org/r/83162/#review81992
Ship It!
Attachment #8797498 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 141•8 years ago
|
||
mozreview-review |
Comment on attachment 8797499 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/position-dynamic-changes.
https://reviewboard.mozilla.org/r/83164/#review81994
Ship It!
Attachment #8797499 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 142•8 years ago
|
||
mozreview-review |
Comment on attachment 8797500 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/position-sticky.
https://reviewboard.mozilla.org/r/83166/#review81996
Ship It!
Attachment #8797500 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 143•8 years ago
|
||
mozreview-review |
Comment on attachment 8797501 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/printing.
https://reviewboard.mozilla.org/r/83168/#review81998
Ship It!
Attachment #8797501 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 144•8 years ago
|
||
mozreview-review |
Comment on attachment 8797502 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/reftest-sanity.
https://reviewboard.mozilla.org/r/83170/#review82000
Ship It!
Attachment #8797502 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 145•8 years ago
|
||
mozreview-review |
Comment on attachment 8797503 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/scrolling.
https://reviewboard.mozilla.org/r/83172/#review82004
Ship It!
Attachment #8797503 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 146•8 years ago
|
||
mozreview-review |
Comment on attachment 8797504 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/selection.
https://reviewboard.mozilla.org/r/83174/#review82008
Ship It!
Attachment #8797504 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 147•8 years ago
|
||
mozreview-review |
Comment on attachment 8797505 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/svg.
https://reviewboard.mozilla.org/r/83176/#review82012
r=me, with obsolete bug reference cleaned up:
::: layout/reftests/svg/moz-only/reftest.list:5
(Diff revision 1)
>
> # XBL tests
> -skip-if((B2G&&browserIsRemote)||Mulet) == xbl-basic-01.svg pass.svg # Initial mulet triage: parity with B2G/B2G Desktop
> -skip-if((B2G&&browserIsRemote)||Mulet) == xbl-basic-02.svg pass.svg # Initial mulet triage: parity with B2G/B2G Desktop
> -skip-if(B2G||Mulet) == xbl-basic-03.svg pass.svg # bug 773482 # Initial mulet triage: parity with B2G/B2G Desktop
> +== xbl-basic-01.svg pass.svg
> +== xbl-basic-02.svg pass.svg
> +== xbl-basic-03.svg pass.svg # bug 773482
Let's drop this straggling reference to bug 773482 here.
Attachment #8797505 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 148•8 years ago
|
||
mozreview-review |
Comment on attachment 8797506 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/table-background.
https://reviewboard.mozilla.org/r/83178/#review82022
Ship It!
Attachment #8797506 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 149•8 years ago
|
||
mozreview-review |
Comment on attachment 8797507 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/table-overflow.
https://reviewboard.mozilla.org/r/83180/#review82024
Ship It!
Attachment #8797507 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 150•8 years ago
|
||
mozreview-review |
Comment on attachment 8797508 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/table-width.
https://reviewboard.mozilla.org/r/83182/#review82028
Ship It!
Attachment #8797508 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 151•8 years ago
|
||
mozreview-review |
Comment on attachment 8797509 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/text.
https://reviewboard.mozilla.org/r/83184/#review82030
r=me, with one nit addressed:
::: layout/reftests/text/reftest.list:61
(Diff revision 1)
> # D2D/DirectWrite results depend on the rendering mode chosen, so considering this as random for now.
> skip-if(!(d2d||cocoaWidget)) random-if(d2d) != subpixel-glyphs-x-2a.html subpixel-glyphs-x-2b.html
> -skip-if(B2G||Mulet) HTTP(..) == subpixel-glyphs-x-3a.html subpixel-glyphs-x-3b.html # Initial mulet triage: parity with B2G/B2G Desktop
> +HTTP(..) == subpixel-glyphs-x-3a.html subpixel-glyphs-x-3b.html
> # No platforms do subpixel positioning vertically
> fuzzy-if(Android,19,2) == subpixel-glyphs-y-1a.html subpixel-glyphs-y-1b.html
> -fuzzy-if((Android||B2G),231,653) == subpixel-lineheight-1a.html subpixel-lineheight-1b.html
> +fuzzy-if((Android),231,653) == subpixel-lineheight-1a.html subpixel-lineheight-1b.html
Please drop straggling extra parens around (Android) here.
Attachment #8797509 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 152•8 years ago
|
||
mozreview-review |
Comment on attachment 8797510 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/text-decoration.
https://reviewboard.mozilla.org/r/83186/#review82032
Ship It!
Attachment #8797510 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 153•8 years ago
|
||
mozreview-review |
Comment on attachment 8797511 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/text-overflow.
https://reviewboard.mozilla.org/r/83188/#review82034
r=me, with 1 whitespace nit fixed:
::: layout/reftests/text-overflow/reftest.list:23
(Diff revision 1)
> == aligned-baseline.html aligned-baseline-ref.html
> -skip-if(Android||B2G) fuzzy-if(skiaContent,1,5) == clipped-elements.html clipped-elements-ref.html
> +skip-if(Android) fuzzy-if(skiaContent,1,5) == clipped-elements.html clipped-elements-ref.html
> HTTP(..) == theme-overflow.html theme-overflow-ref.html
> -skip-if(B2G||Mulet) HTTP(..) == table-cell.html table-cell-ref.html # Initial mulet triage: parity with B2G/B2G Desktop
> -skip-if(Mulet) fuzzy-if(gtkWidget,10,32) HTTP(..) == two-value-syntax.html two-value-syntax-ref.html # MULET: Bug 1144079: Re-enable Mulet mochitests and reftests taskcluster-specific disables
> -skip-if(B2G||Mulet) HTTP(..) == single-value.html single-value-ref.html # Initial mulet triage: parity with B2G/B2G Desktop
> +HTTP(..) == table-cell.html table-cell-ref.html
> +fuzzy-if(gtkWidget,10,32) HTTP(..) == two-value-syntax.html two-value-syntax-ref.html
> +HTTP(..) == single-value.html single-value-ref.html
Please drop the trailing space character on this line.
Attachment #8797511 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 154•8 years ago
|
||
mozreview-review |
Comment on attachment 8797512 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/text-shadow.
https://reviewboard.mozilla.org/r/83190/#review82038
Ship It!
Attachment #8797512 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 155•8 years ago
|
||
mozreview-review |
Comment on attachment 8797513 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/text-svgglyphs.
https://reviewboard.mozilla.org/r/83192/#review82042
Ship It!
Attachment #8797513 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 156•8 years ago
|
||
mozreview-review |
Comment on attachment 8797514 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/text-transform.
https://reviewboard.mozilla.org/r/83194/#review82044
Ship It!
::: layout/reftests/text-transform/reftest.list:21
(Diff revision 1)
> HTTP(..) != graphite-small-caps-1.html graphite-small-caps-1-notref.html
> == uppercase-1.html uppercase-ref.html
> == uppercase-szlig-1.html uppercase-szlig-ref.html
> # these use DejaVu Sans via @font-face for consistency of results
> -skip-if(B2G||Mulet) fuzzy-if(cocoaWidget,250,15) HTTP(..) == all-upper.html all-upper-ref.html # bug 773482, 1140292 # Initial mulet triage: parity with B2G/B2G Desktop
> -skip-if(B2G||Mulet) fuzzy-if(cocoaWidget,250,15) fuzzy-if(skiaContent,1,1800) HTTP(..) == all-lower.html all-lower-ref.html # bug 773482, 1140292 # Initial mulet triage: parity with B2G/B2G Desktop
> +fuzzy-if(cocoaWidget,250,15) HTTP(..) == all-upper.html all-upper-ref.html # 1140292
> +fuzzy-if(cocoaWidget,250,15) fuzzy-if(skiaContent,1,1800) HTTP(..) == all-lower.html all-lower-ref.html # 1140292
Please add the word "bug" to the comments on these two lines. ("bug 1140292")
Attachment #8797514 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 157•8 years ago
|
||
mozreview-review |
Comment on attachment 8797515 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/transform.
https://reviewboard.mozilla.org/r/83196/#review82046
Ship It!
Attachment #8797515 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 158•8 years ago
|
||
mozreview-review |
Comment on attachment 8797516 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/transform-3d.
https://reviewboard.mozilla.org/r/83198/#review82048
Ship It!
Attachment #8797516 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 159•8 years ago
|
||
mozreview-review |
Comment on attachment 8797517 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/reftest.list.
https://reviewboard.mozilla.org/r/83200/#review82066
r=me with one comment updated (or removed if necessary), per below:
::: layout/reftests/reftest.list:255
(Diff revision 1)
> # marquee/
> include marquee/reftest.list
>
> # native-theme/
> -# skipping for B2G since something around radio-nonnative.html makes the whole suite hang
> -skip-if(B2G||Android||Mulet) include native-theme/reftest.list # Initial mulet triage: parity with B2G/B2G Desktop
> +# skipping for Android since something around radio-nonnative.html makes the whole suite hang
> +skip-if(Android) include native-theme/reftest.list
I suspect the modified comment here ("skipping for android since...") is actually wrong. Based on hg blame, it looks like we disabled this on Android *separately* from when we disabled it on B2G, as part of some theming rewrite in bug 1223526. So the old B2G explanatory comment is unlikely to accurately explain why it's also disabled on Android.
SO: we should replace this comment with something more accurate. I asked mfinkle & gbrown over in bug 1223526 comment 33 what we should replace it with. (That the bug where we added Android to this skip-if annotation.)
(If we don't get a response over there right away & end up being blocked on this, I'd be OK landing this with the "skipping for..." comment removed entirely, and perhaps handling the Android annotation in a followup.)
Attachment #8797517 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 160•8 years ago
|
||
mozreview-review |
Comment on attachment 8797518 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/w3c-css.
https://reviewboard.mozilla.org/r/83202/#review82068
Ship It!
Attachment #8797518 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 161•8 years ago
|
||
mozreview-review |
Comment on attachment 8797519 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/webcomponents.
https://reviewboard.mozilla.org/r/83204/#review82070
Ship It!
Attachment #8797519 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 162•8 years ago
|
||
mozreview-review |
Comment on attachment 8797520 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/webm-video.
https://reviewboard.mozilla.org/r/83206/#review82074
Ship It!
Attachment #8797520 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 163•8 years ago
|
||
mozreview-review |
Comment on attachment 8797521 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/writing-mode.
https://reviewboard.mozilla.org/r/83208/#review82076
Ship It!
Attachment #8797521 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 164•8 years ago
|
||
mozreview-review |
Comment on attachment 8797522 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/xul.
https://reviewboard.mozilla.org/r/83210/#review82078
Ship It!
Attachment #8797522 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 165•8 years ago
|
||
mozreview-review |
Comment on attachment 8797523 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/xul-document-load.
https://reviewboard.mozilla.org/r/83212/#review82080
Ship It!
Attachment #8797523 -
Flags: review?(dholbert) → review+
Reporter | ||
Comment 166•8 years ago
|
||
mozreview-review-reply |
Comment on attachment 8797517 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/reftest.list.
https://reviewboard.mozilla.org/r/83200/#review82066
> I suspect the modified comment here ("skipping for android since...") is actually wrong. Based on hg blame, it looks like we disabled this on Android *separately* from when we disabled it on B2G, as part of some theming rewrite in bug 1223526. So the old B2G explanatory comment is unlikely to accurately explain why it's also disabled on Android.
>
> SO: we should replace this comment with something more accurate. I asked mfinkle & gbrown over in bug 1223526 comment 33 what we should replace it with. (That the bug where we added Android to this skip-if annotation.)
>
> (If we don't get a response over there right away & end up being blocked on this, I'd be OK landing this with the "skipping for..." comment removed entirely, and perhaps handling the Android annotation in a followup.)
OK -- from looking at the commit* that made this "skip-if(Android)", it looks like the accurate explanatory comment here would be something like:
# (no XUL theme on Android)
So: Please replace the patch's current comment here ("Skipping for Android since something around radio-nonnative.html makes the whole suite hang") with that.
* https://hg.mozilla.org/integration/fx-team/rev/67a9dd1470e7#l3.36
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Reporter | ||
Comment 246•8 years ago
|
||
mozreview-review |
Comment on attachment 8797455 [details]
Bug 1307332 - Remove B2G and Mulet annotations from reftest.list: layout/reftests/bugs.
https://reviewboard.mozilla.org/r/83076/#review83006
Thanks again for taking this on! This is awesome.
r=me, just a few bug number comments to be removed (which only exist to explain the b2g annotations that you're removing in this patch).
::: layout/reftests/bugs/reftest.list:320
(Diff revision 2)
> == 296361-1.html 296361-ref.html
> == 296904-1.html 296904-1-ref.html
> -skip-if(B2G||Mulet) == 299136-1.html 299136-1-ref.html # Initial mulet triage: parity with B2G/B2G Desktop
> +== 299136-1.html 299136-1-ref.html
> == 299837-1.html 299837-1-ref.html
> -skip-if((B2G&&browserIsRemote)||Mulet) == 299837-2.xul 299837-2-ref.xul # bug 974780 # Initial mulet triage: parity with B2G/B2G Desktop
> -random-if(d2d) skip-if((B2G&&browserIsRemote)||Mulet) == 299837-3.xul 299837-3-ref.xul # bug 587631, 974780 # Initial mulet triage: parity with B2G/B2G Desktop
> +== 299837-2.xul 299837-2-ref.xul
> +random-if(d2d) == 299837-3.xul 299837-3-ref.xul # bug 587631, 974780
Drop ", 974780" from the end of this line.
::: layout/reftests/bugs/reftest.list:640
(Diff revision 2)
> == 370525-rowspan-2b.html 370525-rowspan-2b-ref.html
> == 370525-rowspan-3.html 370525-rowspan-3-ref.html
> == 370525-rowspan-4.html 370525-rowspan-4-ref.html
> == 370525-sib.html 370525-sib-ref.html
> == 370586-1.xhtml 370586-1-ref.xhtml
> -skip-if(B2G||Mulet) == 370629-1.html 370629-1-ref.html # bug 1060869 # Initial mulet triage: parity with B2G/B2G Desktop
> +== 370629-1.html 370629-1-ref.html # bug 1060869
Drop the "bug 1060869" bug number comment here.
That bug number was added at the same time as the annotation, here:
https://hg.mozilla.org/mozilla-central/rev/1cbca597b025#l6.12
...so it should be removed with the annotation.
::: layout/reftests/bugs/reftest.list:775
(Diff revision 2)
> == 389074-1.html 389074-1-ref.html
> == 389224-1.html 389224-1-ref.html
> == 389224-2.html about:blank
> -skip-if(B2G||Mulet) == 389468-1.html 389468-1-ref.html # Initial mulet triage: parity with B2G/B2G Desktop
> +== 389468-1.html 389468-1-ref.html
> == 389623-1.html 389623-1-ref.html
> -skip-if((B2G&&browserIsRemote)||Mulet) == 389636-1.html about:blank # assertion test # bug 975911 # Initial mulet triage: parity with B2G/B2G Desktop
> +== 389636-1.html about:blank # assertion test # bug 975911
Please drop the "bug 975911" comment at the end of this line.
::: layout/reftests/bugs/reftest.list:810
(Diff revision 2)
> == 393671-3.html 393671-3-ref.html
> == 393760-1.xml 393760-1-ref.xml
> fuzzy-if(skiaContent,1,500) == 393760-2.xml 393760-2-ref.xml
> == 394111-1.html about:blank # Really an assertion test rather than a rendering test
> == 394534-1.html 394534-1-ref.html
> -skip-if((B2G&&browserIsRemote)||Mulet) == 394676-1.xhtml 394676-1-ref.xhtml # bug 975911 # Initial mulet triage: parity with B2G/B2G Desktop
> +== 394676-1.xhtml 394676-1-ref.xhtml # bug 975911
As above, drop the comment about bug 975911 here.
Attachment #8797455 -
Flags: review?(dholbert) → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Description
•