Revert of Load data URI images in an async way according to spec

Revert of Load data URI images in an async way according to spec (patchset #5 id:80001 of https://codereview.chromium.org/2173003002/ )

Reason for revert:
Reverting due to https://bugs.chromium.org/p/chromium/issues/detail?id=632277

Seems like the dev tools' screen capture functionality relies on the sync bahavior of data URIs (issue reproduces with the patch, and does not when reverted)

R=pdr, csharrison

Original issue's description:
> Load data URI images in an async way according to spec
>
> According to https://github.com/ResponsiveImagesCG/picture-element/issues/223#issuecomment-69772765
> we seem to be the only browser that loads data URIs immediately (as if they were cached).
> It was introduced as part of the move to async image loading: https://codereview.chromium.org/200923002
>
> This is now causing issues: https://code.google.com/p/chromium/issues/detail?id=469131
>
> This CL changes this behavior to be spec compliant and load data URIs asynchronously, like other resources.
>
> This is relanding https://codereview.chromium.org/1492303003
>
> BUG=514206
>
> Committed: https://crrev.com/d0c9ac3bf5bb4843c2c189a72e7b6c39e6743831
> Cr-Commit-Position: refs/heads/master@{#407616}

TBR=pdr@chromium.org,csharrison@chromium.org,nyquist@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514206
Committed: https://crrev.com/11a418f2c62a7902e0ca4445aa4769a25416e4e0
Cr-Commit-Position: refs/heads/master@{#410391}

[Yoav Weiss, 2016-08-03 04:11:28]

The CQ bit was checked by yoav@yoav.ws

[Yoav Weiss, 2016-08-03 04:11:28]

Created Revert of Load data URI images in an async way according to spec

[commit-bot: I haz the power, 2016-08-03 04:11:38]

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

[Charlie Harrison, 2016-08-03 04:13:29]

lgtm :( was really hoping this time it would go through!

[pdr., 2016-08-03 04:13:46]

noooooo LGTM :(

[commit-bot: I haz the power, 2016-08-03 05:33:28]

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

[commit-bot: I haz the power, 2016-08-03 05:33:29]

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

[Yoav Weiss, 2016-08-08 16:47:50]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2016-08-08 16:48:12]

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-08-08 18:03:40]

Description was changed from

==========
Revert of Load data URI images in an async way according to spec (patchset #5
id:80001 of https://codereview.chromium.org/2173003002/ )

Reason for revert:
Reverting due to https://bugs.chromium.org/p/chromium/issues/detail?id=632277

Seems like the dev tools' screen capture functionality relies on the sync
bahavior of data URIs (issue reproduces with the patch, and does not when
reverted)

R=pdr, csharrison

Original issue's description:
> Load data URI images in an async way according to spec
> 
> According to
https://github.com/ResponsiveImagesCG/picture-element/issues/223#issuecomment...
> we seem to be the only browser that loads data URIs immediately (as if they
were cached).
> It was introduced as part of the move to async image loading:
https://codereview.chromium.org/200923002
> 
> This is now causing issues:
https://code.google.com/p/chromium/issues/detail?id=469131
> 
> This CL changes this behavior to be spec compliant and load data URIs
asynchronously, like other resources.
> 
> This is relanding https://codereview.chromium.org/1492303003
> 
> BUG=514206
> 
> Committed: https://crrev.com/d0c9ac3bf5bb4843c2c189a72e7b6c39e6743831
> Cr-Commit-Position: refs/heads/master@{#407616}

TBR=pdr@chromium.org,csharrison@chromium.org,nyquist@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514206
==========

to

==========
Revert of Load data URI images in an async way according to spec (patchset #5
id:80001 of https://codereview.chromium.org/2173003002/ )

Reason for revert:
Reverting due to https://bugs.chromium.org/p/chromium/issues/detail?id=632277

Seems like the dev tools' screen capture functionality relies on the sync
bahavior of data URIs (issue reproduces with the patch, and does not when
reverted)

R=pdr, csharrison

Original issue's description:
> Load data URI images in an async way according to spec
> 
> According to
https://github.com/ResponsiveImagesCG/picture-element/issues/223#issuecomment...
> we seem to be the only browser that loads data URIs immediately (as if they
were cached).
> It was introduced as part of the move to async image loading:
https://codereview.chromium.org/200923002
> 
> This is now causing issues:
https://code.google.com/p/chromium/issues/detail?id=469131
> 
> This CL changes this behavior to be spec compliant and load data URIs
asynchronously, like other resources.
> 
> This is relanding https://codereview.chromium.org/1492303003
> 
> BUG=514206
> 
> Committed: https://crrev.com/d0c9ac3bf5bb4843c2c189a72e7b6c39e6743831
> Cr-Commit-Position: refs/heads/master@{#407616}

TBR=pdr@chromium.org,csharrison@chromium.org,nyquist@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514206
==========

[commit-bot: I haz the power, 2016-08-08 18:03:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-08 18:05:26]

Description was changed from

==========
Revert of Load data URI images in an async way according to spec (patchset #5
id:80001 of https://codereview.chromium.org/2173003002/ )

Reason for revert:
Reverting due to https://bugs.chromium.org/p/chromium/issues/detail?id=632277

Seems like the dev tools' screen capture functionality relies on the sync
bahavior of data URIs (issue reproduces with the patch, and does not when
reverted)

R=pdr, csharrison

Original issue's description:
> Load data URI images in an async way according to spec
> 
> According to
https://github.com/ResponsiveImagesCG/picture-element/issues/223#issuecomment...
> we seem to be the only browser that loads data URIs immediately (as if they
were cached).
> It was introduced as part of the move to async image loading:
https://codereview.chromium.org/200923002
> 
> This is now causing issues:
https://code.google.com/p/chromium/issues/detail?id=469131
> 
> This CL changes this behavior to be spec compliant and load data URIs
asynchronously, like other resources.
> 
> This is relanding https://codereview.chromium.org/1492303003
> 
> BUG=514206
> 
> Committed: https://crrev.com/d0c9ac3bf5bb4843c2c189a72e7b6c39e6743831
> Cr-Commit-Position: refs/heads/master@{#407616}

TBR=pdr@chromium.org,csharrison@chromium.org,nyquist@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514206
==========

to

==========
Revert of Load data URI images in an async way according to spec (patchset #5
id:80001 of https://codereview.chromium.org/2173003002/ )

Reason for revert:
Reverting due to https://bugs.chromium.org/p/chromium/issues/detail?id=632277

Seems like the dev tools' screen capture functionality relies on the sync
bahavior of data URIs (issue reproduces with the patch, and does not when
reverted)

R=pdr, csharrison

Original issue's description:
> Load data URI images in an async way according to spec
>
> According to
https://github.com/ResponsiveImagesCG/picture-element/issues/223#issuecomment...
> we seem to be the only browser that loads data URIs immediately (as if they
were cached).
> It was introduced as part of the move to async image loading:
https://codereview.chromium.org/200923002
>
> This is now causing issues:
https://code.google.com/p/chromium/issues/detail?id=469131
>
> This CL changes this behavior to be spec compliant and load data URIs
asynchronously, like other resources.
>
> This is relanding https://codereview.chromium.org/1492303003
>
> BUG=514206
>
> Committed: https://crrev.com/d0c9ac3bf5bb4843c2c189a72e7b6c39e6743831
> Cr-Commit-Position: refs/heads/master@{#407616}

TBR=pdr@chromium.org,csharrison@chromium.org,nyquist@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=514206

Committed: https://crrev.com/11a418f2c62a7902e0ca4445aa4769a25416e4e0
Cr-Commit-Position: refs/heads/master@{#410391}
==========

[commit-bot: I haz the power, 2016-08-08 18:05:27]

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