LayoutTests:
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 7 Nov 2006 21:05:01 +0000 (21:05 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 7 Nov 2006 21:05:01 +0000 (21:05 +0000)
        Rolled out change for <rdar://problem/4820814>.

        Beth's planning on landing a new change for it soon, but in the mean time
        we need layout tests succeeding again.

WebCore:

        Rolled out change for <rdar://problem/4820814>.

        Beth's planning on landing a new change for it soon, but in the mean time
        we need layout tests succeeding again.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@17645 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/frames/onlyCommentInIFrame-expected.checksum [deleted file]
LayoutTests/fast/frames/onlyCommentInIFrame-expected.png [deleted file]
LayoutTests/fast/frames/onlyCommentInIFrame-expected.txt [deleted file]
LayoutTests/fast/frames/onlyCommentInIFrame.html [deleted file]
LayoutTests/fast/frames/resources/comment.html [deleted file]
LayoutTests/fast/frames/resources/commentX.xhtml [deleted file]
WebCore/ChangeLog
WebCore/html/HTMLDocument.cpp
WebCore/html/HTMLParser.cpp
WebCore/page/FrameView.cpp
WebCore/rendering/HitTestResult.cpp

index 611fe16264c35be789656198aba3f46cf98556dd..276880e9d292192da0d819d55540ec6c235e7783 100644 (file)
@@ -1,3 +1,10 @@
+2006-11-07  Darin Adler  <darin@apple.com>
+
+        Rolled out change for <rdar://problem/4820814>.
+
+        Beth's planning on landing a new change for it soon, but in the mean time
+        we need layout tests succeeding again.
+
 2006-11-07  David Harrison  <harrison@apple.com>
 
         Reviewed by Darin.
diff --git a/LayoutTests/fast/frames/onlyCommentInIFrame-expected.checksum b/LayoutTests/fast/frames/onlyCommentInIFrame-expected.checksum
deleted file mode 100644 (file)
index c5d7216..0000000
+++ /dev/null
@@ -1 +0,0 @@
-655becf51f13d8a2827fbc4070800270
\ No newline at end of file
diff --git a/LayoutTests/fast/frames/onlyCommentInIFrame-expected.png b/LayoutTests/fast/frames/onlyCommentInIFrame-expected.png
deleted file mode 100644 (file)
index 9fd8d7a..0000000
Binary files a/LayoutTests/fast/frames/onlyCommentInIFrame-expected.png and /dev/null differ
diff --git a/LayoutTests/fast/frames/onlyCommentInIFrame-expected.txt b/LayoutTests/fast/frames/onlyCommentInIFrame-expected.txt
deleted file mode 100644 (file)
index 668b725..0000000
+++ /dev/null
@@ -1,41 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x600
-  RenderBlock {HTML} at (0,0) size 800x600
-    RenderBody {BODY} at (8,8) size 784x584
-      RenderText {#text} at (0,0) size 780x54
-        text run at (0,0) width 780: "The documents in the src attributes of the iframes below only contain comments. This test is for a bug where we would only"
-        text run at (0,18) width 773: "construct RenderViews for such iframes. We need to propery construct an empty frame instead, so the RenderView should"
-        text run at (0,36) width 502: "have a RenderBlock and RenderBody below it in the dump of the RenderTree. "
-      RenderBR {BR} at (502,50) size 0x0
-      RenderPartObject {IFRAME} at (0,54) size 784x10
-        layer at (0,0) size 767x8
-          RenderView at (0,0) size 767x6
-        layer at (0,0) size 767x8
-          RenderBlock {HTML} at (0,0) size 767x8
-            RenderBody {BODY} at (8,8) size 751x0
-      RenderText {#text} at (0,0) size 0x0
-      RenderBR {BR} at (0,0) size 0x0
-      RenderBR {BR} at (0,64) size 0x18
-      RenderPartObject {IFRAME} at (0,82) size 784x10
-        layer at (0,0) size 767x130
-          RenderView at (0,0) size 767x6
-        layer at (0,0) size 767x130
-          RenderBlock {html} at (0,0) size 767x130
-            RenderBody {body} at (8,18) size 751x94
-              RenderBlock (anonymous) at (0,0) size 751x0
-                RenderInline {parsererror} at (0,0) size 0x0 [bgcolor=#FFDDDD] [border: (2px solid #CC7777)]
-              RenderBlock (anonymous) at (0,0) size 751x94
-                RenderBlock {h3} at (0,0) size 751x22
-                  RenderText {#text} at (0,0) size 324x22
-                    text run at (0,0) width 324: "This page contains the following errors:"
-                RenderBlock {div} at (0,40) size 751x14
-                  RenderText {#text} at (0,0) size 490x14
-                    text run at (0,0) width 490: "error on line 1 at column 30: Extra content at the end of the document"
-                    text run at (490,0) width 0: " "
-                RenderBlock {h3} at (0,72) size 751x22
-                  RenderText {#text} at (0,0) size 429x22
-                    text run at (0,0) width 429: "Below is a rendering of the page up to the first error."
-              RenderBlock (anonymous) at (0,112) size 751x0
-                RenderInline {parsererror} at (0,0) size 0x0 [bgcolor=#FFDDDD] [border: (2px solid #CC7777)]
-      RenderText {#text} at (0,0) size 0x0
diff --git a/LayoutTests/fast/frames/onlyCommentInIFrame.html b/LayoutTests/fast/frames/onlyCommentInIFrame.html
deleted file mode 100644 (file)
index 2f9c13a..0000000
+++ /dev/null
@@ -1,11 +0,0 @@
-<HTML>
-
-<BODY>
-The documents in the src attributes of the iframes below only contain comments. This test is for a bug where we would only construct RenderViews for such iframes. We need to propery construct an empty frame instead, so the RenderView should have a RenderBlock and RenderBody below it in the dump of the RenderTree.
-<br>
-<IFRAME src="resources/comment.html" width="100%" height="10"  frameborder=1 ></IFRAME>
-<br>
-<br>
-<IFRAME src="resources/commentX.xhtml" width="100%" height="10"  frameborder=1 ></IFRAME>
-
-</HTML>
\ No newline at end of file
diff --git a/LayoutTests/fast/frames/resources/comment.html b/LayoutTests/fast/frames/resources/comment.html
deleted file mode 100644 (file)
index 252a833..0000000
+++ /dev/null
@@ -1 +0,0 @@
-<!--BBM Banner Placeholder-->
\ No newline at end of file
diff --git a/LayoutTests/fast/frames/resources/commentX.xhtml b/LayoutTests/fast/frames/resources/commentX.xhtml
deleted file mode 100644 (file)
index 252a833..0000000
+++ /dev/null
@@ -1 +0,0 @@
-<!--BBM Banner Placeholder-->
\ No newline at end of file
index c8e1d21ac0ec3ef1c865895ca8e580751098db9e..13132b97a2904f9c2e4039f28cd47717e39bb99a 100644 (file)
@@ -1,3 +1,10 @@
+2006-11-07  Darin Adler  <darin@apple.com>
+
+        Rolled out change for <rdar://problem/4820814>.
+
+        Beth's planning on landing a new change for it soon, but in the mean time
+        we need layout tests succeeding again.
+
 2006-11-07  Darin Adler  <darin@apple.com>
 
         Reviewed by Brady.
index 0c851d0ada1d6bba3785d66fa5aafd01108f7443..cef8df3e69ea247d38a3a072d7b186aded75e1ad 100644 (file)
@@ -126,7 +126,7 @@ Tokenizer *HTMLDocument::createTokenizer()
 
 bool HTMLDocument::childAllowed( Node *newChild )
 {
-    return newChild->hasTagName(htmlTag);
+    return newChild->hasTagName(htmlTag) || newChild->isCommentNode();
 }
 
 PassRefPtr<Element> HTMLDocument::createElement(const String &name, ExceptionCode& ec)
index bd6129bebf98805c69b0feb5eec49960e2eb2bf3..5023ba0a4f71e50d579f09c6b2c18b9c297f0e39 100644 (file)
@@ -305,16 +305,6 @@ bool HTMLParser::handleError(Node* n, bool flat, const AtomicString& localName,
     if (n->isTextNode()) {
         if (current->hasTagName(selectTag))
             return false;
-    } else if (n->isCommentNode() && !head) { 
-        head = new HTMLHeadElement(document);
-        e = head;
-        insertNode(e);
-        if (head) {
-            head->addChild(n);
-            if (!n->attached() && !m_fragment)
-                n->attach();
-        }
-        return true;
     } else if (n->isHTMLElement()) {
         HTMLElement* h = static_cast<HTMLElement*>(n);
         if (h->hasLocalName(trTag) || h->hasLocalName(thTag) || h->hasLocalName(tdTag)) {
index d819487787a5821bcb4adf9a1eea8d82b8fe98e6..b40a136d76a7029cf897712655794041c4eee97c 100644 (file)
@@ -595,11 +595,6 @@ void FrameView::handleMousePressEvent(const PlatformMouseEvent& mouseEvent)
     
     MouseEventWithHitTestResults mev = prepareMouseEvent(false, true, false, mouseEvent);
 
-    if (!mev.targetNode()) {
-        invalidateClick();
-        return;
-    }
-
     Frame* subframe = subframeForTargetNode(mev.targetNode());
     if (subframe && passMousePressEventToSubframe(mev, subframe)) {
         invalidateClick();
index 9e330c101573571e8e6d6b250de1f64cde5060fe..b8550b2124fab585586daec9d1bf37015d8865e7 100644 (file)
@@ -129,14 +129,11 @@ String HitTestResult::spellingToolTip() const
 {
     // Return the tool tip string associated with this point, if any. Only markers associated with bad grammar
     // currently supply strings, but maybe someday markers associated with misspelled words will also.
-    if (!m_innerNonSharedNode)
-        return String();
-    
     DocumentMarker* marker = m_innerNonSharedNode->document()->markerContainingPoint(m_point, DocumentMarker::Grammar);
-    if (!marker)
-        return String();
+    if (marker)
+        return marker->description;
     
-    return marker->description;
+    return String();
 }
 
 String HitTestResult::title() const