Closed Bug 762571 Opened 12 years ago Closed 12 years ago

Intermittent browser_layoutview.js | #element-size has the right value after style update. - Got 160x160, expected 190x210 | .size > span has the right value after style update. - Got 100x100, expected 100x150 (and one more)

Categories

(DevTools :: Inspector, defect)

Product:
DevTools
Component:
Inspector
Platform:
x86_64
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Firefox 19

People

(Reporter: emorley, Assigned: paul)

References

Details

(Keywords: intermittent-failure, Whiteboard: [fixed-in-fx-team])

Attachments

(1 file)

patch v1
1.44 KB, patch
past
: review+
Details | Diff | Splinter Review 
Rev4 MacOSX Lion 10.7 mozilla-central debug test mochitest-other on 2012-06-04 08:29:02 PDT for push b4a1b0485188

slave: talos-r4-lion-077

https://tbpl.mozilla.org/php/getParsedLog.php?id=12358141&tree=Firefox

{
TEST-START | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js
++DOCSHELL 0x15aab6100 == 16 [id = 2664]
++DOMWINDOW == 43 (0x15aa3d230) [serial = 7085] [outer = 0x0]
++DOMWINDOW == 44 (0x13d760180) [serial = 7086] [outer = 0x15aa3d1b0]
WARNING: NS_ENSURE_TRUE(mMutable) failed: file ../../../../netwerk/base/src/nsSimpleURI.cpp, line 258
WARNING: NS_ENSURE_TRUE(mMutable) failed: file ../../../../netwerk/base/src/nsSimpleURI.cpp, line 258
TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | Console message: [JavaScript Error: "The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must to be declared in the document or in the transfer protocol." {file: "data:text/html,%3Cstyle%3Ediv%20%7B%20position%3A%20absolute%3B%20top%3A%2042px%3B%20left%3A%2042px%3B%20height%3A%20100px%3B%20width%3A%20100px%3B%20border%3A%2010px%20solid%20black%3B%20padding%3A%2020px%3B%20margin%3A%2030px%20auto%3B%7D%3C%2Fstyle%3E%3Cdiv%3E%3C%2Fdiv%3E" line: 0}]
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | node found
++DOCSHELL 0x15a1aec50 == 17 [id = 2665]
++DOMWINDOW == 45 (0x13d8cbf30) [serial = 7087] [outer = 0x0]
++DOCSHELL 0x13d8b8510 == 18 [id = 2666]
++DOMWINDOW == 46 (0x15a1857b0) [serial = 7088] [outer = 0x0]
WARNING: Subdocument container has no frame: file ../../../layout/base/nsDocumentViewer.cpp, line 2408
++DOMWINDOW == 47 (0x159916a00) [serial = 7089] [outer = 0x15a185730]
++DOMWINDOW == 48 (0x15a11d990) [serial = 7090] [outer = 0x13d8cbeb0]
++DOCSHELL 0x108a21c80 == 19 [id = 2667]
++DOMWINDOW == 49 (0x13d8ba180) [serial = 7091] [outer = 0x0]
TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | Inspector open
TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | Console message: [JavaScript Warning: "XUL box for hbox element contained an inline span child, forcing all its children to be wrapped in a block." {file: "resource:///modules/highlighter.jsm" line: 535}]
++DOMWINDOW == 50 (0x13f84e7f0) [serial = 7092] [outer = 0x13d8ba100]
WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80040111: file ../../../../content/events/src/nsContentEventHandler.cpp, line 91
TEST-INFO | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | Layout view ready
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | LayoutView document is alive.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | true
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | #element-size has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .size > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .margin.top > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .margin.left > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .margin.bottom > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .margin.right > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .padding.top > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .padding.left > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .padding.bottom > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .padding.right > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .border.top > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .border.left > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .border.bottom > span has the right value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .border.right > span has the right value.
--DOMWINDOW == 49 (0x13f57b180) [serial = 7084] [outer = 0x13f52f8a0] [url = about:blank]
--DOMWINDOW == 48 (0x13f52f920) [serial = 7082] [outer = 0x0] [url = about:blank]
--DOMWINDOW == 47 (0x13d752ac0) [serial = 7083] [outer = 0x159960ea0] [url = about:blank]
--DOMWINDOW == 46 (0x159960f20) [serial = 7081] [outer = 0x0] [url = about:blank]
--DOMWINDOW == 45 (0x158e05d70) [serial = 7078] [outer = 0x15aa9a060] [url = chrome://browser/content/inspector.html]
--DOMWINDOW == 44 (0x13e6cc790) [serial = 7077] [outer = 0x13bff6cc0] [url = about:blank]
--DOMWINDOW == 43 (0x13bff6d40) [serial = 7073] [outer = 0x0] [url = about:blank]
--DOMWINDOW == 42 (0x118a7d660) [serial = 7076] [outer = 0x13fb3d7c0] [url = about:blank]
--DOMWINDOW == 41 (0x13fb3d840) [serial = 7072] [outer = 0x0] [url = about:blank]
--DOMWINDOW == 40 (0x13d842f50) [serial = 7075] [outer = 0x15aa9a060] [url = about:blank]
--DOMWINDOW == 39 (0x15aa9a0e0) [serial = 7074] [outer = 0x0] [url = chrome://browser/content/inspector.html]
--DOMWINDOW == 38 (0x1570af6b0) [serial = 7071] [outer = 0x15871c9d0] [url = about:blank]
--DOMWINDOW == 37 (0x15871ca50) [serial = 7070] [outer = 0x0] [url = data:text/html,<div>hello%20from%20an%20iframe!</div>]
--DOMWINDOW == 36 (0x1565d5b00) [serial = 7066] [outer = 0x0] [url = chrome://browser/content/inspector.html]
--DOMWINDOW == 35 (0x15651c3b0) [serial = 7055] [outer = 0x0] [url = chrome://browser/content/devtools/layoutview/view.xhtml]
--DOMWINDOW == 34 (0x1599f92f0) [serial = 7051] [outer = 0x0] [url = about:blank]
--DOMWINDOW == 33 (0x15848a5c0) [serial = 7042] [outer = 0x0] [url = chrome://browser/content/devtools/layoutview/view.xhtml]
--DOMWINDOW == 32 (0x1584dff20) [serial = 7041] [outer = 0x0] [url = chrome://browser/content/devtools/cssruleview.xul]
--DOMWINDOW == 31 (0x1584ed9c0) [serial = 7040] [outer = 0x0] [url = chrome://browser/content/devtools/csshtmltree.xul]
--DOMWINDOW == 30 (0x159967270) [serial = 7021] [outer = 0x0] [url = chrome://browser/content/devtools/layoutview/view.xhtml]
--DOMWINDOW == 29 (0x13bfce270) [serial = 7018] [outer = 0x0] [url = about:blank]
--DOMWINDOW == 28 (0x1584744f0) [serial = 7015] [outer = 0x0] [url = chrome://browser/content/devtools/layoutview/view.xhtml]
--DOMWINDOW == 27 (0x13b523270) [serial = 7012] [outer = 0x0] [url = about:blank]
--DOCSHELL 0x13d843fb0 == 18 [id = 2663]
--DOCSHELL 0x15aa18400 == 17 [id = 2662]
--DOCSHELL 0x140b8e8d0 == 16 [id = 2659]
--DOCSHELL 0x157542ef0 == 15 [id = 2658]
--DOCSHELL 0x15847e460 == 14 [id = 2657]
--DOMWINDOW == 26 (0x1587d9580) [serial = 7080] [outer = 0x1599bab70] [url = about:blank]
--DOMWINDOW == 25 (0x1599babf0) [serial = 7079] [outer = 0x0] [url = data:text/html,basic%20tests%20for%20inspector]
--DOCSHELL 0x1599cb4b0 == 13 [id = 2661]
--DOCSHELL 0x15a1d33a0 == 12 [id = 2660]
--DOMWINDOW == 24 (0x15a1c2980) [serial = 7069] [outer = 0x1565f9260] [url = http://mochi.test:8888/browser/browser/devtools/highlighter/test/browser_inspector_treePanel_input.html]
--DOMWINDOW == 23 (0x1565f92e0) [serial = 7067] [outer = 0x0] [url = http://mochi.test:8888/browser/browser/devtools/highlighter/test/browser_inspector_treePanel_input.html]
--DOCSHELL 0x15758f5a0 == 11 [id = 2656]
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | #element-size has the right value after style update. - Got 160x160, expected 190x210
Stack trace:
    JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 428
    JS frame :: chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js :: test2 :: line 119
    native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .size > span has the right value after style update. - Got 100x100, expected 100x150
Stack trace:
    JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 428
    JS frame :: chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js :: test2 :: line 119
    native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0

TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .margin.top > span has the right value after style update.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .margin.left > span has the right value after style update.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .margin.bottom > span has the right value after style update.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .margin.right > span has the right value after style update.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .padding.top > span has the right value after style update.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .padding.left > span has the right value after style update.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .padding.bottom > span has the right value after style update.
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js | .padding.right > span has the right value after style update. - Got 20, expected 50
Stack trace:
    JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 428
    JS frame :: chrome://mochitests/content/browser/browser/devtools/layoutview/test/browser_layoutview.js :: test2 :: line 119
    native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
}
Blocks: 805931
As test author, please may you take a look at this? :-)
Flags: needinfo?(paul)
I'm looking at this tomorrow.
Flags: needinfo?(paul)
(In reply to Paul Rouget [:paul] from comment #72)
> I'm looking at this tomorrow.

Thank you :-)
Assignee: nobody → paul
> setTimeout(test2, 200); // Should be enough to get a MozAfterPaint event

Hmmf. I did that.
Attached patch patch v1 — — Splinter Review 

    
  

        Attachment #679110 -
        Flags: review?(past)

        Attachment #679110 -
        Flags: review?(past) → review+

    
  
Whiteboard: [orange] → [orange][land-in-fx-team]

    
    

    
      
  

    
    

    
  
https://hg.mozilla.org/integration/fx-team/rev/5c608a70bfc4
Status: NEW → ASSIGNED
Whiteboard: [orange][land-in-fx-team] → [orange][fixed-in-fx-team]

    
    

    
      
  

    
    

    
      
  

    
    

    
      
  

    
    

    
  
Did this ever land in m-c?

    
    

    
  
(In reply to Paul Rouget [:paul] from comment #82)
> Did this ever land in m-c?

To answer that question I normally just take the changeset hash and stick it on the end of https://hg.mozilla.org/mozilla-central/rev/

In this case, "yes, but looks like bugs weren't updated after the merge".

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/5c608a70bfc4
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
Whiteboard: [orange][fixed-in-fx-team] → [fixed-in-fx-team]

    
  
Product: Firefox → DevTools

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: