1721910 - xul.css is used on android and imports tooltip.css which is not available there

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, defect)

Description

[:Gijs (he/him)]

In bug 1721627 I'm experimenting with making tests fail if we request internal resources that aren't present. The idea is to prevent patches from landing if they mis-spell (or forget to hg add, or forget to include in jar.mn or similar) subresources like CSS files and images, by failing if we hit chrome: or resource: URLs in tests that are not available (ie where the internal jar/file code cannot find the relevant file).

The android jobs fail because they include xul.css somewhere, and that includes chrome://global/skin/tooltip.css (see https://searchfox.org/mozilla-central/rev/2022d34edd5817948b27f093cd203ae6adb18a25/toolkit/content/xul.css#19 ), which we don't ship on android.

See e.g. this job on try, which has this logcat which says:

07-22 13:38:37.006  5013  5028 I Gecko   : Missing file: chrome://global/skin/tooltip.css
07-22 13:38:37.006  5013  5028 F MOZ_CRASH: Hit MOZ_CRASH(Shouldn't request non-existing chrome or resource URLs!) at /builds/worker/checkouts/gecko/netwerk/base/nsNetUtil.cpp:3329

Presumably we should do one or more of:

1. stop including/loading xul.css on android (why is that even happening?)
2. ifdef out the tooltip.css include in xul.css so it's not there on android
3. include the linux tooltip.css on android
4. write a custom tooltip.css file for android (though it's not clear to me how many android devices would have pointing devices so that we care about the tooltips anywhere...)

:agi, can you help clarify what should happen here?

Comment 1

[[ex-Mozilla] Agi Sferro | :agi]

I would rather copy linux's tooltip.css for now. I can see some stuff in xul.css might be used in some about: pages that we have too. In general I would hope we could move any platform-specific stuff in xul.css to somewhere in browser.

Comment 2

[:Gijs (he/him)]

bug 1724718 is unearthing more stuff so I'm gonna take a stab at just fixing this.

Comment 3

[:Gijs (he/him)]

Try: https://treeherder.mozilla.org/jobs?repo=try&revision=757d2fe1cf4087f9a8ad6b7a3a1d270f24a814be

Patch preview: https://hg.mozilla.org/try/rev/8730c901f92a9df91b3f968d10c5f853bc5ccd4b

Comment 4

[:Gijs (he/him)]

Attachment: Bug 1721910 - make it obvious that android needs some toolkit theming, and reduce duplication there, r?agi!,jaws!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0591d05f4fa0
make it obvious that android needs some toolkit theming, and reduce duplication there, r=agi,desktop-theme-reviewers,dao

Comment 6

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/0591d05f4fa0