Changeset 176713 in webkit

Timestamp:

Dec 3, 2014, 2:08:41 AM (12 years ago)

Author:

dino@apple.com

Message:

Missing support for innerHTML on SVGElement
​https://bugs.webkit.org/show_bug.cgi?id=136903

Unreviewed followup from r176630. Minor style nits that I missed in my review.

• dom/Element.h: Remove unnecessary parameter name.
• html/parser/HTMLTreeBuilder.cpp: Whitespace cleanup.

(WebCore::HTMLTreeBuilder::adjustedCurrentStackItem):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/Element.h (modified) (2 diffs)
• html/parser/HTMLTreeBuilder.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-12-02  Dean Jackson  <dino@apple.com>
+
+        Missing support for innerHTML on SVGElement
+        https://bugs.webkit.org/show_bug.cgi?id=136903
+
+        Unreviewed followup from r176630. Minor style nits that I missed in my review.
+
+        * dom/Element.h: Remove unnecessary parameter name.
+        * html/parser/HTMLTreeBuilder.cpp: Whitespace cleanup.
+        (WebCore::HTMLTreeBuilder::adjustedCurrentStackItem):
+
 2014-12-03  Eva Balazsfalvi  <evab.u-szeged@partner.samsung.com>
 

trunk/Source/WebCore/dom/Element.h

@@ -567 +567 @@
     virtual void parserDidSetAttributes();
 
-    void clearTabIndexExplicitlyIfNeeded();    
+    void clearTabIndexExplicitlyIfNeeded();
     void setTabIndexExplicitly(short);
 
@@ -577 +577 @@
     // svgAttributeChanged (called when element.className.baseValue is set)
     void classAttributeChanged(const AtomicString& newClassString);
-    
-    static void mergeWithNextTextNode(Text& node, ExceptionCode& ec);
+
+    static void mergeWithNextTextNode(Text& node, ExceptionCode&);
 
 private:

trunk/Source/WebCore/html/parser/HTMLTreeBuilder.cpp

@@ -2840 +2840 @@
     setInsertionMode(InsertionMode::Text);
 }
-    
+
 // http://www.whatwg.org/specs/web-apps/current-work/#adjusted-current-node
 HTMLStackItem* HTMLTreeBuilder::adjustedCurrentStackItem() const
@@ -2847 +2847 @@
     if (isParsingFragment() && m_tree.openElements()->hasOnlyOneElement())
         return m_fragmentContext.contextElementStackItem();
-        
+
     return m_tree.currentStackItem();
 }
-
 
 // http://www.whatwg.org/specs/web-apps/current-work/multipage/tree-construction.html#tree-construction