632408 - [SeaMonkey, Windows] mochitest-chrome: permanent "test_leaf_layers_partition_browser_window.xul | Leaf layers should form a non-overlapping partition of the browser window" failure, since landing

Status: RESOLVED WORKSFORME

(Core :: Layout, defect)

Description

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1297172956.1297177331.6991.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2011/02/08 05:49:16
{
...
Document data:text/html,<html%20style='overflow:scroll'><script>opener.doTest(window)</script> loaded successfully
7408 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/layout/base/test/chrome/test_leaf_layers_partition_browser_window.xul | Leaf layers should form a non-overlapping partition of the browser window
}

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Improve test code somewhat [Checked in: Comment 2]

I can't reproduce on my local Windows 2000 with an opt build.

Yet, this patch should at least help to know what is going on, if not fixing the (timing?) issue.

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 510642 [details] [diff] [review]
(Av1) Improve test code somewhat
[Checked in: Comment 2]

http://hg.mozilla.org/mozilla-central/rev/8e550551c79c

Comment 3

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1297263431.1297267366.30508.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2011/02/09 06:57:11
{
WARNING: Nv3DVStreaming CoCreateInstance failed (disabled).: file e:/builds/slave/comm-cen-trunk-w32-dbg/build/mozilla/gfx/layers/d3d9/Nv3DVUtils.cpp, line 85
WARNING: Direct3D 9-accelerated layers are not supported on this system.: file e:/builds/slave/comm-cen-trunk-w32-dbg/build/mozilla/gfx/layers/d3d9/LayerManagerD3D9.cpp, line 89
...
7408 INFO TEST-PASS | chrome://mochitests/content/chrome/layout/base/test/chrome/test_leaf_layers_partition_browser_window.xul | Waiting for mozPaintCount to increase. - 0 should equal 0
Document data:text/html,<html%20style='overflow:scroll'><script>opener.doTest(window);</script> loaded successfully
7409 INFO TEST-PASS | chrome://mochitests/content/chrome/layout/base/test/chrome/test_leaf_layers_partition_browser_window.xul | mozPaintCount has increased - 1 should not equal 0
7410 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/layout/base/test/chrome/test_leaf_layers_partition_browser_window.xul | Leaf layers should form a non-overlapping partition of the browser window
}

