Title: [104260] trunk/Source/WebCore
Revision
104260
Author
[email protected]
Date
2012-01-05 18:13:29 -0800 (Thu, 05 Jan 2012)

Log Message

https://bugs.webkit.org/show_bug.cgi?id=75593
Patch by Yongjun Zhang <[email protected]> on 2012-01-05
Reviewed by Alexey Proskuryakov.

Null-check 'page' variable before use, to follow the common usage pattern of m_frame->page()
throughout the rest of FrameView.cpp.

* page/FrameView.cpp:
(WebCore::FrameView::notifyPageThatContentAreaWillPaint):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (104259 => 104260)


--- trunk/Source/WebCore/ChangeLog	2012-01-06 01:58:34 UTC (rev 104259)
+++ trunk/Source/WebCore/ChangeLog	2012-01-06 02:13:29 UTC (rev 104260)
@@ -1,3 +1,14 @@
+2012-01-05  Yongjun Zhang  <[email protected]>
+
+        https://bugs.webkit.org/show_bug.cgi?id=75593
+        Reviewed by Alexey Proskuryakov.
+
+        Null-check 'page' variable before use, to follow the common usage pattern of m_frame->page()
+        throughout the rest of FrameView.cpp.
+
+        * page/FrameView.cpp:
+        (WebCore::FrameView::notifyPageThatContentAreaWillPaint):
+
 2012-01-04  Hajime Morrita  <[email protected]>
 
         [Refactoring] Moving between TreeScopes should be done by its own class.

Modified: trunk/Source/WebCore/page/FrameView.cpp (104259 => 104260)


--- trunk/Source/WebCore/page/FrameView.cpp	2012-01-06 01:58:34 UTC (rev 104259)
+++ trunk/Source/WebCore/page/FrameView.cpp	2012-01-06 02:13:29 UTC (rev 104260)
@@ -2596,6 +2596,9 @@
 void FrameView::notifyPageThatContentAreaWillPaint() const
 {
     Page* page = m_frame->page();
+    if (!page)
+        return;
+
     const HashSet<ScrollableArea*>* scrollableAreas = page->scrollableAreaSet();
     if (!scrollableAreas)
         return;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to