Title: [136625] trunk/Source/WebKit/chromium
Revision
136625
Author
[email protected]
Date
2012-12-04 20:33:22 -0800 (Tue, 04 Dec 2012)

Log Message

[Chromium] Removed obsolete local variable from WebViewImpl.cpp.
https://bugs.webkit.org/show_bug.cgi?id=104050

Patch by David Dorwin <[email protected]> on 2012-12-04
Reviewed by Kent Tamura.

These lines were added with the file in 50739#L761 but the later use
of this variable at L779 has since been removed.

* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::close):

Modified Paths

Diff

Modified: trunk/Source/WebKit/chromium/ChangeLog (136624 => 136625)


--- trunk/Source/WebKit/chromium/ChangeLog	2012-12-05 04:28:23 UTC (rev 136624)
+++ trunk/Source/WebKit/chromium/ChangeLog	2012-12-05 04:33:22 UTC (rev 136625)
@@ -1,3 +1,16 @@
+2012-12-04  David Dorwin  <[email protected]>
+
+        [Chromium] Removed obsolete local variable from WebViewImpl.cpp.
+        https://bugs.webkit.org/show_bug.cgi?id=104050
+
+        Reviewed by Kent Tamura.
+
+        These lines were added with the file in 50739#L761 but the later use
+        of this variable at L779 has since been removed.
+
+        * src/WebViewImpl.cpp:
+        (WebKit::WebViewImpl::close):
+
 2012-12-04  Alpha Lam  <[email protected]>
 
         [chromium] Implement cache eviction logic in ImageDecodingStore

Modified: trunk/Source/WebKit/chromium/src/WebViewImpl.cpp (136624 => 136625)


--- trunk/Source/WebKit/chromium/src/WebViewImpl.cpp	2012-12-05 04:28:23 UTC (rev 136624)
+++ trunk/Source/WebKit/chromium/src/WebViewImpl.cpp	2012-12-05 04:33:22 UTC (rev 136625)
@@ -1529,15 +1529,12 @@
 
 void WebViewImpl::close()
 {
-    RefPtr<WebFrameImpl> mainFrameImpl;
-
     if (m_page) {
         // Initiate shutdown for the entire frameset.  This will cause a lot of
         // notifications to be sent.
-        if (m_page->mainFrame()) {
-            mainFrameImpl = WebFrameImpl::fromFrame(m_page->mainFrame());
+        if (m_page->mainFrame())
             m_page->mainFrame()->loader()->frameDetached();
-        }
+
         m_page.clear();
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to