Bug 116390 - [WK2] Regression(r150305): broke build if INDEXED_DATABASE is enabled
Summary: [WK2] Regression(r150305): broke build if INDEXED_DATABASE is enabled
Status: RESOLVED DUPLICATE of bug 116389
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on:
Blocks: 116360
  Show dependency treegraph
 
Reported: 2013-05-18 02:43 PDT by Chris Dumez
Modified: 2013-05-18 14:25 PDT (History)
13 users (show)

See Also:


Attachments
Patch (6.03 KB, patch)
2013-05-18 03:01 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2013-05-18 02:43:51 PDT
<http://trac.webkit.org/changeset/150305> broke the WebKit2 build if INDEXED_DATABASE is enabled at compile time:
[ 84%] [ 84%] Building CXX object Source/WebKit2/CMakeFiles/ewebkit2.dir/WebProcess/WebPage/WebPageGroupProxy.cpp.o
Building CXX object Source/WebKit2/CMakeFiles/ewebkit2.dir/WebProcess/WebPage/WebUndoStep.cpp.o
[ 84%] Building CXX object Source/WebKit2/CMakeFiles/ewebkit2.dir/WebProcess/WebPage/CoordinatedGraphics/CoordinatedLayerTreeHost.cpp.o
/mnt/buildbot/efl-linux-slave-3/efl-linux-32-release/build/Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp: In constructor 'WebKit::WebPageGroupProxy::WebPageGroupProxy(const WebKit::WebPageGroupData&)':
/mnt/buildbot/efl-linux-slave-3/efl-linux-32-release/build/Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp:66:84: error: 'class WebKit::WebKeyValueStorageManager' has no member named 'indexedDBDatabaseDirectory'
In file included from /mnt/buildbot/efl-linux-slave-3/efl-linux-32-release/build/Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp:29:0:
/mnt/buildbot/efl-linux-slave-3/efl-linux-32-release/build/Source/WebKit2/WebProcess/WebProcess.h: In member function 'T* WebKit::WebProcess::supplement() [with T = WebKit::WebKeyValueStorageManager]':
/mnt/buildbot/efl-linux-slave-3/efl-linux-32-release/build/Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp:65:116:   instantiated from here
/mnt/buildbot/efl-linux-slave-3/efl-linux-32-release/build/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*'
/mnt/buildbot/efl-linux-slave-3/efl-linux-32-release/build/Source/WebKit2/WebProcess/WebProcess.h:91:5: error: control reaches end of non-void function [-Werror=return-type]
cc1plus: all warnings being treated as errors
Comment 1 Chris Dumez 2013-05-18 02:45:27 PDT
I temporarily disabled INDEXED_DATABASE at compile-time for EFL to quickly fix the build and put our bots back up <http://trac.webkit.org/changeset/150335>.

We need to reenable this as soon as the build issue is addressed.
Comment 2 Chris Dumez 2013-05-18 03:01:10 PDT
Created attachment 202199 [details]
Patch

Andersca, is it the proper way to fix it or is more work involved?
Comment 3 Zan Dobersek 2013-05-18 04:01:19 PDT
Comment on attachment 202199 [details]
Patch

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

> Source/WebKit2/WebProcess/WebPage/WebPageGroupProxy.cpp:-67
> -#endif

Removing this fixes the build failures, but it also stops setting the path for the IDB database. I don't think that's quite OK.
Comment 4 Zan Dobersek 2013-05-18 04:27:58 PDT

*** This bug has been marked as a duplicate of bug 116389 ***
Comment 5 Zan Dobersek 2013-05-18 04:50:36 PDT
I (accidentally) marked this bug as a dupe of bug #116389, but feel free to switch the duplicate resolution as you please. My main interest is still to get the build failures resolved ASAP. Sorry for the noise.
Comment 6 Chris Dumez 2013-05-18 14:25:53 PDT
(In reply to comment #5)
> I (accidentally) marked this bug as a dupe of bug #116389, but feel free to switch the duplicate resolution as you please. My main interest is still to get the build failures resolved ASAP. Sorry for the noise.

Sure, no problem. Sorry I failed to notice someone had already fixed a bug.
I can reenable INDEXED_DATABASE on EFL in a separate patch later on.