WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 69516
Web Inspector: Inspector fails to start if there was at least one watch expression.
https://bugs.webkit.org/show_bug.cgi?id=69516
Summary
Web Inspector: Inspector fails to start if there was at least one watch expre...
Vsevolod Vlasov
Reported
2011-10-06 04:11:43 PDT
Inspector fails to start if there was at least one watch expression added. [19960:19960:652471493820:ERROR:CONSOLE(569)] "Uncaught TypeError: Cannot read property 'paused' of undefined", source: chrome-devtools://devtools/ConsoleView.js (569)
Attachments
Patch
(3.33 KB, patch)
2011-10-06 05:41 PDT
,
Pavel Feldman
yurys
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Vsevolod Vlasov
Comment 1
2011-10-06 04:13:05 PDT
Related lines from ConsoleView.js: evalInInspectedWindow: function(expression, objectGroup, includeCommandLineAPI, doNotPauseOnExceptions, returnByValue, callback) { if (WebInspector.panels.scripts.paused) { <-- fails here. WebInspector.panels.scripts.evaluateInSelectedCallFrame(expression, objectGroup, includeCommandLineAPI, returnByValue, callback); return; }
Pavel Feldman
Comment 2
2011-10-06 05:41:58 PDT
Created
attachment 109949
[details]
Patch
Yury Semikhatsky
Comment 3
2011-10-06 05:49:43 PDT
Comment on
attachment 109949
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=109949&action=review
> Source/WebCore/inspector/front-end/WatchExpressionsSidebarPane.js:60 > this._wasShown = true;
Please move this line right after the if (this._wasShown) above.
Pavel Feldman
Comment 4
2011-10-06 06:03:59 PDT
Committed
r96804
: <
http://trac.webkit.org/changeset/96804
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug