[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

eric at webkit.org eric at webkit.org
Wed Dec 22 11:49:27 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit 42936e10a82bb04a45f89cac6e69593c76a1073c
Author: eric at webkit.org <eric at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Sun Aug 8 07:55:52 2010 +0000

    2010-08-08  Adam Barth  <abarth at webkit.org>
    
            Reviewed by Dimitri Glazkov.
    
            frame()->selection() can't be null
            https://bugs.webkit.org/show_bug.cgi?id=43681
    
            SelectionController is a component of Frame.  If frame is non-null, it
            can't possibly be null.
    
            * html/HTMLAnchorElement.cpp:
            (WebCore::HTMLAnchorElement::setActive):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@64946 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index d05319b..ea68d6c 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -2,6 +2,19 @@
 
         Reviewed by Dimitri Glazkov.
 
+        frame()->selection() can't be null
+        https://bugs.webkit.org/show_bug.cgi?id=43681
+
+        SelectionController is a component of Frame.  If frame is non-null, it
+        can't possibly be null.
+
+        * html/HTMLAnchorElement.cpp:
+        (WebCore::HTMLAnchorElement::setActive):
+
+2010-08-08  Adam Barth  <abarth at webkit.org>
+
+        Reviewed by Dimitri Glazkov.
+
         SVGLength uses ASSERTs to validate user input
         https://bugs.webkit.org/show_bug.cgi?id=43680
 
diff --git a/WebCore/html/HTMLAnchorElement.cpp b/WebCore/html/HTMLAnchorElement.cpp
index 6f9b17c..0d2e92d 100644
--- a/WebCore/html/HTMLAnchorElement.cpp
+++ b/WebCore/html/HTMLAnchorElement.cpp
@@ -237,8 +237,7 @@ void HTMLAnchorElement::setActive(bool down, bool pause)
             // Don't set the link to be active if the current selection is in the same editable block as
             // this link
             case EditableLinkLiveWhenNotFocused:
-                if (down && document()->frame() && document()->frame()->selection() &&
-                    document()->frame()->selection()->rootEditableElement() == rootEditableElement())
+                if (down && document()->frame() && document()->frame()->selection()->rootEditableElement() == rootEditableElement())
                     return;
                 break;
             

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list