Clear link element sheet before clearing ownerNode.

Clear link element sheet before clearing ownerNode.

clearOwnerNode synchronously updates the list of active stylesheets, and
if the m_sheet member of HTMLLinkElement is set, it is considered to be
an active stylesheet. If it is later added to the StyleResolver, it will
crash when ownerNode and ownerDocument are null.

This was already fixed a long time ago for StyleElement in [1].

[1] https://codereview.chromium.org/13508006

R=dstockwell@chromium.org
BUG=426959
Committed: https://crrev.com/c22d69c2cf412a27f10419bef5c246236b87d70c
Cr-Commit-Position: refs/heads/master@{#361867}

[rune, 2015-11-25 23:28:05]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-25 23:31:59]

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

[commit-bot: I haz the power, 2015-11-26 01:35:31]

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

[commit-bot: I haz the power, 2015-11-26 01:35:31]

Dry run: Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_TIMED_OUT, no build
URL)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)

[rune, 2015-11-26 10:04:47]

rune@opera.com changed reviewers:
+ kochi@chromium.org

[kochi, 2015-11-26 10:50:50]

lgtm

[rune, 2015-11-26 10:54:39]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2015-11-26 10:55:05]

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

[commit-bot: I haz the power, 2015-11-26 10:59:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-26 11:00:32]

Description was changed from

==========
Clear link element sheet before clearing ownerNode.

clearOwnerNode synchronously updates the list of active stylesheets, and
if the m_sheet member of HTMLLinkElement is set, it is considered to be
an active stylesheet. If it is later added to the StyleResolver, it will
crash when ownerNode and ownerDocument are null.

This was already fixed a long time ago for StyleElement in [1].

[1] https://codereview.chromium.org/13508006

R=dstockwell@chromium.org
BUG=426959
==========

to

==========
Clear link element sheet before clearing ownerNode.

clearOwnerNode synchronously updates the list of active stylesheets, and
if the m_sheet member of HTMLLinkElement is set, it is considered to be
an active stylesheet. If it is later added to the StyleResolver, it will
crash when ownerNode and ownerDocument are null.

This was already fixed a long time ago for StyleElement in [1].

[1] https://codereview.chromium.org/13508006

R=dstockwell@chromium.org
BUG=426959

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

[commit-bot: I haz the power, 2015-11-26 11:00:33]

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