Revert of SVGImage: check requestCount() to check all subresources are loaded

Revert of SVGImage: check requestCount() to check all subresources are loaded (patchset #2 id:20001 of https://codereview.chromium.org/1515963005/ )

Reason for revert:
The RELEASE_ASSERT fails reproducibly and causes crashes in the wild.
https://crbug.com/571515

Original issue's description:
> SVGImage: check requestCount() to check all subresources are loaded
>
> Document::loadEventFinished() was used to check all subresources of SVG are
> loaded in RELEASE_ASSERT() added by
> https://chromiumcodereview.appspot.com/22999031.
> However, it is sufficient to check that ResourceFetcher::requestCount() is
> zero and we don't have to wait load event completion of the SVG document.
>
> This CL changes the RELEASE_ASSERT() to check ResourceFetcher::requestCount().
>
> Also, this CL checks the assertion in ImageLoader::notifyFinished(), because
> currentFrameHasSingleSecurityOrigin() can be called anytime after
> |ImageLoader::m_imageComplete| is set to true in notifyFinished() and thus
> the assertion must hold at that point.
>
> This is preparation for https://codereview.chromium.org/1475863005/, which
> will invoke SVG Document's load event async and thus the load event completion
> can be after SVGImage::currentFrameHasSingleSecurityOrigin() is called.
>
> BUG=569511
>
> Committed: https://crrev.com/3065748ccfc14c45da2e151348b85d34d94bf76b
> Cr-Commit-Position: refs/heads/master@{#365164}

TBR=kouhei@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=569511
Committed: https://crrev.com/8a1fcdb2aa873f0fb81e937c4e97156d41351735
Cr-Commit-Position: refs/heads/master@{#366529}

[hiroshige, 2015-12-22 02:58:54]

Created Revert of SVGImage: check requestCount() to check all subresources are
loaded

[commit-bot: I haz the power, 2015-12-22 02:59:16]

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

[commit-bot: I haz the power, 2015-12-22 03:00:14]

Description was changed from

==========
Revert of SVGImage: check requestCount() to check all subresources are loaded
(patchset #2 id:20001 of https://codereview.chromium.org/1515963005/ )

Reason for revert:
The RELEASE_ASSERT fails reproducibly and causes crashes in the wild.
https://crbug.com/571515


Original issue's description:
> SVGImage: check requestCount() to check all subresources are loaded
> 
> Document::loadEventFinished() was used to check all subresources of SVG are
> loaded in RELEASE_ASSERT() added by
> https://chromiumcodereview.appspot.com/22999031.
> However, it is sufficient to check that ResourceFetcher::requestCount() is
> zero and we don't have to wait load event completion of the SVG document.
> 
> This CL changes the RELEASE_ASSERT() to check ResourceFetcher::requestCount().
> 
> Also, this CL checks the assertion in ImageLoader::notifyFinished(), because
> currentFrameHasSingleSecurityOrigin() can be called anytime after
> |ImageLoader::m_imageComplete| is set to true in notifyFinished() and thus
> the assertion must hold at that point.
> 
> This is preparation for https://codereview.chromium.org/1475863005/, which
> will invoke SVG Document's load event async and thus the load event completion
> can be after SVGImage::currentFrameHasSingleSecurityOrigin() is called.
> 
> BUG=569511
> 
> Committed: https://crrev.com/3065748ccfc14c45da2e151348b85d34d94bf76b
> Cr-Commit-Position: refs/heads/master@{#365164}

TBR=kouhei@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=569511
==========

to

==========
Revert of SVGImage: check requestCount() to check all subresources are loaded
(patchset #2 id:20001 of https://codereview.chromium.org/1515963005/ )

Reason for revert:
The RELEASE_ASSERT fails reproducibly and causes crashes in the wild.
https://crbug.com/571515


Original issue's description:
> SVGImage: check requestCount() to check all subresources are loaded
> 
> Document::loadEventFinished() was used to check all subresources of SVG are
> loaded in RELEASE_ASSERT() added by
> https://chromiumcodereview.appspot.com/22999031.
> However, it is sufficient to check that ResourceFetcher::requestCount() is
> zero and we don't have to wait load event completion of the SVG document.
> 
> This CL changes the RELEASE_ASSERT() to check ResourceFetcher::requestCount().
> 
> Also, this CL checks the assertion in ImageLoader::notifyFinished(), because
> currentFrameHasSingleSecurityOrigin() can be called anytime after
> |ImageLoader::m_imageComplete| is set to true in notifyFinished() and thus
> the assertion must hold at that point.
> 
> This is preparation for https://codereview.chromium.org/1475863005/, which
> will invoke SVG Document's load event async and thus the load event completion
> can be after SVGImage::currentFrameHasSingleSecurityOrigin() is called.
> 
> BUG=569511
> 
> Committed: https://crrev.com/3065748ccfc14c45da2e151348b85d34d94bf76b
> Cr-Commit-Position: refs/heads/master@{#365164}

TBR=kouhei@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=569511
==========

[commit-bot: I haz the power, 2015-12-22 03:00:15]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-22 03:01:20]

Description was changed from

==========
Revert of SVGImage: check requestCount() to check all subresources are loaded
(patchset #2 id:20001 of https://codereview.chromium.org/1515963005/ )

Reason for revert:
The RELEASE_ASSERT fails reproducibly and causes crashes in the wild.
https://crbug.com/571515


Original issue's description:
> SVGImage: check requestCount() to check all subresources are loaded
> 
> Document::loadEventFinished() was used to check all subresources of SVG are
> loaded in RELEASE_ASSERT() added by
> https://chromiumcodereview.appspot.com/22999031.
> However, it is sufficient to check that ResourceFetcher::requestCount() is
> zero and we don't have to wait load event completion of the SVG document.
> 
> This CL changes the RELEASE_ASSERT() to check ResourceFetcher::requestCount().
> 
> Also, this CL checks the assertion in ImageLoader::notifyFinished(), because
> currentFrameHasSingleSecurityOrigin() can be called anytime after
> |ImageLoader::m_imageComplete| is set to true in notifyFinished() and thus
> the assertion must hold at that point.
> 
> This is preparation for https://codereview.chromium.org/1475863005/, which
> will invoke SVG Document's load event async and thus the load event completion
> can be after SVGImage::currentFrameHasSingleSecurityOrigin() is called.
> 
> BUG=569511
> 
> Committed: https://crrev.com/3065748ccfc14c45da2e151348b85d34d94bf76b
> Cr-Commit-Position: refs/heads/master@{#365164}

TBR=kouhei@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=569511
==========

to

==========
Revert of SVGImage: check requestCount() to check all subresources are loaded
(patchset #2 id:20001 of https://codereview.chromium.org/1515963005/ )

Reason for revert:
The RELEASE_ASSERT fails reproducibly and causes crashes in the wild.
https://crbug.com/571515

Original issue's description:
> SVGImage: check requestCount() to check all subresources are loaded
>
> Document::loadEventFinished() was used to check all subresources of SVG are
> loaded in RELEASE_ASSERT() added by
> https://chromiumcodereview.appspot.com/22999031.
> However, it is sufficient to check that ResourceFetcher::requestCount() is
> zero and we don't have to wait load event completion of the SVG document.
>
> This CL changes the RELEASE_ASSERT() to check ResourceFetcher::requestCount().
>
> Also, this CL checks the assertion in ImageLoader::notifyFinished(), because
> currentFrameHasSingleSecurityOrigin() can be called anytime after
> |ImageLoader::m_imageComplete| is set to true in notifyFinished() and thus
> the assertion must hold at that point.
>
> This is preparation for https://codereview.chromium.org/1475863005/, which
> will invoke SVG Document's load event async and thus the load event completion
> can be after SVGImage::currentFrameHasSingleSecurityOrigin() is called.
>
> BUG=569511
>
> Committed: https://crrev.com/3065748ccfc14c45da2e151348b85d34d94bf76b
> Cr-Commit-Position: refs/heads/master@{#365164}

TBR=kouhei@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=569511

Committed: https://crrev.com/8a1fcdb2aa873f0fb81e937c4e97156d41351735
Cr-Commit-Position: refs/heads/master@{#366529}
==========

[commit-bot: I haz the power, 2015-12-22 03:01:21]

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