Bug 115345

Summary: [Qt][WK2] REGRESSION(r149259): broke Qt WK2 build
Product: WebKit Reporter: Zoltan Arvai <zarvai>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abecsi, allan.jensen, andersca, cmarcelo, commit-queue, hausmann, jturcotte, kadam, kling, menard
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 115259    
Attachments:
Description Flags
c++0x
none
patch: c++11 for Qt WK2 none

Description Zoltan Arvai 2013-04-29 01:22:42 PDT
Errors messages on Qt WK2 bots:

In file included from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.cpp:27:0:
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.h:81:54: error: 'm_pluginControllers' was not declared in this scope[0m
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.h:81:51: error: '>>' should be '> >' within a nested template argument list[0m
/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.cpp: In member function 'virtual void WebKit::WebProcessConnection::didClose(CoreIPC::Connection*)':
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.cpp:170:16: error: ISO C++ forbids declaration of 'pluginController' with no type [-fpermissive][0m
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.cpp:170:34: error: range-based-for loops are not allowed in C++98 mode[0m
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.cpp:171:51: error: request for member 'get' in 'pluginController', which is of non-class type 'int'[0m
ICECC[15102] 11:18:35: Compiled on 10.6.11.18
make[3]: *** [.obj/release-shared/PluginProcess/WebProcessConnection.o] Error 1
make[3]: *** Waiting for unfinished jobs....
In file included from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/PluginProcess.cpp:38:0:
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.h:81:54: error: 'm_pluginControllers' was not declared in this scope[0m
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.h:81:51: error: '>>' should be '> >' within a nested template argument list[0m
ICECC[15101] 11:18:35: Compiled on 10.6.11.10
make[3]: *** [.obj/release-shared/PluginProcess/PluginProcess.o] Error 1
In file included from /ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/PluginControllerProxy.cpp:42:0:
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.h:81:54: error: 'm_pluginControllers' was not declared in this scope[0m
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.h:81:51: error: '>>' should be '> >' within a nested template argument list[0m
ICECC[15100] 11:18:36: Compiled on 10.109.212.1
make[3]: *** [.obj/release-shared/PluginProcess/PluginControllerProxy.o] Error 1
In file included from generated/WebProcessConnectionMessageReceiver.cpp:29:0:
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.h:81:54: error: 'm_pluginControllers' was not declared in this scope[0m
[1;31m/ramdisk/qt-linux-32-release-webkit2/build/Source/WebKit2/PluginProcess/WebProcessConnection.h:81:51: error: '>>' should be '> >' within a nested template argument list[0m


After changing Source/WebKit2/PluginProcess/WebProcessConnection.h
-    HashMap<uint64_t, OwnPtr<PluginControllerProxy>> m_pluginControllers;
+    HashMap<uint64_t, OwnPtr<PluginControllerProxy> > m_pluginControllers;
getting other error message:

/home/azbest/webkit/WebKit/Source/WebKit2/PluginProcess/WebProcessConnection.cpp: In member function 'virtual void WebKit::WebProcessConnection::didClose(CoreIPC::Connection*)':
/home/azbest/webkit/WebKit/Source/WebKit2/PluginProcess/WebProcessConnection.cpp:170:16: error: ISO C++ forbids declaration of 'pluginController' with no type [-fpermissive]
/home/azbest/webkit/WebKit/Source/WebKit2/PluginProcess/WebProcessConnection.cpp:170:34: error: range-based-for loops are not allowed in C++98 mode
/home/azbest/webkit/WebKit/Source/WebKit2/PluginProcess/WebProcessConnection.cpp:171:51: error: request for member 'get' in 'pluginController', which is of non-class type 'int'
Comment 1 Zoltan Arvai 2013-04-29 04:23:01 PDT
Created attachment 199989 [details]
c++0x

Enabling c++0x on Qt WK2 is fixing the build, but I don't know if we want enable it or not.
Comment 2 Zoltan Arvai 2013-04-29 06:40:21 PDT
Created attachment 200000 [details]
patch: c++11 for Qt WK2

With CONFIG += c++11 Qt WebKit2 builds fine on linux and windows, too.
Comment 3 WebKit Commit Bot 2013-04-29 09:06:13 PDT
Comment on attachment 200000 [details]
patch: c++11 for Qt WK2

Clearing flags on attachment: 200000

Committed r149291: <http://trac.webkit.org/changeset/149291>
Comment 4 WebKit Commit Bot 2013-04-29 09:06:16 PDT
All reviewed patches have been landed.  Closing bug.