Closed Bug 1020038 Opened 5 years ago Closed 5 years ago

Intermittent browser_layoutview_update-in-iframes.js | - Got 100x100, expected 200x100

Categories

(DevTools :: General, defect)

31 Branch
x86_64
Windows 7
defect
Not set

Tracking

(firefox31 unaffected, firefox32 wontfix, firefox33 wontfix, firefox34 disabled, firefox35 disabled, firefox36 disabled, firefox-esr24 unaffected)

RESOLVED FIXED
Firefox 36
Tracking Status
firefox31 --- unaffected
firefox32 --- wontfix
firefox33 --- wontfix
firefox34 --- disabled
firefox35 --- disabled
firefox36 --- disabled
firefox-esr24 --- unaffected

People

(Reporter: KWierso, Assigned: pbro)

References

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

https://tbpl.mozilla.org/php/getParsedLog.php?id=40992664&tree=Mozilla-Inbound
Windows 7 32-bit mozilla-inbound opt test mochitest-devtools-chrome on 2014-06-03 16:16:02 PDT for push 9a6bafdf9a28

slave: t-w732-ix-022



16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'padding-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "padding-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'padding-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "padding-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found '-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found '-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'auto'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "auto"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'auto'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "auto"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'auto'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "auto"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'auto'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "auto"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'scroll'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "scroll"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'scroll'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "scroll"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'border-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "border-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'border-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "border-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found '-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found '-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'none'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "none"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'none'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "none"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'padding-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "padding-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'padding-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "padding-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found '-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found '-box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "-box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'box'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "box"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'repeat'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "repeat"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'repeat'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "repeat"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'auto'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "auto"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'auto'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "auto"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'auto'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "auto"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "Expected color but found 'auto'.  Error in parsing value for 'color'.  Declaration dropped." {file: "resource://gre-resources/hiddenWindow.html" line: 0 column: 0 source: "auto"}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Console message: [JavaScript Warning: "XUL box for span element contained an inline #text child, forcing all its children to be wrapped in a block." {file: "chrome://browser/content/devtools/layoutview/view.xhtml" line: 0}]
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Checking that the layout-view shows the right value
16:28:08     INFO -  TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js |
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Listening for layout-view changes and modifying its size
16:28:08     INFO -  TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Layout-view got updated
16:28:08     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | Checking that the layout-view shows the right value after update
16:28:08  WARNING -  TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js |  - Got 100x100, expected 200x100
16:28:08     INFO -  Stack trace:
16:28:08     INFO -      JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 752
16:28:08     INFO -      JS frame :: chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js :: <TOP_LEVEL> :: line 60
16:28:08     INFO -      JS frame :: resource://gre/modules/Task.jsm :: TaskImpl_run :: line 282
16:28:08     INFO -      JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: Handler.prototype.process :: line 863
16:28:08     INFO -      JS frame :: resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js :: this.PromiseWalker.walkerLoop :: line 742
16:28:08     INFO -      native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
16:28:08     INFO -  TEST-INFO | MEMORY STAT vsize after test: 971014144
16:28:08     INFO -  TEST-INFO | MEMORY STAT vsizeMaxContiguous after test: 603496448
16:28:08     INFO -  TEST-INFO | MEMORY STAT residentFast after test: 444080128
16:28:08     INFO -  TEST-INFO | MEMORY STAT heapAllocated after test: 171798440
16:28:08     INFO -  INFO TEST-END | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview_update-in-iframes.js | finished in 1755ms
Patrick, it appears that this has basically been failing since the day it landed. Since unfortunately we're past the point where backing out is really practical, is there any chance you could find some time to take a look? :)
Blocks: 1007021
Flags: needinfo?(pbrosset)
I didn't notice that this has been failing intermittently for so long. I'll take a look at this today.
At least there's an error with a stacktrace before the assertion fails, so I have a good idea why this happens.
Assignee: nobody → pbrosset
Flags: needinfo?(pbrosset)
This patch is about not sending "display-change" events for nodes that are dead.
The thing is, the inspector actor keeps dead nodes around in its _refMap property, not that it's a problem seeing how we use that map anyway, but for the "display-change" use case, it was causing events to be sent for nodes that don't exist anymore.
So the simple fix is just to filter these nodes out.

Pending try build: https://tbpl.mozilla.org/?tree=Try&rev=c4a59fa3b04b
Attachment #8452959 - Flags: review?(mratcliffe)
Comment on attachment 8452959 [details] [diff] [review]
bug1020038-intermittent-browser_layoutview_update-in-iframes.js v1.patch

Review of attachment 8452959 [details] [diff] [review]:
-----------------------------------------------------------------

r+
Attachment #8452959 - Flags: review?(mratcliffe) → review+
That patch seems to be taking care of this intermittent error.
Pushed to fx-team: https://hg.mozilla.org/integration/fx-team/rev/c0a83d33c525
https://hg.mozilla.org/mozilla-central/rev/c0a83d33c525
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
Still hitting it :(
Status: RESOLVED → REOPENED
Flags: needinfo?(pbrosset)
Resolution: FIXED → ---
Target Milestone: Firefox 33 → ---
Time to acknowledge defeat here and disable the test.
Flags: needinfo?(pbrosset)
Disabling the test for now. r=me.
We can try and fix it again when working on the layoutview tests for e10s.
Attachment #8503939 - Flags: review+
Flags: needinfo?(pbrosset)
https://hg.mozilla.org/mozilla-central/rev/a540d8bbcb0c
Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Target Milestone: Firefox 35 → Firefox 36
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.