Commit 3429aa03 authored by timothy@apple.com's avatar timothy@apple.com

Fix a crash in Mail when forwarding a specific HTML email.

        Caused by the fix for <rdar://problem/6512520> (r42096).
        Unable to create a test case that works outside of Mail.

        <rdar://problem/6796069>

        Reviewed by Ada Chan.

        * editing/markup.cpp:
        (WebCore::createMarkup): Null check pastEnd before trying to
        call Range::compareBoundaryPoints.

git-svn-id: svn://svn.chromium.org/blink/trunk@42753 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 37823851
2009-04-22 Timothy Hatcher <timothy@apple.com>
Fix a crash in Mail when forwarding a specific HTML email.
Caused by the fix for <rdar://problem/6512520> (r42096).
Unable to create a test case that works outside of Mail.
<rdar://problem/6796069>
Reviewed by Ada Chan.
* editing/markup.cpp:
(WebCore::createMarkup): Null check pastEnd before trying to
call Range::compareBoundaryPoints.
2009-04-22 Sam Weinig <sam@webkit.org> 2009-04-22 Sam Weinig <sam@webkit.org>
Rubber-stamped by Darin Adler. Rubber-stamped by Darin Adler.
...@@ -808,7 +808,7 @@ String createMarkup(const Range* range, Vector<Node*>* nodes, EAnnotateForInterc ...@@ -808,7 +808,7 @@ String createMarkup(const Range* range, Vector<Node*>* nodes, EAnnotateForInterc
markups.append(interchangeNewlineString); markups.append(interchangeNewlineString);
startNode = visibleStart.next().deepEquivalent().node(); startNode = visibleStart.next().deepEquivalent().node();
if (Range::compareBoundaryPoints(startNode, 0, pastEnd, 0) >= 0) { if (pastEnd && Range::compareBoundaryPoints(startNode, 0, pastEnd, 0) >= 0) {
if (deleteButton) if (deleteButton)
deleteButton->enable(); deleteButton->enable();
return interchangeNewlineString; return interchangeNewlineString;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment