Closed
Bug 706734
Opened 13 years ago
Closed 13 years ago
"this.veilContainer is null" in browser/devtools/highlighter/inspector.jsm
Categories
(DevTools :: Inspector, defect)
DevTools
Inspector
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 11
People
(Reporter: heycam, Assigned: paul)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
1000 bytes,
patch
|
rcampbell
:
review+
|
Details | Diff | Splinter Review |
This exception crops up in a few inspector b-c mochitests, as a result of bug 692466 looks like. For example see https://tbpl.mozilla.org/php/getParsedLog.php?id=7658526&tree=Try#error0 (which is a build that has uncaught-exceptions-cause-test-failure turned on): TEST-START | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | InspectorUI variable exists TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector is not highlighting TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector.store is empty TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector is highlighting TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector Tree Panel is not open TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector Sidebar is not open TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | InspectorUI.store is not empty TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector.store.length = 1 TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | selection matches the div element TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector is not highlighting TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector Tree Panel is closed TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector Sidebar is not open TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector.store.length = 1 TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | an unexpected uncaught JS exception reported through window.onerror - Script error. at :0 Stack trace: JS frame :: chrome://mochikit/content/tests/SimpleTest/SimpleTest.js :: simpletestOnerror :: line 964 native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0 JavaScript error: resource:///modules/inspector.jsm, line 705: this.veilContainer is null TEST-INFO | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Console message: [JavaScript Error: "this.veilContainer is null" {file: "resource:///modules/inspector.jsm" line: 705}] TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector is highlighting TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector Tree Panel is not open TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | Inspector.store.length = 1 TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | selection matches the div element TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | sidebar is open TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | rule view is open TEST-PASS | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | RuleView elements.length == 1 INFO TEST-END | chrome://mochitests/content/browser/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js | finished in 1278ms
Reporter | ||
Updated•13 years ago
|
Assignee | ||
Comment 1•13 years ago
|
||
Victor spot the problem. I didn't clear the timeout before destroying the inspector.
Assignee | ||
Comment 2•13 years ago
|
||
I am not checking if `this.transitionDisabler` is null or not. This is done by `clearTimeout`.
Comment 3•13 years ago
|
||
Comment on attachment 578215 [details] [diff] [review] patch v1 funny, I've been hitting this testing an unrelated patch. Thanks for fixing!
Attachment #578215 -
Flags: review?(rcampbell) → review+
Comment 4•13 years ago
|
||
Comment on attachment 578215 [details] [diff] [review] patch v1 actually let's move that inside closeInspectorUI() in the "if (highlighter)" section right before the destroy call.
Comment 5•13 years ago
|
||
disregard comment #4, your patch is fine as it is.
Reporter | ||
Comment 6•13 years ago
|
||
As part of the patch, can you please remove these calls to ignoreAllUncaughtExceptions(), which I just added in bug 703176? http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/test/browser_inspector_duplicate_ruleview.js#l114 http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/test/browser_inspector_initialization.js#l221 http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/test/browser_inspector_ruleviewstore.js#l140
Comment 7•13 years ago
|
||
(In reply to Cameron McCormack (:heycam) from comment #6) > As part of the patch, can you please remove these calls to > ignoreAllUncaughtExceptions(), which I just added in bug 703176? > > http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/ > test/browser_inspector_duplicate_ruleview.js#l114 > > http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/ > test/browser_inspector_initialization.js#l221 > > http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/ > test/browser_inspector_ruleviewstore.js#l140 or you could open a separate bug to do this.
Assignee | ||
Comment 8•13 years ago
|
||
(In reply to Cameron McCormack (:heycam) from comment #6) > As part of the patch, can you please remove these calls to > ignoreAllUncaughtExceptions(), which I just added in bug 703176? > > http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/ > test/browser_inspector_duplicate_ruleview.js#l114 > > http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/ > test/browser_inspector_initialization.js#l221 > > http://hg.mozilla.org/mozilla-central/file/tip/browser/devtools/highlighter/ > test/browser_inspector_ruleviewstore.js#l140 Patch backed out and re-introduced in inbound. We need to land that in fx-team. I want to try to commit this current patch myself (just got commit access level 3), so if you don't mind, I will keep it simple and just push this little change.
Assignee | ||
Updated•13 years ago
|
Whiteboard: [land-in-fx-team] <- paul will do it!!!
Assignee | ||
Comment 9•13 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/917ffaaafcd7
Whiteboard: [land-in-fx-team] <- paul will do it!!! → [fixed-in-fx-team]
Reporter | ||
Comment 10•13 years ago
|
||
(In reply to Paul Rouget [:paul] from comment #8) > I want to try to commit this current patch myself (just got commit access > level 3), so if you don't mind, I will keep it simple and just push this > little change. Yes, that's fine, I'll make those other changes myself once your patch gets merged into m-c. Thanks!
Comment 11•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/917ffaaafcd7
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 11
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•