1346893 - Move mobile/android/themes/core/content.css to layout/style/res/

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement, P3)

Description

[Mats Palmgren (inactive)]

It seems to me that mobile/android/themes/core/content.css is effectively
an UA style sheet since it has a lot of styles that affect HTML web content.
As such, I tend to think it should live under layout/style/res/ so that
changes to it are reviewed under the same rules as UA style sheet changes
on other platforms.  It's rather confusing to have some rules for say
HTML checkbox/radio controls live in layout/style/res/forms.css (there's
even a %ifdef Android section in that file) and then have additional
rules for the same controls under mobile/...

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1346893 - Move the Geckoview UA sheet to layout/style. r=snorp

Make it a regular stylesheet.

This allows it to be cached in shared memory if possible, and will allow me to
stop adding the stylesheet based on a pref for bug 1618202.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8c1c55d16e1a
Move the Geckoview UA sheet to layout/style. r=snorp

Comment 3

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fd054e762cfa
Backed out changeset 8c1c55d16e1a for bc failures on browser_all_files_referenced.js.

Comment 4

[Cosmin Sabou [:CosminS]]

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=8c1c55d16e1ad18934cff6bed55e5329bb7bb81b&selectedJob=290624130

Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=290624130&repo=autoland

Backout link: https://hg.mozilla.org/integration/autoland/rev/fd054e762cfa002027014764b6292012931e30a7

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/830880e1c188
Move the Geckoview UA sheet to layout/style. r=snorp

Comment 6

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/830880e1c188