[Webkit-unassigned] [Bug 101725] [V8] Inspector does not callback IDB methods in context

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=101725





--- Comment #16 from Dan Carney <dcarney at google.com>  2012-11-26 01:35:05 PST ---
(From update of attachment 173236)
View in context: https://bugs.webkit.org/attachment.cgi?id=173236&action=review

>> Source/WebCore/inspector/InspectorIndexedDBAgent.cpp:675
>>      RefPtr<DatabaseLoader> databaseLoader = DatabaseLoader::create(document, requestCallback);
> 
> Why do we have to enter V8 context while calling a native function which by the way accepts ScriptExecutionContext and can enter the context if it need one? It seems completely wrong to me. Looks like we should make DatabaseLoader::create instead not to make assumptions about the v8 context stack, is there any problem with that approach?

if ScriptExecutionContext were enough to determine context, I would be using it.  Unfortunately, for isolated worlds, ScriptExecutionContext is simply not enough. It could be that the context needs to be stored somewhere earlier and reused at the points I changed, but since I know nothing about the way inspector works, I can't say what the correct solution here is at all.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.