Title: [119974] trunk/Source/WebKit/chromium
- Revision
- 119974
- Author
- [email protected]
- Date
- 2012-06-11 06:20:47 -0700 (Mon, 11 Jun 2012)
Log Message
[Chromium] Should call frameDetached() in WebPagePopupImpl::close()
https://bugs.webkit.org/show_bug.cgi?id=88754
Reviewed by Adam Barth.
* src/WebPagePopupImpl.cpp:
(WebKit::WebPagePopupImpl::close): Calls FrameLoader::frameDetached().
Modified Paths
Diff
Modified: trunk/Source/WebKit/chromium/ChangeLog (119973 => 119974)
--- trunk/Source/WebKit/chromium/ChangeLog 2012-06-11 12:49:31 UTC (rev 119973)
+++ trunk/Source/WebKit/chromium/ChangeLog 2012-06-11 13:20:47 UTC (rev 119974)
@@ -1,3 +1,13 @@
+2012-06-11 Kent Tamura <[email protected]>
+
+ [Chromium] Should call frameDetached() in WebPagePopupImpl::close()
+ https://bugs.webkit.org/show_bug.cgi?id=88754
+
+ Reviewed by Adam Barth.
+
+ * src/WebPagePopupImpl.cpp:
+ (WebKit::WebPagePopupImpl::close): Calls FrameLoader::frameDetached().
+
2012-06-11 Min Qin <[email protected]>
Adding a flag to show fullscreen media controls in chromium
Modified: trunk/Source/WebKit/chromium/src/WebPagePopupImpl.cpp (119973 => 119974)
--- trunk/Source/WebKit/chromium/src/WebPagePopupImpl.cpp 2012-06-11 12:49:31 UTC (rev 119973)
+++ trunk/Source/WebKit/chromium/src/WebPagePopupImpl.cpp 2012-06-11 13:20:47 UTC (rev 119974)
@@ -286,6 +286,8 @@
void WebPagePopupImpl::close()
{
+ if (m_page && m_page->mainFrame())
+ m_page->mainFrame()->loader()->frameDetached();
m_page.clear();
m_widgetClient = 0;
deref();
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
