Bug 116389

Summary: REGRESSION(r150305): WebKit2 build broken with INDEXED_DATABASE enabled
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: WebKit2Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, bdakin, cdumez, mrobinson, oleg_smirnov
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 116430    
Attachments:
Description Flags
Fix the build none

Description Carlos Garcia Campos 2013-05-18 02:25:35 PDT
../../Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp: In constructor 'WebKit::WebPageGroupProxy::WebPageGroupProxy(const WebKit::WebPageGroupData&)':
../../Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp:66:84: error: 'class WebKit::WebKeyValueStorageManager' has no member named 'indexedDBDatabaseDirectory'
In file included from ../../Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp:29:0:
../../Source/WebKit2/WebProcess/WebProcess.h: In instantiation of 'T* WebKit::WebProcess::supplement() [with T = WebKit::WebKeyValueStorageManager]':
../../Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp:65:116:   required from here
../../Source/WebKit2/WebProcess/WebProcess.h:90:70: error: invalid static_cast from type 'WTF::HashMap<const char*, WTF::OwnPtr<WebKit::WebProcessSupplement>, WTF::PtrHash<const char*> >::MappedPeekType {aka WebKit::WebProcessSupplement*}' to type 'WebKit::WebKeyValueStorageManager*'
../../Source/WebKit2/WebProcess/WebProcess.h: In member function 'T* WebKit::WebProcess::supplement() [with T = WebKit::WebKeyValueStorageManager]':
../../Source/WebKit2/WebProcess/WebProcess.h:91:5: warning: control reaches end of non-void function [-Wreturn-type]
make[1]: *** [Source/WebKit2/WebProcess/WebPage/libwebkit2gtk_3_0_la-WebPageGroupProxy.lo] Error 1

There's no longer WebKeyValueStorageManager in the web process after r150305
Comment 1 Carlos Garcia Campos 2013-05-18 02:27:55 PDT
Created attachment 202197 [details]
Fix the build

I'm not sure whether this is the right fix or it should be moved to the ui process now, but it fixes the build for now.
Comment 2 Zan Dobersek 2013-05-18 04:27:58 PDT
*** Bug 116390 has been marked as a duplicate of this bug. ***
Comment 3 Chris Dumez 2013-05-18 14:21:58 PDT
*** Bug 116396 has been marked as a duplicate of this bug. ***
Comment 4 Martin Robinson 2013-05-18 16:13:47 PDT
Comment on attachment 202197 [details]
Fix the build

Going to review and land this one since it's a build fix, which means it's blocking forward progress. If someone has a better idea for how to fix this let's work it out later.
Comment 5 Martin Robinson 2013-05-18 16:14:40 PDT
Comment on attachment 202197 [details]
Fix the build

Clearing flags on attachment: 202197

Committed r150344: <http://trac.webkit.org/changeset/150344>
Comment 6 Martin Robinson 2013-05-18 16:14:43 PDT
All reviewed patches have been landed.  Closing bug.