Bug 57830

Summary: Web Inspector:Duplicated display of storage db when running Web SQL from an iframe
Product: WebKit Reporter: Ilya Tikhonovsky <loislo>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
test1.html
none
parent.html
none
[patch] initial version none

Description Ilya Tikhonovsky 2011-04-05 05:22:06 PDT
Created attachment 88204 [details]
test1.html

What steps will reproduce the problem?
1. Go to page with iframe and a database in it
2. See the Storage section in the dev tools
3. Every time the iframe is loaded the icon of the database is duplicated 

What is the expected output?
Show one single instance of the database in the dev tools

What do you see instead?
A duplicate for every reload
Comment 1 Ilya Tikhonovsky 2011-04-05 05:22:33 PDT
Created attachment 88205 [details]
parent.html
Comment 2 Ilya Tikhonovsky 2011-04-05 05:36:17 PDT
Created attachment 88206 [details]
[patch] initial version
Comment 3 Pavel Feldman 2011-04-05 05:55:59 PDT
Comment on attachment 88206 [details]
[patch] initial version

View in context: https://bugs.webkit.org/attachment.cgi?id=88206&action=review

> Source/WebCore/inspector/InspectorDatabaseAgent.cpp:224
> +        resource->setDatabase(database);

What is this needed for?
Comment 4 Ilya Tikhonovsky 2011-04-14 02:09:19 PDT
Comment on attachment 88206 [details]
[patch] initial version

This patch covers typical use cases.
The page reopens Database and we have to update our internal ref to it because the old instance of DB usually has been closed.
Comment 5 Ilya Tikhonovsky 2011-04-14 05:32:29 PDT
Comment on attachment 88206 [details]
[patch] initial version

Clearing flags on attachment: 88206

Committed r83841: <http://trac.webkit.org/changeset/83841>
Comment 6 Ilya Tikhonovsky 2011-04-14 05:32:36 PDT
All reviewed patches have been landed.  Closing bug.