Changeset 151353 in webkit

Timestamp:

Jun 8, 2013 11:36:47 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[Coordinated Graphics] Pass the ownership of GraphicsSurface explicitly
​https://bugs.webkit.org/show_bug.cgi?id=117379

Patch by Jae Hyun Park <​jae.park@company100.net> on 2013-06-08
Reviewed by Noam Rosenthal.

When creating WebCoordinatedSurface with GraphicsSurface, it is clearer
to pass the ownership of the created GraphicsSurface explicitly.

• Shared/CoordinatedGraphics/WebCoordinatedSurface.cpp:

(WebKit::WebCoordinatedSurface::create):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• Shared/CoordinatedGraphics/WebCoordinatedSurface.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2013-06-08  Jae Hyun Park  <jae.park@company100.net>
+
+        [Coordinated Graphics] Pass the ownership of GraphicsSurface explicitly
+        https://bugs.webkit.org/show_bug.cgi?id=117379
+
+        Reviewed by Noam Rosenthal.
+
+        When creating WebCoordinatedSurface with GraphicsSurface, it is clearer
+        to pass the ownership of the created GraphicsSurface explicitly.
+
+        * Shared/CoordinatedGraphics/WebCoordinatedSurface.cpp:
+        (WebKit::WebCoordinatedSurface::create):
+
 2013-06-07  Anders Carlsson  <andersca@apple.com>
 

trunk/Source/WebKit2/Shared/CoordinatedGraphics/WebCoordinatedSurface.cpp

@@ -157 +157 @@
         RefPtr<GraphicsSurface> surface = GraphicsSurface::create(handle.m_size, surfaceFlags, handle.m_graphicsSurfaceToken);
         if (surface)
-            return adoptRef(new WebCoordinatedSurface(handle.m_size, handle.m_flags, PassRefPtr<GraphicsSurface>(surface)));
+            return adoptRef(new WebCoordinatedSurface(handle.m_size, handle.m_flags, surface.release()));
     }
 #endif