Revert of HTMLImageElement: do not use fallback content for ImageDocument

Revert of HTMLImageElement: do not use fallback content for ImageDocument (patchset #2 id:20001 of https://codereview.chromium.org/2343613002/ )

Reason for revert:
Speculatively reverting to fix broken layout tests on Windows (https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win10, https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win7)

Tests are

fast/backgrounds/border-radius-split-background-image.html
fast/backgrounds/border-radius-split-background.html
fast/borders/border-styles-split.html
paint/invalidation/background-resize-height.html

Original issue's description:
> HTMLImageElement: do not use fallback content for ImageDocument
>
> After https://codereview.chromium.org/1879793003,
> ImageLoader::hasPendingActivity() became false for ImageDocument and thus
> HTMLImageElement::selectSourceURL() didn't consider the image as still loading
> during load (|imageStillLoading| became false).
> This caused ImageDocument not to be displayed progressively.
>
> This CL makes HTMLImageElement to use primary content for ImageDocument.
> This causes ImageDocument not to be replaced with fallback content, but I
> expect that is more acceptable than images not displayed progressively.
>
> BUG=632495
>
> Committed: https://crrev.com/c5e43f7edf4e55155fcda51be0285f37b20c2e76
> Cr-Commit-Position: refs/heads/master@{#420432}

TBR=yhirano@chromium.org,japhet@chromium.org,ellyjones@chromium.org,hiroshige@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=632495
Committed: https://crrev.com/70e44d10fc52150a6af5a7de760cb9181587fbb4
Cr-Commit-Position: refs/heads/master@{#420499}

[Garrett Casto, 2016-09-22 23:26:11]

The CQ bit was checked by gcasto@chromium.org

[Garrett Casto, 2016-09-22 23:26:12]

Created Revert of HTMLImageElement: do not use fallback content for
ImageDocument

[commit-bot: I haz the power, 2016-09-22 23:27:03]

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-09-22 23:27:59]

Description was changed from

==========
Revert of HTMLImageElement: do not use fallback content for ImageDocument
(patchset #2 id:20001 of https://codereview.chromium.org/2343613002/ )

Reason for revert:
Speculatively reverting to fix broken layout tests on Windows
(https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win10,
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win7)

Tests are 

fast/backgrounds/border-radius-split-background-image.html 
fast/backgrounds/border-radius-split-background.html
fast/borders/border-styles-split.html
paint/invalidation/background-resize-height.html 

Original issue's description:
> HTMLImageElement: do not use fallback content for ImageDocument
> 
> After https://codereview.chromium.org/1879793003,
> ImageLoader::hasPendingActivity() became false for ImageDocument and thus
> HTMLImageElement::selectSourceURL() didn't consider the image as still loading
> during load (|imageStillLoading| became false).
> This caused ImageDocument not to be displayed progressively.
> 
> This CL makes HTMLImageElement to use primary content for ImageDocument.
> This causes ImageDocument not to be replaced with fallback content, but I
> expect that is more acceptable than images not displayed progressively.
> 
> BUG=632495
> 
> Committed: https://crrev.com/c5e43f7edf4e55155fcda51be0285f37b20c2e76
> Cr-Commit-Position: refs/heads/master@{#420432}

TBR=yhirano@chromium.org,japhet@chromium.org,ellyjones@chromium.org,hiroshige...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=632495
==========

to

==========
Revert of HTMLImageElement: do not use fallback content for ImageDocument
(patchset #2 id:20001 of https://codereview.chromium.org/2343613002/ )

Reason for revert:
Speculatively reverting to fix broken layout tests on Windows
(https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win10,
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win7)

Tests are 

fast/backgrounds/border-radius-split-background-image.html 
fast/backgrounds/border-radius-split-background.html
fast/borders/border-styles-split.html
paint/invalidation/background-resize-height.html 

Original issue's description:
> HTMLImageElement: do not use fallback content for ImageDocument
> 
> After https://codereview.chromium.org/1879793003,
> ImageLoader::hasPendingActivity() became false for ImageDocument and thus
> HTMLImageElement::selectSourceURL() didn't consider the image as still loading
> during load (|imageStillLoading| became false).
> This caused ImageDocument not to be displayed progressively.
> 
> This CL makes HTMLImageElement to use primary content for ImageDocument.
> This causes ImageDocument not to be replaced with fallback content, but I
> expect that is more acceptable than images not displayed progressively.
> 
> BUG=632495
> 
> Committed: https://crrev.com/c5e43f7edf4e55155fcda51be0285f37b20c2e76
> Cr-Commit-Position: refs/heads/master@{#420432}

TBR=yhirano@chromium.org,japhet@chromium.org,ellyjones@chromium.org,hiroshige...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=632495
==========

[commit-bot: I haz the power, 2016-09-22 23:28:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-22 23:30:56]

Description was changed from

==========
Revert of HTMLImageElement: do not use fallback content for ImageDocument
(patchset #2 id:20001 of https://codereview.chromium.org/2343613002/ )

Reason for revert:
Speculatively reverting to fix broken layout tests on Windows
(https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win10,
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win7)

Tests are 

fast/backgrounds/border-radius-split-background-image.html 
fast/backgrounds/border-radius-split-background.html
fast/borders/border-styles-split.html
paint/invalidation/background-resize-height.html 

Original issue's description:
> HTMLImageElement: do not use fallback content for ImageDocument
> 
> After https://codereview.chromium.org/1879793003,
> ImageLoader::hasPendingActivity() became false for ImageDocument and thus
> HTMLImageElement::selectSourceURL() didn't consider the image as still loading
> during load (|imageStillLoading| became false).
> This caused ImageDocument not to be displayed progressively.
> 
> This CL makes HTMLImageElement to use primary content for ImageDocument.
> This causes ImageDocument not to be replaced with fallback content, but I
> expect that is more acceptable than images not displayed progressively.
> 
> BUG=632495
> 
> Committed: https://crrev.com/c5e43f7edf4e55155fcda51be0285f37b20c2e76
> Cr-Commit-Position: refs/heads/master@{#420432}

TBR=yhirano@chromium.org,japhet@chromium.org,ellyjones@chromium.org,hiroshige...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=632495
==========

to

==========
Revert of HTMLImageElement: do not use fallback content for ImageDocument
(patchset #2 id:20001 of https://codereview.chromium.org/2343613002/ )

Reason for revert:
Speculatively reverting to fix broken layout tests on Windows
(https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win10,
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Win7)

Tests are

fast/backgrounds/border-radius-split-background-image.html
fast/backgrounds/border-radius-split-background.html
fast/borders/border-styles-split.html
paint/invalidation/background-resize-height.html

Original issue's description:
> HTMLImageElement: do not use fallback content for ImageDocument
>
> After https://codereview.chromium.org/1879793003,
> ImageLoader::hasPendingActivity() became false for ImageDocument and thus
> HTMLImageElement::selectSourceURL() didn't consider the image as still loading
> during load (|imageStillLoading| became false).
> This caused ImageDocument not to be displayed progressively.
>
> This CL makes HTMLImageElement to use primary content for ImageDocument.
> This causes ImageDocument not to be replaced with fallback content, but I
> expect that is more acceptable than images not displayed progressively.
>
> BUG=632495
>
> Committed: https://crrev.com/c5e43f7edf4e55155fcda51be0285f37b20c2e76
> Cr-Commit-Position: refs/heads/master@{#420432}

TBR=yhirano@chromium.org,japhet@chromium.org,ellyjones@chromium.org,hiroshige...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=632495

Committed: https://crrev.com/70e44d10fc52150a6af5a7de760cb9181587fbb4
Cr-Commit-Position: refs/heads/master@{#420499}
==========

[commit-bot: I haz the power, 2016-09-22 23:30:57]

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

[Henrik Grunell, 2016-09-23 09:22:39]

Doesn't seem to be it. Still fails in
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/1...
(#420574).

[hiroshige, 2016-09-23 13:22:55]

On 2016/09/23 09:22:39, Henrik Grunell wrote:
> Doesn't seem to be it. Still fails in
>
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/1...
> (#420574).

If so, can we reland the original CL?

[hiroshige, 2016-09-23 15:44:43]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2364073002/ by hiroshige@chromium.org.

The reason for reverting is: As per comment #8, this CL wasn't the cause of bot
failures.
https://codereview.chromium.org/2365523003/#msg8
.