Refactor HTMLLinkElement

Refactor HTMLLinkElement

- HTMLLinkElement should create LinkLoader only when needed
- Remove LinkRequestBuilder and have createFetchRequest() method on
  HTMLLinkElement instead
- Make LinkResource simple:
  - Provide access to the owner
  - Interfaces to receive events common to all LinkResources
  - LinkResourceType getter
- Remove shouldLoadResource() from LinkResource
- Remove loadingFrame() from LinkResource

LinkStyle clean up
- Remove LoadReturnValue from LinkStyle. We can just run the
  block inside the last if-clause in process() in
  loadStylesheetIfNeeded() where NotNeeded was returned
- Remove setFetchFollowingCORS() and clearFetchFollowingCORS(). They
  don't add much value for readability. We can just change the variable
  directly
- Add a helper to access m_owner->document() for const methods
- Use the non-const document() helper method where possible

HTMLImportsController
- Move HTMLImport retrieval logic to load() method from LinkImport

BUG=671531

[tyoshino (SeeGerritForStatus), 2016-12-06 08:32:47]

Description was changed from

==========
Refactor HTMLLinkElement

BUG=none
==========

to

==========
Refactor HTMLLinkElement

- HTMLLinkElement should create LinkLoader only when needed
- Remove LinkRequestBuilder and have createFetchRequest() method on
  HTMLLinkElement instead
- Make LinkResource simple:
  - Provide access to the owner
  - Interfaces to receive events common to all LinkResources
  - LinkResourceType getter
- Remove shouldLoadResource() from LinkResource
- Remove loadingFrame() from LinkResource

LinkStyle clean up
- Remove LoadReturnValue from LinkStyle. We can just run the
  block inside the last if-clause in process() in
  loadStylesheetIfNeeded() where NotNeeded was returned
- Remove setFetchFollowingCORS() and clearFetchFollowingCORS(). They
  don't add much value for readability. We can just change the variable
  directly
- Add a helper to access m_owner->document() for const methods
- Use the non-const document() helper method where possible

HTMLImportsController
- Move HTMLImport retrieval logic to load() method from LinkImport

BUG=none
==========

[tyoshino (SeeGerritForStatus), 2016-12-06 08:34:01]

Patchset #1 (id:1) has been deleted

[tyoshino (SeeGerritForStatus), 2016-12-06 08:34:08]

Patchset #1 (id:20001) has been deleted

[tyoshino (SeeGerritForStatus), 2016-12-06 08:40:11]

Description was changed from

==========
Refactor HTMLLinkElement

- HTMLLinkElement should create LinkLoader only when needed
- Remove LinkRequestBuilder and have createFetchRequest() method on
  HTMLLinkElement instead
- Make LinkResource simple:
  - Provide access to the owner
  - Interfaces to receive events common to all LinkResources
  - LinkResourceType getter
- Remove shouldLoadResource() from LinkResource
- Remove loadingFrame() from LinkResource

LinkStyle clean up
- Remove LoadReturnValue from LinkStyle. We can just run the
  block inside the last if-clause in process() in
  loadStylesheetIfNeeded() where NotNeeded was returned
- Remove setFetchFollowingCORS() and clearFetchFollowingCORS(). They
  don't add much value for readability. We can just change the variable
  directly
- Add a helper to access m_owner->document() for const methods
- Use the non-const document() helper method where possible

HTMLImportsController
- Move HTMLImport retrieval logic to load() method from LinkImport

BUG=none
==========

to

==========
Refactor HTMLLinkElement

- HTMLLinkElement should create LinkLoader only when needed
- Remove LinkRequestBuilder and have createFetchRequest() method on
  HTMLLinkElement instead
- Make LinkResource simple:
  - Provide access to the owner
  - Interfaces to receive events common to all LinkResources
  - LinkResourceType getter
- Remove shouldLoadResource() from LinkResource
- Remove loadingFrame() from LinkResource

LinkStyle clean up
- Remove LoadReturnValue from LinkStyle. We can just run the
  block inside the last if-clause in process() in
  loadStylesheetIfNeeded() where NotNeeded was returned
- Remove setFetchFollowingCORS() and clearFetchFollowingCORS(). They
  don't add much value for readability. We can just change the variable
  directly
- Add a helper to access m_owner->document() for const methods
- Use the non-const document() helper method where possible

HTMLImportsController
- Move HTMLImport retrieval logic to load() method from LinkImport

BUG=671531
==========

[tyoshino (SeeGerritForStatus), 2016-12-06 08:40:31]

The CQ bit was checked by tyoshino@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-06 08:40:42]

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