Changeset 63180 in webkit

Timestamp:

Jul 12, 2010 11:53:16 PM (14 years ago)

Author:

eric@webkit.org

Message:

2010-07-12 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix line number handling in HTMLTreeBuilder to fix a zillion layout tests
​https://bugs.webkit.org/show_bug.cgi?id=42143

Covered by a zillion layout tests.

• html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processScriptStartTag):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/HTMLTreeBuilder.cpp (modified) (2 diffs)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-07-12  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Adam Barth.
+
+        Fix line number handling in HTMLTreeBuilder to fix a zillion layout tests
+        https://bugs.webkit.org/show_bug.cgi?id=42143
+
+        Covered by a zillion layout tests.
+
+        * html/HTMLTreeBuilder.cpp:
+        (WebCore::HTMLTreeBuilder::processEndTag):
+        (WebCore::HTMLTreeBuilder::processScriptStartTag):
+
 2010-07-12  Jay Civelli  <jcivelli@chromium.org>
 

trunk/WebCore/html/HTMLTreeBuilder.cpp

@@ -2252 +2252 @@
             ASSERT(m_tree.currentElement()->hasTagName(scriptTag));
             m_scriptToProcess = m_tree.currentElement();
+            m_scriptToProcessStartLine = m_lastScriptElementStartLine + 1;
             m_tree.openElements()->pop();
             setInsertionMode(m_originalInsertionMode);
@@ -2776 +2777 @@
     m_tokenizer->setState(HTMLTokenizer::ScriptDataState);
     m_originalInsertionMode = m_insertionMode;
+    m_lastScriptElementStartLine = m_tokenizer->lineNumber();
     setInsertionMode(TextMode);
 }