[Webkit-unassigned] [Bug 118230] New: [GTK] Use PassOwnPtr for the returned value of createBackingStore

[bugzilla-daemon at webkit.org]

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

           Summary: [GTK] Use PassOwnPtr for the returned value of
                    createBackingStore
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebKit Gtk
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: kevin.cs.oh at gmail.com


I think it would be better that createBackingStore returns  PassOwnPtr instread of OwnPtr to keep compatibility with c++0x.
Otherwise we face following break with the c++0x compile option.

[shivamidow at LenovoStation:Release]$ make -j 3
make  all-am
make[1]: Entering directory `/home/shivamidow/Projects/WebKit/WebKitBuild/Release'
/bin/mkdir -p ./.deps/DerivedSources
  CXX    Source/WebKit/gtk/WebCoreSupport/libwebkitgtk_3_0_la-ChromeClientGtk.lo
  CXX    Source/WebKit/gtk/WebCoreSupport/libwebkitgtk_3_0_la-FrameLoaderClientGtk.lo
  CXX    Source/WebKit/gtk/WebCoreSupport/libwebkitgtk_3_0_la-FrameNetworkingContextGtk.lo
  CXX    Source/WebKit/gtk/WebCoreSupport/libwebkitgtk_3_0_la-GeolocationClientGtk.lo
In file included from ../../Source/WTF/wtf/VectorTraits.h:25:0,
                 from ../../Source/WTF/wtf/Vector.h:31,
                 from ../../Source/WebCore/platform/graphics/FloatRect.h:31,
                 from ../../Source/WebCore/platform/graphics/FloatQuad.h:33,
                 from ../../Source/WebCore/accessibility/AccessibilityObject.h:33,
                 from ../../Source/WebCore/accessibility/AXObjectCache.h:29,
                 from ../../Source/WebCore/page/ChromeClient.h:25,
                 from ../../Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.h:24,
                 from ../../Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp:26:
../../Source/WTF/wtf/OwnPtr.h: In member function 'virtual void WebKit::ChromeClient::attachRootGraphicsLayer(WebCore::Frame*, WebCore::GraphicsLayer*)':
../../Source/WTF/wtf/OwnPtr.h:97:17: error: 'WTF::OwnPtr<T>& WTF::OwnPtr<T>::operator=(const WTF::OwnPtr<T>&) [with T = WebCore::WidgetBackingStore; WTF::OwnPtr<T> = WTF::OwnPtr<WebCore::WidgetBackingStore>]' is private
../../Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp:1072:96: error: within this context
In file included from ../../Source/WTF/wtf/VectorTraits.h:25:0,
                 from ../../Source/WTF/wtf/Vector.h:31,
                 from ../../Source/WebCore/platform/graphics/FloatRect.h:31,
                 from ../../Source/WebCore/platform/graphics/FloatQuad.h:33,
                 from ../../Source/WebCore/accessibility/AccessibilityObject.h:33,
                 from ../../Source/WebCore/accessibility/AXObjectCache.h:29,
                 from ../../Source/WebCore/page/ChromeClient.h:25,
                 from ../../Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.h:24,
                 from ../../Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp:26:
../../Source/WTF/wtf/OwnPtr.h:97:17: error: 'WTF::OwnPtr<T>& WTF::OwnPtr<T>::operator=(const WTF::OwnPtr<T>&) [with T = WebCore::WidgetBackingStore; WTF::OwnPtr<T> = WTF::OwnPtr<WebCore::WidgetBackingStore>]' is private
../../Source/WebKit/gtk/WebCoreSupport/ChromeClientGtk.cpp:1076:115: error: within this context
make[1]: *** [Source/WebKit/gtk/WebCoreSupport/libwebkitgtk_3_0_la-ChromeClientGtk.lo] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory `/home/shivamidow/Projects/WebKit/WebKitBuild/Release'
make: *** [all] Error 2

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