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