1361645 - stylo: Add a styloVsGecko flag for the compareStyloToGecko test

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement)

Description

[Shing Lyu [:shinglyu]]

As dbaron suggested in Bug 1344132 comment 22, we can create a flag for styloVsGecko, so the flagss now become:

stylo => the linux64-stylo platform
styloVsGecko => linux64-stylo AND reftest.compareStyloToGecko=true

This will give us finer-grained control on the test expectation and better readability in reftest.lists

Comment 1

[Shing Lyu [:shinglyu]]

Attachment: Bug 1361645 - Added the styloVsGecko reftest condition

Review commit: https://reviewboard.mozilla.org/r/135740/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/135740/

Comment 2

[Shing Lyu [:shinglyu]]

Attachment: Bug 1361645 - Change reftest condition stylo to styloVsGecko

Review commit: https://reviewboard.mozilla.org/r/135742/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/135742/

Comment 3

[Shing Lyu [:shinglyu]]

Comment on attachment 8864028 [details]
Bug 1361645 - Added the styloVsGecko reftest condition

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/135740/diff/1-2/

Comment 4

[Shing Lyu [:shinglyu]]

Comment on attachment 8864029 [details]
Bug 1361645 - Change reftest condition stylo to styloVsGecko

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/135742/diff/1-2/

Comment 5

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Comment on attachment 8864028 [details]
Bug 1361645 - Added the styloVsGecko reftest condition

https://reviewboard.mozilla.org/r/135740/#review140220

r=dbaron with the following comment

::: layout/tools/reftest/reftest.jsm:717
(Diff revision 2)
>      sandbox.webrtc = false;
>  #endif
>  
>  #ifdef MOZ_STYLO
>      sandbox.stylo = true;
> +    sandbox.styloVsGecko = gCompareStyloToGecko == true;

Drop the "== true".  (If there were a real concern that it might not be a boolean, use "!!" -- but I don't think there is.)

Comment 6

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Comment on attachment 8864029 [details]
Bug 1361645 - Change reftest condition stylo to styloVsGecko

https://reviewboard.mozilla.org/r/135742/#review140222

r=dbaron, although independently, I'm a little concerned by the errors Xidorn is fixing in bug 1362255

Comment 7

[Shing Lyu [:shinglyu]]

Comment on attachment 8864028 [details]
Bug 1361645 - Added the styloVsGecko reftest condition

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/135740/diff/2-3/

Comment 8

[Shing Lyu [:shinglyu]]

Comment on attachment 8864029 [details]
Bug 1361645 - Change reftest condition stylo to styloVsGecko

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/135742/diff/2-3/

Comment 9

[Mozilla Autoland]

We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s 6c4f2fa26f03 -d 5ae2cc64391c: rebasing 398221:6c4f2fa26f03 "Bug 1361645 - Added the styloVsGecko reftest condition r=dbaron"
rebasing 398222:0073754821e2 "Bug 1361645 - Change reftest condition stylo to styloVsGecko r=dbaron" (tip)
merging layout/reftests/backgrounds/reftest.list
merging layout/reftests/bugs/reftest.list
merging layout/reftests/css-placeholder/input/reftest.list
merging layout/reftests/forms/input/number/reftest.list
warning: conflicts while merging layout/reftests/bugs/reftest.list! (edit, then use 'hg resolve --mark')
warning: conflicts while merging layout/reftests/css-placeholder/input/reftest.list! (edit, then use 'hg resolve --mark')
warning: conflicts while merging layout/reftests/forms/input/number/reftest.list! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)

Comment 10

[Shing Lyu [:shinglyu]]

Comment on attachment 8864028 [details]
Bug 1361645 - Added the styloVsGecko reftest condition

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/135740/diff/3-4/

Comment 11

[Shing Lyu [:shinglyu]]

Comment on attachment 8864029 [details]
Bug 1361645 - Change reftest condition stylo to styloVsGecko

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/135742/diff/3-4/

Comment 12

[Shing Lyu [:shinglyu]]

Comment on attachment 8864029 [details]
Bug 1361645 - Change reftest condition stylo to styloVsGecko

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/135742/diff/4-5/

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b055c4e40da5
Added the styloVsGecko reftest condition r=dbaron
https://hg.mozilla.org/integration/autoland/rev/2ad8dd54b5f5
Change reftest condition stylo to styloVsGecko r=dbaron

Comment 14

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b055c4e40da5
https://hg.mozilla.org/mozilla-central/rev/2ad8dd54b5f5