Bug 43788 - Web Inspector: move databse processing to native layer.
Summary: Web Inspector: move databse processing to native layer.
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: Pavel Feldman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-10 06:07 PDT by Pavel Feldman
Modified: 2010-08-10 09:14 PDT (History)
12 users (show)

See Also:


Attachments
[PATCH] Proposed change. Will add to XCode proj on landing. (41.56 KB, patch)
2010-08-10 06:14 PDT, Pavel Feldman
yurys: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pavel Feldman 2010-08-10 06:07:03 PDT
Rationale:
- Slowly move out everything from InjectedScript
- Databases from iframes can now be processed
- Removed async dispatching on InjectedScript and its support from generator.
Comment 1 Pavel Feldman 2010-08-10 06:14:07 PDT
Created attachment 64007 [details]
[PATCH] Proposed change. Will add to XCode proj on landing.
Comment 2 Yury Semikhatsky 2010-08-10 06:45:50 PDT
Comment on attachment 64007 [details]
[PATCH] Proposed change. Will add to XCode proj on landing.

WebCore/inspector/InspectorBackend.cpp:139
 +      *transactionId = m_inspectorController->m_storageAgent->executeSQL(database, query);
Please introduce a getter for storage agent as we do for other agents.

WebCore/inspector/InspectorStorageAgent.h:34
 +  #include "wtf/PassOwnPtr.h"
should be <wtf/PassRefPtr.h> ?

WebCore/inspector/InspectorStorageAgent.cpp:49
 +  #include "wtf/Vector.h"
<wtf/Vector.h>

WebCore/inspector/InspectorStorageAgent.cpp:39
 +  #include "SQLValue.h"
wrong include order
Comment 3 Pavel Feldman 2010-08-10 09:07:07 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	C	WebCore/inspector/front-end/Database.js => WebCore/inspector/InspectorStorageAgent.h
	M	WebCore/CMakeLists.txt
	M	WebCore/ChangeLog
	M	WebCore/GNUmakefile.am
	M	WebCore/WebCore.gypi
	M	WebCore/WebCore.pro
	M	WebCore/WebCore.vcproj/WebCore.vcproj
	M	WebCore/WebCore.xcodeproj/project.pbxproj
	M	WebCore/bindings/js/JSInjectedScriptHostCustom.cpp
	M	WebCore/bindings/v8/custom/V8InjectedScriptHostCustom.cpp
	M	WebCore/inspector/CodeGeneratorInspector.pm
	M	WebCore/inspector/InjectedScript.cpp
	M	WebCore/inspector/InjectedScript.h
	M	WebCore/inspector/InjectedScriptHost.cpp
	M	WebCore/inspector/InjectedScriptHost.h
	M	WebCore/inspector/InjectedScriptHost.idl
	M	WebCore/inspector/Inspector.idl
	M	WebCore/inspector/InspectorBackend.cpp
	M	WebCore/inspector/InspectorBackend.h
	M	WebCore/inspector/InspectorController.cpp
	M	WebCore/inspector/InspectorController.h
	A	WebCore/inspector/InspectorStorageAgent.cpp
	M	WebCore/inspector/front-end/Database.js
	M	WebCore/inspector/front-end/DatabaseQueryView.js
	M	WebCore/inspector/front-end/DatabaseTableView.js
	M	WebCore/inspector/front-end/InjectedScript.js
	M	WebCore/inspector/front-end/InjectedScriptAccess.js
	M	WebCore/inspector/front-end/StoragePanel.js
Committed r65072
Comment 4 WebKit Review Bot 2010-08-10 09:14:22 PDT
http://trac.webkit.org/changeset/65072 might have broken Qt Linux Release minimal
The following changes are on the blame list:
http://trac.webkit.org/changeset/65072
http://trac.webkit.org/changeset/65069
http://trac.webkit.org/changeset/65070
http://trac.webkit.org/changeset/65071