Title: [157444] trunk/Source/WebCore
Revision
157444
Author
[email protected]
Date
2013-10-15 00:02:55 -0700 (Tue, 15 Oct 2013)

Log Message

Remove redundant Document::getElementById
https://bugs.webkit.org/show_bug.cgi?id=122813

Reviewed by Andreas Kling.

Merge https://chromium.googlesource.com/chromium/blink/+/4e8f1c5316415614b84370c602beae4a1008299f

This function simply calls virtual TreeScope::getElementById and Document inherits from TreeScope.

* WebCore.exp.in:
* dom/Document.cpp:
* dom/Document.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (157443 => 157444)


--- trunk/Source/WebCore/ChangeLog	2013-10-15 05:55:40 UTC (rev 157443)
+++ trunk/Source/WebCore/ChangeLog	2013-10-15 07:02:55 UTC (rev 157444)
@@ -1,3 +1,18 @@
+2013-10-15  Ryosuke Niwa  <[email protected]>
+
+        Remove redundant Document::getElementById
+        https://bugs.webkit.org/show_bug.cgi?id=122813
+
+        Reviewed by Andreas Kling.
+
+        Merge https://chromium.googlesource.com/chromium/blink/+/4e8f1c5316415614b84370c602beae4a1008299f
+
+        This function simply calls virtual TreeScope::getElementById and Document inherits from TreeScope.
+
+        * WebCore.exp.in:
+        * dom/Document.cpp:
+        * dom/Document.h:
+
 2013-10-14  Santosh Mahto  <[email protected]>
 
         in safari,the background-color of input[type="search"] can't work

Modified: trunk/Source/WebCore/WebCore.exp.in (157443 => 157444)


--- trunk/Source/WebCore/WebCore.exp.in	2013-10-15 05:55:40 UTC (rev 157443)
+++ trunk/Source/WebCore/WebCore.exp.in	2013-10-15 07:02:55 UTC (rev 157444)
@@ -1649,7 +1649,6 @@
 __ZNK7WebCore7RunLoop9TimerBase8isActiveEv
 __ZNK7WebCore8Document11completeURLERKN3WTF6StringE
 __ZNK7WebCore8Document13axObjectCacheEv
-__ZNK7WebCore8Document14getElementByIdERKN3WTF12AtomicStringE
 __ZNK7WebCore8Document31displayStringModifiedByEncodingERKN3WTF6StringE
 __ZNK7WebCore8Document4bodyEv
 __ZNK7WebCore8Document4pageEv
@@ -1709,6 +1708,7 @@
 __ZNK7WebCore9PageCache10frameCountEv
 __ZNK7WebCore9RenderBox11clientWidthEv
 __ZNK7WebCore9RenderBox12clientHeightEv
+__ZNK7WebCore9TreeScope14getElementByIdERKN3WTF12AtomicStringE
 __ZTVN7WebCore12ChromeClientE
 __ZTVN7WebCore14LoaderStrategyE
 __ZTVN7WebCore14StaticNodeListE

Modified: trunk/Source/WebCore/dom/Document.cpp (157443 => 157444)


--- trunk/Source/WebCore/dom/Document.cpp	2013-10-15 05:55:40 UTC (rev 157443)
+++ trunk/Source/WebCore/dom/Document.cpp	2013-10-15 07:02:55 UTC (rev 157444)
@@ -656,11 +656,6 @@
 #endif
 }
 
-Element* Document::getElementById(const AtomicString& id) const
-{
-    return TreeScope::getElementById(id);
-}
-
 Element* Document::getElementByAccessKey(const String& key)
 {
     if (key.isEmpty())

Modified: trunk/Source/WebCore/dom/Document.h (157443 => 157444)


--- trunk/Source/WebCore/dom/Document.h	2013-10-15 05:55:40 UTC (rev 157443)
+++ trunk/Source/WebCore/dom/Document.h	2013-10-15 07:02:55 UTC (rev 157444)
@@ -249,8 +249,6 @@
     using ContainerNode::ref;
     using ContainerNode::deref;
 
-    Element* getElementById(const AtomicString& id) const;
-
     virtual bool canContainRangeEndPoint() const OVERRIDE { return true; }
 
     Element* getElementByAccessKey(const String& key);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to