Make Editor::tidyUpHTMLStructure() handles BODY element correctly

Make Editor::tidyUpHTMLStructure() handles BODY element correctly

This patch makes |Editor::tidyUpHTMLStructure()| to handle BODY element
correctly when it is document element.

Before this patch, |Editor::tidyUpHTMLStructure()| attempt to move existing
BODY element itself, this patch checks this case and not to move it into
itself.

This patch also exports HEAD element to use in unit test and adds
"HTMLHeadElement.h" into GYPI file for sane.

BUG=595606
TEST=run_webkit_unittests --gtest_filter=EditorTest.tidyUpHTMLStructure*

Committed: https://crrev.com/f0371cdb39e34540ebb9ea0daaa962976b551b56
Cr-Commit-Position: refs/heads/master@{#382811}

[yosin_UTC9, 2016-03-23 05:38:50]

Description was changed from

==========
2016-03-23T13:44:04

BUG=595606
==========

to

==========
Make Editor::tidyUpHTMLStructure() handles BODY element correctly

This patch makes |Editor::tidyUpHTMLStructure()| to handle BODY element
correctly when it is document element.

Before this patch, |Editor::tidyUpHTMLStructure()| attempt to move existing
BODY element itself, this patch checks this case and not to move it into
itself.

This patch also exports HEAD element to use in unit test and adds
"HTMLHeadElement.h" into GYPI file for sane.

BUG=595606
TEST=run_webkit_unittests --gtest_filter=EditorTest.tidyUpHTMLStructure*
==========

[yosin_UTC9, 2016-03-23 05:38:50]

yosin@chromium.org changed reviewers:
+ tkent@chromium.org, yoichio@chromium.org

[yosin_UTC9, 2016-03-23 05:50:20]

PTAL

Actual changes are:

# OLD
  if (document.documentElement())
     body->appendChild(document.documentElement());

# NEW
  if (document.documentElement() && body != document.documentElement())
     body->appendChild(document.documentElement());

[tkent, 2016-03-23 06:04:22]

lgtm.  Thank you for fixing this and adding tests.

[yosin_UTC9, 2016-03-23 06:15:15]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-03-23 06:15:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1828623002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1828623002/1

[commit-bot: I haz the power, 2016-03-23 07:54:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-23 07:54:39]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yosin_UTC9, 2016-03-23 08:14:28]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-03-23 08:14:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1828623002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1828623002/1

[commit-bot: I haz the power, 2016-03-23 09:13:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-23 09:15:41]

Description was changed from

==========
Make Editor::tidyUpHTMLStructure() handles BODY element correctly

This patch makes |Editor::tidyUpHTMLStructure()| to handle BODY element
correctly when it is document element.

Before this patch, |Editor::tidyUpHTMLStructure()| attempt to move existing
BODY element itself, this patch checks this case and not to move it into
itself.

This patch also exports HEAD element to use in unit test and adds
"HTMLHeadElement.h" into GYPI file for sane.

BUG=595606
TEST=run_webkit_unittests --gtest_filter=EditorTest.tidyUpHTMLStructure*
==========

to

==========
Make Editor::tidyUpHTMLStructure() handles BODY element correctly

This patch makes |Editor::tidyUpHTMLStructure()| to handle BODY element
correctly when it is document element.

Before this patch, |Editor::tidyUpHTMLStructure()| attempt to move existing
BODY element itself, this patch checks this case and not to move it into
itself.

This patch also exports HEAD element to use in unit test and adds
"HTMLHeadElement.h" into GYPI file for sane.

BUG=595606
TEST=run_webkit_unittests --gtest_filter=EditorTest.tidyUpHTMLStructure*

Committed: https://crrev.com/f0371cdb39e34540ebb9ea0daaa962976b551b56
Cr-Commit-Position: refs/heads/master@{#382811}
==========

[commit-bot: I haz the power, 2016-03-23 09:15:42]

Patchset 1 (id:??) landed as
https://crrev.com/f0371cdb39e34540ebb9ea0daaa962976b551b56
Cr-Commit-Position: refs/heads/master@{#382811}