1413166 - Add gtk-csd tokens to configure titlebar drawing on Gtk+

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P2)

Comment 1

[Martin Stránský [:stransky] (ni? me)]

We need to configure titlebar rendering on Linux/Gtk+.

Comment 2

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1413166 - Add gtk-csd atoms to configure rendering to titlebar on Linux/Gtk+,

Based on patch by Andrew Comminos [:acomminos] <[e-mail]>

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

Comment 3

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1413166 - Bind gtk_csd* atoms with LookAndFeel,

Based on patch by Andrew Comminos [:acomminos] <[e-mail]>

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

Comment 4

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1413166 - Export gtk_csd* atoms as media features,

Based on patch by Andrew Comminos [:acomminos] <[e-mail]>

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

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8923815 [details]
Bug 1413166 - Add gtk-csd atoms to configure rendering to titlebar on Linux/Gtk+,

https://reviewboard.mozilla.org/r/194960/#review200228

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8923816 [details]
Bug 1413166 - Bind gtk_csd* atoms with LookAndFeel,

https://reviewboard.mozilla.org/r/194962/#review200230

Comment 7

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8923817 [details]
Bug 1413166 - Export gtk_csd* atoms as media features,

https://reviewboard.mozilla.org/r/194964/#review200234

This looks fine, but needs tests that ensure that we parse these new media features properly...

Comment 8

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1413166 - Added parsing tests for moz-gtk-csd-* media features,

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

Comment 9

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8924140 [details]
Bug 1413166 - Added parsing tests for moz-gtk-csd-* media features,

https://reviewboard.mozilla.org/r/195392/#review200784

Ugh.  I didn't realize our coverage for this stuff got nerfed in bug 1410074.  :(

Thank you for adding these tests; I'll follow up with Emilio about adding real parseability tests here.

Comment 10

[Emilio Cobos Álvarez (:emilio)]

Given these are toggles, it shouldn't be hard to add to:

  http://searchfox.org/mozilla-central/rev/423b2522c48e1d654e30ffc337164d677f934ec3/layout/style/test/chrome/bug418986-2.js#41

To test that at least one of the possible values matches, and that resistFingerprinting works properly with them.

Comment 11

[Martin Stránský [:stransky] (ni? me)]

Attachment: Bug 1413166 - Added moz-gtk-csd-* media features to resistFingerprinting tests,

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

Comment 12

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8924502 [details]
Bug 1413166 - Added moz-gtk-csd-* media features to resistFingerprinting tests,

https://reviewboard.mozilla.org/r/195776/#review201054

Comment 13

[Martin Stránský [:stransky] (ni? me)]

Thanks!

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3b8671ab0719
Add gtk-csd atoms to configure rendering to titlebar on Linux/Gtk+, r=bz
https://hg.mozilla.org/integration/autoland/rev/9e5f5924dad8
Bind gtk_csd* atoms with LookAndFeel, r=bz
https://hg.mozilla.org/integration/autoland/rev/16fd21f5ad8c
Export gtk_csd* atoms as media features, r=bz
https://hg.mozilla.org/integration/autoland/rev/e1ef2adcc815
Added parsing tests for moz-gtk-csd-* media features, r=bz
https://hg.mozilla.org/integration/autoland/rev/b39bf4bf8127
Added moz-gtk-csd-* media features to resistFingerprinting tests, r=bz

Comment 15

[Attila Craciun [:arny]]

Backed out 5 changesets (bug 1413166) for failing valgrind-test at Conditional jump or move depends on uninitialised value(s) at nsCSSRuleProcessor::InitSystemMetrics / InitSystemMetrics / nsCSSRuleProcessor::HasSystemMetric / GetSystemMetric

https://hg.mozilla.org/integration/autoland/rev/f62bd9c2b794f18f7a0eea222a592fa53d191258

https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=b39bf4bf81272a623d97ff797d68135ac21c6ac6&filter-classifiedState=unclassified&filter-resultStatus=testfailed&filter-resultStatus=busted&filter-resultStatus=exception

https://treeherder.mozilla.org/logviewer.html#?job_id=141921820&repo=autoland&lineNumber=34355

Comment 16

[Martin Stránský [:stransky] (ni? me)]

Will look at it, Thanks.

Comment 17

[Emilio Cobos Álvarez (:emilio)]

Just for reference, you can now also add the media features to layout/style/test/chrome/test_chrome_only_media_queries.html, for a bit more complete testing.

Comment 18

[Martin Stránský [:stransky] (ni? me)]

(In reply to Attila Craciun [:arny] from comment #15)
> Backed out 5 changesets (bug 1413166) for failing valgrind-test at
> Conditional jump or move depends on uninitialised value(s) at
> nsCSSRuleProcessor::InitSystemMetrics / InitSystemMetrics /
> nsCSSRuleProcessor::HasSystemMetric / GetSystemMetric
> 
> https://hg.mozilla.org/integration/autoland/rev/
> f62bd9c2b794f18f7a0eea222a592fa53d191258
> 
> https://treeherder.mozilla.org/#/
> jobs?repo=autoland&revision=b39bf4bf81272a623d97ff797d68135ac21c6ac6&filter-
> classifiedState=unclassified&filter-resultStatus=testfailed&filter-
> resultStatus=busted&filter-resultStatus=exception
> 
> https://treeherder.mozilla.org/logviewer.
> html#?job_id=141921820&repo=autoland&lineNumber=34355

Filed as Bug 1414774

Comment 19

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ac985ab9e042
Add gtk-csd atoms to configure rendering to titlebar on Linux/Gtk+, r=bz
https://hg.mozilla.org/integration/autoland/rev/a319c7bef811
Bind gtk_csd* atoms with LookAndFeel, r=bz
https://hg.mozilla.org/integration/autoland/rev/7f8a59a04f19
Export gtk_csd* atoms as media features, r=bz
https://hg.mozilla.org/integration/autoland/rev/df4b8747adde
Added parsing tests for moz-gtk-csd-* media features, r=bz
https://hg.mozilla.org/integration/autoland/rev/c7dadba6ceee
Added moz-gtk-csd-* media features to resistFingerprinting tests, r=bz

Comment 20

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/ac985ab9e042
https://hg.mozilla.org/mozilla-central/rev/a319c7bef811
https://hg.mozilla.org/mozilla-central/rev/7f8a59a04f19
https://hg.mozilla.org/mozilla-central/rev/df4b8747adde
https://hg.mozilla.org/mozilla-central/rev/c7dadba6ceee