Title: [146815] trunk/Source/WebCore
Revision
146815
Author
mikhail.pozdnya...@intel.com
Date
2013-03-25 13:53:14 -0700 (Mon, 25 Mar 2013)

Log Message

Remove unused code from Frame class destructor
https://bugs.webkit.org/show_bug.cgi?id=113181

Reviewed by Alexey Proskuryakov.

No new tests. No new functionality.

* page/Frame.cpp:
(WebCore::Frame::~Frame):

    Removed the code which could never have been executed
    as m_view had been set to '0'.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (146814 => 146815)


--- trunk/Source/WebCore/ChangeLog	2013-03-25 20:41:58 UTC (rev 146814)
+++ trunk/Source/WebCore/ChangeLog	2013-03-25 20:53:14 UTC (rev 146815)
@@ -1,3 +1,18 @@
+2013-03-25  Mikhail Pozdnyakov  <mikhail.pozdnya...@intel.com>
+
+        Remove unused code from Frame class destructor
+        https://bugs.webkit.org/show_bug.cgi?id=113181
+
+        Reviewed by Alexey Proskuryakov.
+
+        No new tests. No new functionality.
+
+        * page/Frame.cpp:
+        (WebCore::Frame::~Frame):
+
+            Removed the code which could never have been executed
+            as m_view had been set to '0'.
+
 2013-03-25  Eric Seidel  <e...@webkit.org>
 
         Incorrect parsing due to hash collision

Modified: trunk/Source/WebCore/page/Frame.cpp (146814 => 146815)


--- trunk/Source/WebCore/page/Frame.cpp	2013-03-25 20:41:58 UTC (rev 146814)
+++ trunk/Source/WebCore/page/Frame.cpp	2013-03-25 20:53:14 UTC (rev 146815)
@@ -224,11 +224,6 @@
     HashSet<FrameDestructionObserver*>::iterator stop = m_destructionObservers.end();
     for (HashSet<FrameDestructionObserver*>::iterator it = m_destructionObservers.begin(); it != stop; ++it)
         (*it)->frameDestroyed();
-
-    if (m_view) {
-        m_view->hide();
-        m_view->clearFrame();
-    }
 }
 
 bool Frame::inScope(TreeScope* scope) const
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to