Remove unused includes from core/html .cpp files

Remove unused includes from core/html .cpp files

Splitting up codereview.chromium.org/15447002 into smaller patches.

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

[pdr., 2013-05-22 04:07:41]

LGTM with nits.

https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLDListEle...
File Source/core/html/HTMLDListElement.cpp (left):

https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLDListEle...
Source/core/html/HTMLDListElement.cpp:26: #include "HTMLNames.h"
iwyu might have gotten a little overzealous here. I think dlTag comes with
HTMLNames.h

https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLDirector...
File Source/core/html/HTMLDirectoryElement.cpp (left):

https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLDirector...
Source/core/html/HTMLDirectoryElement.cpp:26: #include "HTMLNames.h"
Similarly here.

https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLMenuElem...
File Source/core/html/HTMLMenuElement.cpp (left):

https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLMenuElem...
Source/core/html/HTMLMenuElement.cpp:26: #include "HTMLNames.h"
And here.

https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLSpanElem...
File Source/core/html/HTMLSpanElement.cpp (left):

https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLSpanElem...
Source/core/html/HTMLSpanElement.cpp:29: #include "HTMLNames.h"
And here.

[Timothy Loh, 2013-05-22 04:29:44]

On 2013/05/22 04:07:41, pdr
wrote:https://codereview.chromium.org/15422005/diff/1/Source/core/html/HTMLDListElement.cpp#oldcode26
> Source/core/html/HTMLDListElement.cpp:26: #include "HTMLNames.h"
> iwyu might have gotten a little overzealous here. I think dlTag comes with
> HTMLNames.h

Good catch! I was running it as a release compilation so it would've been
ignored as it's in an ASSERT. I'll do future iwyu runs as debug compilations.

[commit-bot: I haz the power, 2013-05-22 04:30:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/timloh@chromium.org/15422005/5001

[commit-bot: I haz the power, 2013-05-22 08:12:45]

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

[commit-bot: I haz the power, 2013-05-22 08:24:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/timloh@chromium.org/15422005/5001

[commit-bot: I haz the power, 2013-05-22 12:23:11]

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

[commit-bot: I haz the power, 2013-05-23 02:05:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/timloh@chromium.org/15422005/5001

[commit-bot: I haz the power, 2013-05-23 02:06:15]

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

Patch:       Source/core/html/HTMLFrameSetElement.cpp
Index: Source/core/html/HTMLFrameSetElement.cpp
diff --git a/Source/core/html/HTMLFrameSetElement.cpp
b/Source/core/html/HTMLFrameSetElement.cpp
index
6699bc8a3f81ae3e0d764e523a875fdff00e9929..2d84006f46f0fd751300b6ea54dbe9f45cee8b2e
100644
--- a/Source/core/html/HTMLFrameSetElement.cpp
+++ b/Source/core/html/HTMLFrameSetElement.cpp
@@ -27,16 +27,11 @@
 #include "CSSPropertyNames.h"
 #include "HTMLNames.h"
 #include "bindings/v8/ScriptEventListener.h"
-#include "core/dom/Attribute.h"
 #include "core/dom/Document.h"
 #include "core/dom/Event.h"
 #include "core/dom/EventNames.h"
 #include "core/dom/MouseEvent.h"
 #include "core/dom/NodeRenderingContext.h"
-#include "core/dom/Text.h"
-#include "core/loader/FrameLoader.h"
-#include "core/loader/FrameLoaderClient.h"
-#include "core/page/Frame.h"
 #include "core/platform/Length.h"
 #include "core/rendering/RenderFrameSet.h"

[commit-bot: I haz the power, 2013-05-23 03:24:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/timloh@chromium.org/15422005/27001

[commit-bot: I haz the power, 2013-05-23 04:33:41]

Change committed as 150957