Introduce isHTMLHtmlElement and toHTMLHtmlElement

Introduce isHTMLHtmlElement and toHTMLHtmlElement

To avoid direct use of static_cast, this patch introduces isHTMLHtmlElement and toHTMLHtmlElement for code cleanup.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=154113

[jochen (gone - plz use gerrit), 2013-07-12 11:22:49]

lgtm

[commit-bot: I haz the power, 2013-07-12 11:29:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kangil.han@samsung.com/19098002/1

[commit-bot: I haz the power, 2013-07-12 11:30:04]

Failed to apply patch for Source/core/html/MediaDocument.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/html/MediaDocument.cpp
  Hunk #1 FAILED at 72.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/html/MediaDocument.cpp.rej

Patch:       Source/core/html/MediaDocument.cpp
Index: Source/core/html/MediaDocument.cpp
diff --git a/Source/core/html/MediaDocument.cpp
b/Source/core/html/MediaDocument.cpp
index
c3a5511e02b0e11faae1565722cebe9dc7efcbf6..03e176a1ebe6f980854e8ec04fc6ff82d21fbf1e
100644
--- a/Source/core/html/MediaDocument.cpp
+++ b/Source/core/html/MediaDocument.cpp
@@ -72,7 +72,7 @@ void MediaDocumentParser::createDocumentStructure()
     RefPtr<Element> rootElement = document()->createElement(htmlTag, false);
     document()->appendChild(rootElement, IGNORE_EXCEPTION);
     document()->setCSSTarget(rootElement.get());
-    static_cast<HTMLHtmlElement*>(rootElement.get())->insertedByParser();
+    toHTMLHtmlElement(rootElement.get())->insertedByParser();
 
     if (document()->frame())
         document()->frame()->loader()->dispatchDocumentElementAvailable();

[kangil, 2013-07-12 11:30:38]

On 2013/07/12 11:30:04, I haz the power (commit-bot) wrote:
> Failed to apply patch for Source/core/html/MediaDocument.cpp:
> While running patch -p1 --forward --force --no-backup-if-mismatch;
>   patching file Source/core/html/MediaDocument.cpp
>   Hunk #1 FAILED at 72.
>   1 out of 1 hunk FAILED -- saving rejects to file
> Source/core/html/MediaDocument.cpp.rej
> 
> Patch:       Source/core/html/MediaDocument.cpp
> Index: Source/core/html/MediaDocument.cpp
> diff --git a/Source/core/html/MediaDocument.cpp
> b/Source/core/html/MediaDocument.cpp
> index
>
c3a5511e02b0e11faae1565722cebe9dc7efcbf6..03e176a1ebe6f980854e8ec04fc6ff82d21fbf1e
> 100644
> --- a/Source/core/html/MediaDocument.cpp
> +++ b/Source/core/html/MediaDocument.cpp
> @@ -72,7 +72,7 @@ void MediaDocumentParser::createDocumentStructure()
>      RefPtr<Element> rootElement = document()->createElement(htmlTag, false);
>      document()->appendChild(rootElement, IGNORE_EXCEPTION);
>      document()->setCSSTarget(rootElement.get());
> -    static_cast<HTMLHtmlElement*>(rootElement.get())->insertedByParser();
> +    toHTMLHtmlElement(rootElement.get())->insertedByParser();
>  
>      if (document()->frame())
>          document()->frame()->loader()->dispatchDocumentElementAvailable();

Will do rebaseline. :)

[commit-bot: I haz the power, 2013-07-12 11:44:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kangil.han@samsung.com/19098002/7001

[commit-bot: I haz the power, 2013-07-12 12:01:28]

Retried try job too often on linux_layout_rel for step(s) webkit_python_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_layo...

[commit-bot: I haz the power, 2013-07-12 12:04:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kangil.han@samsung.com/19098002/7001

[commit-bot: I haz the power, 2013-07-12 13:30:20]

Change committed as 154113