Refactor LinkStyle out of HTMLLinkElement

Refactor LinkStyle out of HTMLLinkElement

BUG=

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

[Yoav Weiss, 2016-10-17 17:31:43]

The CQ bit was checked by yoav@yoav.ws to run a CQ dry run

[commit-bot: I haz the power, 2016-10-17 17:32:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-17 17:34:35]

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

[commit-bot: I haz the power, 2016-10-17 17:34:36]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Yoav Weiss, 2016-10-17 17:35:17]

yoav@yoav.ws changed reviewers:
+ csharrison@chromium.org, japhet@chromium.org, kouhei@chromium.org

[Yoav Weiss, 2016-10-17 17:35:18]

Hey gents :)

Link element loading is currently way too messy and needs some love. This is a
first step in cleaning it up, by splitting LinkStyle from HTMLLinkElement. None
of this is really touching loading, but it seems related.

(On a plane with shaky internet so can't rebase. Apologies in advance if it
doesn't apply)

[kouhei (in TOK), 2016-10-17 21:14:29]

kouhei@chromium.org changed reviewers:
+ toyoshim@chromium.org

[kouhei (in TOK), 2016-10-17 21:14:30]

+toyoshim

lgtm % bot failures

[Charlie Harrison, 2016-10-17 21:26:35]

I'll defer to kouhei but this looks good to me too.

https://codereview.chromium.org/2426513003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/LinkStyle.cpp (left):

https://codereview.chromium.org/2426513003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/LinkStyle.cpp:737: DEFINE_TRACE(LinkStyle) {
I think you still need this?

[Yoav Weiss, 2016-10-18 17:21:00]

The CQ bit was checked by yoav@yoav.ws to run a CQ dry run

[commit-bot: I haz the power, 2016-10-18 17:21:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-18 18:48:08]

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

[commit-bot: I haz the power, 2016-10-18 18:48:09]

Dry run: This issue passed the CQ dry run.

[Yoav Weiss, 2016-10-18 21:02:43]

The CQ bit was checked by yoav@yoav.ws to run a CQ dry run

[Yoav Weiss, 2016-10-18 21:03:22]

Thanks for reviewing! :)

https://codereview.chromium.org/2426513003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/LinkStyle.cpp (left):

https://codereview.chromium.org/2426513003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/LinkStyle.cpp:737: DEFINE_TRACE(LinkStyle) {
On 2016/10/17 21:26:35, Charlie Harrison wrote:
> I think you still need this?

Yeah, it stayed on HTMLLinkElement, which doesn't make sense. Moved

[commit-bot: I haz the power, 2016-10-18 21:03:48]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-18 22:27:09]

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

[commit-bot: I haz the power, 2016-10-18 22:27:09]

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

[Takashi Toyoshima, 2016-10-19 07:45:27]

PS3 lgtm

[Yoav Weiss, 2016-10-19 13:42:18]

The CQ bit was checked by yoav@yoav.ws

[Yoav Weiss, 2016-10-19 13:42:18]

The patchset sent to the CQ was uploaded after l-g-t-m from kouhei@chromium.org
Link to the patchset: https://codereview.chromium.org/2426513003/#ps40001
(title: "Review comment")

[commit-bot: I haz the power, 2016-10-19 13:42:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-19 14:17:43]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-21 13:07:56]

Description was changed from

==========
Refactor LinkStyle out of HTMLLinkElement

BUG=
==========

to

==========
Refactor LinkStyle out of HTMLLinkElement

BUG=

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

[commit-bot: I haz the power, 2016-10-21 13:07:57]

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