1429943 - 1,800 instances of "HeadlessLookAndFeel::GetIntImpl called with an unrecognized aID" emitted from widget/headless/HeadlessLookAndFeelGTK.cpp during linux64 debug testing

Status: RESOLVED FIXED

(Firefox :: Headless, defect)

Description

[Eric Rahm [:erahm]]

> 1776 WARNING: HeadlessLookAndFeel::GetIntImpl called with an unrecognized aID: file widget/headless/HeadlessLookAndFeelGTK.cpp, line 282

This warning [1] shows up in the following test suites:

>    388 - test-linux64/debug-marionette-headless-e10s MnH
>    240 - test-linux64/debug-mochitest-plain-headless-e10s-15 h15
>     88 - test-linux64/debug-mochitest-plain-headless-e10s-8 h8
>     80 - test-linux64/debug-mochitest-plain-headless-e10s-1 h1
>     80 - test-linux64/debug-mochitest-plain-headless-e10s-9 h9
>     80 - test-linux64/debug-mochitest-plain-headless-e10s-4 h4
>     80 - test-linux64/debug-mochitest-plain-headless-e10s-5 h5
>     80 - test-linux64/debug-mochitest-plain-headless-e10s-10 h10
>     72 - test-linux64/debug-mochitest-plain-headless-e10s-7 h7
>     72 - test-linux64/debug-mochitest-plain-headless-e10s-3 h3
>     72 - test-linux64/debug-mochitest-chrome-1 c1
>     72 - test-linux64/debug-mochitest-plain-headless-e10s-12 h12
>     72 - test-linux64/debug-mochitest-plain-headless-e10s-11 h11
>     64 - test-linux64/debug-mochitest-plain-headless-e10s-13 h13
>     64 - test-linux64/debug-mochitest-plain-headless-e10s-16 h16
>     64 - test-linux64/debug-mochitest-plain-headless-e10s-6 h6
>     64 - test-linux64/debug-mochitest-plain-headless-e10s-14 h14
>     44 - test-linux64/debug-mochitest-plain-headless-e10s-2 h2

It shows up in 78 tests. A few of the most prevalent:

>   1056 - [e10s] Shutdown
>     72 -        browser/components/shell/test/test_headless_screenshot.html
>     56 - [e10s] js/xpconnect/tests/marionette/test_loader_global_sharing.py TestLoaderGlobalSharing.test_global_sharing_settings
>     28 - [e10s] browser/components/migration/tests/marionette/test_refresh_firefox.py TestFirefoxRefresh.testReset
>     16 - [e10s] testing/marionette/harness/marionette_harness/tests/unit/test_navigation.py TestBackForwardNavigation.test_file_url
>     16 - [e10s] testing/marionette/harness/marionette_harness/tests/unit/test_cli_arguments.py TestCommandLineArguments.test_start_in_safe_mode
>     16 - [e10s] testing/marionette/harness/marionette_harness/tests/unit/test_profile_management.py TestProfileManagement.test_clean_profile
>     16 - [e10s] testing/marionette/harness/marionette_harness/tests/unit/test_profile_management.py TestProfileManagement.test_change_preference
>     16 - [e10s] testing/marionette/harness/marionette_harness/tests/unit/test_cli_arguments.py TestCommandLineArguments.test_startup_timeout
>     16 - [e10s] testing/marionette/harness/marionette_harness/tests/unit/test_timeouts.py TestTimeouts.test_reset_timeout

[1] https://hg.mozilla.org/mozilla-central/annotate/2cb19d6666db/widget/headless/HeadlessLookAndFeelGTK.cpp#l282

Comment 1

[Eric Rahm [:erahm]]

Brendan can you take a look? Seems like this might be a legit issue.

Comment 2

[Brendan Dahl [:bdahl]]

Attachment: Bug 1429943 - Define more look and feel values for headless GTK.

Values taken from running Firefox on Ubuntu with GTK.

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

Comment 3

[Brendan Dahl [:bdahl]]

Comment on attachment 8942260 [details]
Bug 1429943 - Define more look and feel values for headless GTK.

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

Comment 4

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 8942260 [details]
Bug 1429943 - Define more look and feel values for headless GTK.

https://reviewboard.mozilla.org/r/212540/#review224386

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8d628d91d882
Define more look and feel values for headless GTK. r=jrmuizel

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/8d628d91d882