Summary: | Web Inspector: remove scriptWorldType from the protocol. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Pavel Feldman <pfeldman> | ||||||
Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | apavlov, buildbot, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, webkit-ews, yurys | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Pavel Feldman
2011-04-12 08:28:32 PDT
Created attachment 89208 [details]
Patch
Attachment 89208 [details] did not build on qt: Build output: http://queues.webkit.org/results/8391424 Created attachment 89212 [details]
[PATCH] that compiles.
Attachment 89208 [details] did not build on win: Build output: http://queues.webkit.org/results/8395096 Comment on attachment 89212 [details] [PATCH] that compiles. View in context: https://bugs.webkit.org/attachment.cgi?id=89212&action=review > Source/WebCore/bindings/js/ScriptDebugServer.cpp:247 > + return true; return (currentWorld(exec) != mainThreadNormalWorld()); ? > Source/WebCore/bindings/v8/DebuggerScript.js:80 > + isContentScript: script.context_data && script.context_data.indexOf("injected") == 0 Please cast the expression to boolean explicitly to avoid surprises when it's an undefined and is casted ToBoolean() in ScriptDebugServer.cpp. Committed r83601: <http://trac.webkit.org/changeset/83601> |