Bug 25093

Summary: REGRESSION: some scripts are showing in the Inspector with only "true" as the source
Product: WebKit Reporter: Timothy Hatcher <timothy>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: aroben, dglazkov, kmccullough, timothy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Fix conversion problem, v1. timothy: review+

Timothy Hatcher
Reported 2009-04-08 09:24:11 PDT
Recently some scripts have started to just show "true" as the source. It seems to be eval scripts and anything using the ScriptView class.
Attachments
Fix conversion problem, v1. (1.94 KB, patch)
2009-04-08 10:39 PDT, Dimitri Glazkov (Google) 		timothy: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Timothy Hatcher
Comment 1 2009-04-08 09:32:09 PDT
<rdar://problem/6769932>
Dimitri Glazkov (Google)
Comment 2 2009-04-08 09:32:26 PDT
I think I fixed it in the last commit. It was this: http://trac.webkit.org/changeset/42295/trunk/WebCore/bindings/js/JSInspectorControllerCustom.cpp But I'll check.
Dimitri Glazkov (Google)
Comment 3 2009-04-08 10:39:45 PDT
Created attachment 29338 [details] Fix conversion problem, v1. WebCore/ChangeLog | 12 ++++++++++++ WebCore/inspector/InspectorController.cpp | 4 ++-- 2 files changed, 14 insertions(+), 2 deletions(-)
Dimitri Glazkov (Google)
Comment 4 2009-04-08 10:40:33 PDT
This was all my fault.
Dimitri Glazkov (Google)
Comment 5 2009-04-08 10:53:28 PDT
Landed as http://trac.webkit.org/changeset/42325.
Note You need to log in before you can comment on or make changes to this bug.