Bug 65912 - Web Inspector: assertion failure in InspectorDOMStorageResource::bind when opening inspector
Summary: Web Inspector: assertion failure in InspectorDOMStorageResource::bind when op...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Andrey Kosyakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-09 05:41 PDT by Andrey Kosyakov
Modified: 2011-08-09 06:24 PDT (History)
9 users (show)

See Also:

Attachments
patch (1.21 KB, patch)
2011-08-09 05:44 PDT, Andrey Kosyakov 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Kosyakov 2011-08-09 05:41:51 PDT 
1. run debug build
2. open web inspector
3. observe inspected page crash

The assertion in InspectorDOMStorageResource.cpp:71 is hit.
Comment 1 Andrey Kosyakov 2011-08-09 05:44:58 PDT 
Created attachment 103349 [details]
patch
Comment 2 Pavel Feldman 2011-08-09 05:59:21 PDT 
Comment on attachment 103349 [details]
patch

This subtle change definitely requires a test!
Comment 3 Andrey Kosyakov 2011-08-09 06:16:25 PDT 
(In reply to comment #2)
> (From update of attachment 103349 [details])
> This subtle change definitely requires a test!

This is difficult to test with current harness, as the agent is enabled very early in the front-end initialization.

Raised Bug 65914 for protocol test framework.
Comment 4 Andrey Kosyakov 2011-08-09 06:24:33 PDT 
Comment on attachment 103349 [details]
patch

Clearing flags on attachment: 103349

Committed r92678: <http://trac.webkit.org/changeset/92678>
Comment 5 Andrey Kosyakov 2011-08-09 06:24:40 PDT 
All reviewed patches have been landed.  Closing bug.