Title: [92240] trunk
- Revision
- 92240
- Author
- commit-qu...@webkit.org
- Date
- 2011-08-02 15:19:25 -0700 (Tue, 02 Aug 2011)
Log Message
Missing null check in WebViewImpl::selectionRange
https://bugs.webkit.org/show_bug.cgi?id=65561
Patch by Varun Jain <varunj...@google.com> on 2011-08-02
Reviewed by Darin Fisher.
* Source/WebKit/chromium/src/WebViewImpl.cpp:
Modified Paths
Diff
Modified: trunk/ChangeLog (92239 => 92240)
--- trunk/ChangeLog 2011-08-02 22:17:51 UTC (rev 92239)
+++ trunk/ChangeLog 2011-08-02 22:19:25 UTC (rev 92240)
@@ -1,3 +1,12 @@
+2011-08-02 Varun Jain <varunj...@google.com>
+
+ Missing null check in WebViewImpl::selectionRange
+ https://bugs.webkit.org/show_bug.cgi?id=65561
+
+ Reviewed by Darin Fisher.
+
+ * Source/WebKit/chromium/src/WebViewImpl.cpp:
+
2011-08-01 Scott Graham <scot...@chromium.org>
REGRESSION (r39725?): Resources removed from document can not be freed until the document is deleted
Modified: trunk/Source/WebKit/chromium/src/WebViewImpl.cpp (92239 => 92240)
--- trunk/Source/WebKit/chromium/src/WebViewImpl.cpp 2011-08-02 22:17:51 UTC (rev 92239)
+++ trunk/Source/WebKit/chromium/src/WebViewImpl.cpp 2011-08-02 22:19:25 UTC (rev 92240)
@@ -1549,6 +1549,8 @@
if (!frame)
return false;
RefPtr<Range> selectedRange = frame->selection()->toNormalizedRange();
+ if (!selectedRange)
+ return false;
RefPtr<Range> range(Range::create(selectedRange->startContainer()->document(),
selectedRange->startContainer(),
selectedRange->startOffset(),
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
