Title: [111124] branches/chromium/1025/Source/WebCore/page/DOMWindow.cpp
- Revision
- 111124
- Author
- [email protected]
- Date
- 2012-03-17 11:53:23 -0700 (Sat, 17 Mar 2012)
Log Message
Merge 106874 - postMessage leaks MemoryEvent object
https://bugs.webkit.org/show_bug.cgi?id=77893
Patch by Greg Simon <[email protected]> on 2012-02-06
Reviewed by Adam Barth.
Changed from PassRefPtr().leakRef() to RefPtr.get() when
calling into FrameLoaderClient to stop +1 extra ref.
* page/DOMWindow.cpp:
(WebCore::DOMWindow::postMessageTimerFired):
[email protected]
Review URL: https://chromiumcodereview.appspot.com/9717030
Modified Paths
Diff
Modified: branches/chromium/1025/Source/WebCore/page/DOMWindow.cpp (111123 => 111124)
--- branches/chromium/1025/Source/WebCore/page/DOMWindow.cpp 2012-03-17 18:19:55 UTC (rev 111123)
+++ branches/chromium/1025/Source/WebCore/page/DOMWindow.cpp 2012-03-17 18:53:23 UTC (rev 111124)
@@ -872,7 +872,7 @@
// Give the embedder a chance to intercept this postMessage because this
// DOMWindow might be a proxy for another in browsers that support
// postMessage calls across WebKit instances.
- if (isCurrentlyDisplayedInFrame() && m_frame->loader()->client()->willCheckAndDispatchMessageEvent(timer->targetOrigin(), PassRefPtr<MessageEvent>(event).leakRef()))
+ if (isCurrentlyDisplayedInFrame() && m_frame->loader()->client()->willCheckAndDispatchMessageEvent(timer->targetOrigin(), event.get()))
return;
if (timer->targetOrigin()) {
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
