Closed Bug 810642 Opened 12 years ago Closed 12 years ago

Intermittent browser_dbg_panesize.js | Height preferences shouldn't have been updated yet.

Categories

(DevTools :: Debugger, defect)

Product:
DevTools
Component:
Debugger
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox18 unaffected)

Status:
RESOLVED FIXED
Milestone:
Firefox 19
Tracking Flags:
Tracking Status
firefox18 --- unaffected

People

(Reporter: philor, Assigned: vporof)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

v1
3.53 KB, patch
past
: review+
Details | Diff | Splinter Review 
https://tbpl.mozilla.org/php/getParsedLog.php?id=16935437&tree=Mozilla-Inbound
Rev5 MacOSX Mountain Lion 10.8 mozilla-inbound debug test mochitest-browser-chrome on 2012-11-10 19:31:59 PST for push 782f9b92815c
slave: talos-mtnlion-r5-006

TEST-START | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js
++DOCSHELL 0x157a80b70 == 13 [id = 2786]
++DOMWINDOW == 40 (0x149b5adb0) [serial = 7368] [outer = 0x0]
++DOMWINDOW == 41 (0x14f7d5250) [serial = 7369] [outer = 0x149b5ad30]
WARNING: NS_ENSURE_TRUE(mMutable) failed: file ../../../../netwerk/base/src/nsSimpleURI.cpp, line 272
WARNING: NS_ENSURE_TRUE(mMutable) failed: file ../../../../netwerk/base/src/nsSimpleURI.cpp, line 272
WARNING: NS_ENSURE_TRUE(frame) failed: file ../../../layout/base/nsPresContext.cpp, line 1178
WARNING: NS_ENSURE_TRUE(frame) failed: file ../../../layout/base/nsPresContext.cpp, line 1178
++DOMWINDOW == 42 (0x1529ec230) [serial = 7370] [outer = 0x149b5ad30]
WARNING: Unable to test style tree integrity -- no content node: file ../../../layout/base/nsCSSFrameConstructor.cpp, line 8253
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js | Shouldn't have a debugger pane for this tab yet.
++DOCSHELL 0x14f706790 == 14 [id = 2787]
++DOMWINDOW == 43 (0x14f7bc760) [serial = 7371] [outer = 0x0]
++DOMWINDOW == 44 (0x14ba45fb0) [serial = 7372] [outer = 0x14f7bc6e0]
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js | toggleDebugger() should return a pane.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js | getDebugger() should return the same pane as toggleDebugger().
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js | The debugger preferences should have a saved height value.
TEST-PASS | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js | The debugger pane height should be the same as the preferred value.
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js | Height preferences shouldn't have been updated yet.
Stack trace:
    JS frame :: chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js :: tab1< :: line 30
    JS frame :: chrome://mochikit/content/browser-test.js :: test_executeSoon/<.run :: line 503
    native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0

DBG-FRONTEND: ToolbarView was instantiated
DBG-FRONTEND: OptionsView was instantiated
DBG-FRONTEND: ChromeGlobalsView was instantiated
DBG-FRONTEND: SourcesView was instantiated
DBG-FRONTEND: FilterView was instantiated
DBG-FRONTEND: StackFramesView was instantiated
DBG-FRONTEND: BreakpointsView was instantiated
DBG-FRONTEND: GlobalSearchView was instantiated
DBG-FRONTEND: Initializing the DebuggerView
DBG-FRONTEND: Initializing the DebuggerView window
DBG-FRONTEND: Initializing the DebuggerView panes
DBG-FRONTEND: Initializing the ToolbarView
DBG-FRONTEND: Initializing the OptionsView
DBG-FRONTEND: Initializing the ChromeGlobalsView
DBG-FRONTEND: Initializing the SourcesView
DBG-FRONTEND: Initializing the FilterView
DBG-FRONTEND: Initializing the StackFramesView
DBG-FRONTEND: Initializing the BreakpointsView
DBG-FRONTEND: Initializing the GlobalSearchView
DBG-FRONTEND: Initializing the DebuggerView editor
++DOCSHELL 0x15b0f3b60 == 15 [id = 2788]
++DOMWINDOW == 45 (0x14f715ab0) [serial = 7373] [outer = 0x0]
++DOMWINDOW == 46 (0x15b0d1280) [serial = 7374] [outer = 0x14f715a30]
++DOCSHELL 0x157acc760 == 16 [id = 2789]
++DOMWINDOW == 47 (0x120fda160) [serial = 7375] [outer = 0x0]
TEST-INFO | chrome://mochitests/content/browser/browser/devtools/debugger/test/browser_dbg_panesize.js | Console message: [JavaScript Warning: "Use of Mutation Events is deprecated. Use MutationObserver instead." {file: "chrome://browser/content/orion.js" line: 3408}]
++DOMWINDOW == 48 (0x14ba33a40) [serial = 7376] [outer = 0x120fda0e0]
++DOMWINDOW == 49 (0x157a025c0) [serial = 7377] [outer = 0x120fda0e0]
WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file ../../../../editor/libeditor/base/nsEditor.cpp, line 3905
WARNING: NS_ENSURE_SUCCESS(res, res) failed with result 0x80004005: file ../../../../editor/libeditor/html/nsHTMLEditRules.cpp, line 304
WARNING: NS_ENSURE_SUCCESS(result, result) failed with result 0x80004005: file ../../../../editor/libeditor/base/nsEditor.cpp, line 3905
WARNING: NS_ENSURE_SUCCESS(res, res) failed with result 0x80004005: file ../../../../editor/libeditor/html/nsHTMLEditRules.cpp, line 7960
JavaScript strict warning: chrome://browser/content/orion.js, line 8219: reference to undefined property h.capture
JavaScript strict warning: chrome://browser/content/orion.js, line 6954: reference to undefined property this._leftDiv
...
Attached patch v1 — — Splinter Review 
It seems that in the very few moments when the planets align in November, although highly unlikely, the generated random value is the same as the already preferred value.
Attachment #680423 - Flags: review?(past)
Comment on attachment 680423 [details] [diff] [review]
v1

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

Although this test will go away when devtools-window is merged, let's avoid panesize-inner failing next year.
Attachment #680423 - Flags: review?(past) → review+
Whiteboard: [orange] → [orange][land-in-fx-team]
https://hg.mozilla.org/integration/mozilla-inbound/rev/56f241626604
Whiteboard: [orange][land-in-fx-team] → [orange]
https://hg.mozilla.org/mozilla-central/rev/56f241626604
Assignee: nobody → vporof
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
Whiteboard: [orange]
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: