Title: [121437] trunk/Source/WebCore
Revision
121437
Author
[email protected]
Date
2012-06-28 09:43:23 -0700 (Thu, 28 Jun 2012)

Log Message

[GTK] [WK2] Memory leak in ResourceHandleSoup.cpp
https://bugs.webkit.org/show_bug.cgi?id=90168

Patch by Sudarsana Nagineni <[email protected]> on 2012-06-28
Reviewed by Martin Robinson.

Fixed a memory leak in WebCoreSynchronousLoader by using adoptGRef
instead of just getting new reference of GMainLoop.

No new tests. No change in behavior.

* platform/network/soup/ResourceHandleSoup.cpp:
(WebCore::WebCoreSynchronousLoader::WebCoreSynchronousLoader):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (121436 => 121437)


--- trunk/Source/WebCore/ChangeLog	2012-06-28 16:35:33 UTC (rev 121436)
+++ trunk/Source/WebCore/ChangeLog	2012-06-28 16:43:23 UTC (rev 121437)
@@ -1,3 +1,18 @@
+2012-06-28  Sudarsana Nagineni  <[email protected]>
+
+        [GTK] [WK2] Memory leak in ResourceHandleSoup.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=90168
+
+        Reviewed by Martin Robinson.
+
+        Fixed a memory leak in WebCoreSynchronousLoader by using adoptGRef
+        instead of just getting new reference of GMainLoop.
+
+        No new tests. No change in behavior.
+
+        * platform/network/soup/ResourceHandleSoup.cpp:
+        (WebCore::WebCoreSynchronousLoader::WebCoreSynchronousLoader):
+
 2012-06-27  Dana Jansens  <[email protected]>
 
         [chromium] Do not multiply bounds by contentsScale in TiledLayerChromium and CanvasLayerTextureUpdater

Modified: trunk/Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp (121436 => 121437)


--- trunk/Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp	2012-06-28 16:35:33 UTC (rev 121436)
+++ trunk/Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp	2012-06-28 16:43:23 UTC (rev 121437)
@@ -88,7 +88,7 @@
         loadingSynchronousRequest = true;
         GRefPtr<GMainContext> innerMainContext = adoptGRef(g_main_context_new());
         g_main_context_push_thread_default(innerMainContext.get());
-        m_mainLoop = g_main_loop_new(innerMainContext.get(), false);
+        m_mainLoop = adoptGRef(g_main_loop_new(innerMainContext.get(), false));
 
         adjustMaxConnections(1);
     }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to