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

Wed Dec 22 11:24:07 UTC 2010

The following commit has been merged in the debian/experimental branch:
commit 84b2c9b6f34d45a47a8773e3afd6c995c9ada529
Author: jschuh at chromium.org <jschuh at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Jul 22 00:36:38 2010 +0000

    2010-07-21  Justin Schuh  <jschuh at chromium.org>
    
            Reviewed by Oliver Hunt.
    
            Prevent DeleteButtonController enable state from changing when not editing
            https://bugs.webkit.org/show_bug.cgi?id=42659
    
            Test: svg/custom/use-invalid-html.xhtml
    
            * dom/ContainerNode.cpp:
            (WebCore::ContainerNode::cloneChildNodes):
    2010-07-21  Justin Schuh  <jschuh at chromium.org>
    
            Reviewed by Oliver Hunt.
    
            Prevent DeleteButtonController enable state from changing when not editing
            https://bugs.webkit.org/show_bug.cgi?id=42659
    
            * svg/custom/use-invalid-html-expected.txt: Added.
            * svg/custom/use-invalid-html.xhtml: Added.
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@63865 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index 326494f..5ecf23e 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,13 @@
+2010-07-21  Justin Schuh  <jschuh at chromium.org>
+
+        Reviewed by Oliver Hunt.
+
+        Prevent DeleteButtonController enable state from changing when not editing
+        https://bugs.webkit.org/show_bug.cgi?id=42659
+
+        * svg/custom/use-invalid-html-expected.txt: Added.
+        * svg/custom/use-invalid-html.xhtml: Added.
+
 2010-07-21  Beth Dakin  <bdakin at apple.com>
 
         Reviewed by Dan Bernstein.
diff --git a/LayoutTests/svg/custom/use-invalid-html-expected.txt b/LayoutTests/svg/custom/use-invalid-html-expected.txt
new file mode 100644
index 0000000..7e333c3
--- /dev/null
+++ b/LayoutTests/svg/custom/use-invalid-html-expected.txt
@@ -0,0 +1,6 @@
+This page contains the following errors:
+
+error on line 19 at column 92: Comment must not contain '--' (double-hyphen)
+Below is a rendering of the page up to the first error.
+
+This test passes if a malformed html document triggers an error, not crash.
diff --git a/LayoutTests/svg/custom/use-invalid-html.xhtml b/LayoutTests/svg/custom/use-invalid-html.xhtml
new file mode 100644
index 0000000..98aea28
--- /dev/null
+++ b/LayoutTests/svg/custom/use-invalid-html.xhtml
@@ -0,0 +1,19 @@
+<html>This test passes if a malformed html document triggers an error, not crash.
+  <svg xmlns="http://www.w3.org/2000/svg"
+      xmlns:xlink="http://www.w3.org/1999/xlink"
+      onload="restyle()">
+    <script>
+    if (window.layoutTestController)
+        layoutTestController.dumpAsText();
+    function restyle() {
+        document.getElementById("b").style.position = "absolute";
+    }
+    </script>
+    <defs>
+        <g id="a" />
+    </defs>
+    <g id="b">
+        <use xlink:href="#a" />
+    </g>
+  </svg>
+<!-- no closing html tag --->
\ No newline at end of file
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index be4c2aa..3aa7b8c 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,15 @@
+2010-07-21  Justin Schuh  <jschuh at chromium.org>
+
+        Reviewed by Oliver Hunt.
+
+        Prevent DeleteButtonController enable state from changing when not editing
+        https://bugs.webkit.org/show_bug.cgi?id=42659
+
+        Test: svg/custom/use-invalid-html.xhtml
+
+        * dom/ContainerNode.cpp:
+        (WebCore::ContainerNode::cloneChildNodes):
+
 2010-07-21  Beth Dakin  <bdakin at apple.com>
 
         Reviewed by Dan Bernstein.
diff --git a/WebCore/dom/ContainerNode.cpp b/WebCore/dom/ContainerNode.cpp
index 23b68ba..e559ba7 100644
--- a/WebCore/dom/ContainerNode.cpp
+++ b/WebCore/dom/ContainerNode.cpp
@@ -694,12 +694,13 @@ void ContainerNode::childrenChanged(bool changedByParser, Node* beforeChange, No
 void ContainerNode::cloneChildNodes(ContainerNode *clone)
 {
     // disable the delete button so it's elements are not serialized into the markup
-    if (document()->frame())
+    bool isEditorEnabled = document()->frame() && document()->frame()->editor()->canEdit();
+    if (isEditorEnabled)
         document()->frame()->editor()->deleteButtonController()->disable();
     ExceptionCode ec = 0;
     for (Node* n = firstChild(); n && !ec; n = n->nextSibling())
         clone->appendChild(n->cloneNode(true), ec);
-    if (document()->frame())
+    if (isEditorEnabled && document()->frame())
         document()->frame()->editor()->deleteButtonController()->enable();
 }
 

-- 
WebKit Debian packaging