Unless this test would somehow depend on D3D9, I have no clue :-(

Comment 4

[Timothy Nikkel (:tnikkel)]

You can't reproduce in opt builds because the dom window utils function it depends on always returns true in an opt build.

Comment 5

[Timothy Nikkel (:tnikkel)]

If you set the environment variable MOZ_DUMP_PAINT_LIST to 1 you will get a lot of paint debug output including a dump of the layer tree. The layer tree dump would probably useful in determining why this is failing.

Comment 6

[Serge Gautherie (:sgautherie)]

(Official debug builds can't run on Windows 2000 anymore.)

Callek, could you help with comment 5? (Or do you know someone who might?)

Comment 7

[Legacy TBPL/Treeherder Robot]

mounir.lamouri%gmail.com
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1299774157.1299777108.3542.gz
Rev3 WINNT 5.1 mozilla-central debug test mochitest-other on 2011/03/10 08:22:37

s: talos-r3-xp-020
7558 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/layout/base/test/chrome/test_leaf_layers_partition_browser_window.xul | Leaf layers should form a non-overlapping partition of the browser window
PROCESS-CRASH | Main app process exited normally | application crashed (minidump found)
Thread 0 (crashed)
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/dom/tests/browser/browser_focus_steal_from_chrome.js | Exited with code -1073741819 during test run
PROCESS-CRASH | chrome://mochitests/content/browser/dom/tests/browser/browser_focus_steal_from_chrome.js | application crashed (minidump found)
Thread 0 (crashed)
TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | missing output line for total leaks!
TEST-UNEXPECTED-FAIL | plugin process 3160 | automationutils.processLeakLog() | missing output line for total leaks!

Comment 8

[Armen [:armenzg]]

Also see bug 640889.
We enabled debug unit tests on XP yesterday EST morning.

Comment 9

[Legacy TBPL/Treeherder Robot]

philor
https://tbpl.mozilla.org/php/getParsedLog.php?id=6296482
Rev3 Fedora 12 mozilla-inbound debug test mochitest-other on 2011-09-06 12:35:00

10426 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/layout/base/test/chrome/test_default_background.xul | Rendering OK, got data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAMYAAABiCAYAAAALFqylAAAAYklEQVR4nO3BMQEAAADCoPVPbQdvoAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAPgML6EAAWVdEKAAAAAASUVORK5CYII=, expected data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAMYAAABiCAYAAAALFqylAAABGElEQVR4nO3ToRHAIADAQIZhuA7HjqB7Fw/ixfuYjPmtDfyN2wHwImNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGBGNAMAYEY0AwBgRjQDAGhAPMQeqe70qscwAAAABJRU5ErkJggg==
10430 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/layout/base/test/chrome/test_leaf_layers_partition_browser_window.xul | Leaf layers should form a non-overlapping partition of the browser window (non-maximized)
10432 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/layout/base/test/chrome/test_leaf_layers_partition_browser_window.xul | Leaf layers should form a non-overlapping partition of the browser window (maximized)

Comment 10

[Phil Ringnalda (:philor)]

Oops, ignore comment 9, an errant click that I thought I'd unclicked.

Comment 11

[Serge Gautherie (:sgautherie)]

(In reply to Timothy Nikkel (:tn) from comment #5)
> If you set the environment variable MOZ_DUMP_PAINT_LIST to 1 you will get a
> lot of paint debug output including a dump of the layer tree.

Thanks for the hint!

I tried this on Windows Vista:
*maximized succeeds: 17x17 area is gray without a resizer.
*non-maximized fails: 17x17 area has 2 "layers", one of which is a resizer.


(In reply to Armen Zambrano G. [:armenzg] - Release Engineer from comment #8)
> Also see bug 640889.
> We enabled debug unit tests on XP yesterday EST morning.

SeaMonkey fails on Windows 2003:
{
12063 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/layout/base/test/chrome/test_leaf_layers_partition_browser_window.xul | Leaf layers should form a non-overlapping partition of the browser window (maximized)
}


Neil, would these be related to SeaMonkey css, like not supporting Windows Aero theme, or the like?

Comment 12

[Serge Gautherie (:sgautherie)]

(In reply to Serge Gautherie (:sgautherie) from comment #11)
> SeaMonkey fails on Windows 2003:
> {
> 12063 ERROR TEST-UNEXPECTED-FAIL |
> chrome://mochitests/content/chrome/layout/base/test/chrome/
> test_leaf_layers_partition_browser_window.xul | Leaf layers should form a
> non-overlapping partition of the browser window (maximized)
> }

[Mozilla/5.0 (Windows NT 5.0; rv:12.0a1) Gecko/20120126 Firefox/12.0a1 SeaMonkey/2.9a1] (custom debug build for Windows 2000)

The issue is that the "menubar + toolbar" layer has the same (full) height as the viewport instead of its own size.
Ftr, the "content area" layer seems fine.

Neil, would this be related to SeaMonkey css or the like?
NB: Robert, this looks like very bug 625288 :-/

Comment 13

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) Improve newer test code some more [Checked in: Comment 15]

NB: This undoes bug 649012 which left this test with inconsistent comment and code.

Comment 14

[[e-mail]]

I have no idea how this test works, it fails on my Firefox build too.

Comment 15

[Serge Gautherie (:sgautherie)]

Comment on attachment 623754 [details] [diff] [review]
(Bv1) Improve newer test code some more
[Checked in: Comment 15]

https://hg.mozilla.org/mozilla-central/rev/3240475004f4


Succeeded as
https://tbpl.mozilla.org/?tree=Try&rev=0971c3f91419

Comment 16

[BugBot [:suhaib / :marco/ :calixte]]

In the process of migrating remaining bugs to the new severity system, the severity for this bug cannot be automatically determined. Please retriage this bug using the new severity system.

Comment 17

[Daniel Holbert [:dholbert]]

This test (and the functionality that it was testing) was renamed from .xul to .xhtml, and then later removed in bug 1728914, specifically this commit:
https://hg.mozilla.org/mozilla-central/rev/f5e32ea819d3

So: nothing left to do here.