Reland: Split the code for fetching images and for decoding images into two separate classes. This w

Reland: https://codereview.chromium.org/2045233002/ Split the code for fetching images and for decoding images into two separate classes. This will all…

Marc approved the small diff.

So I'll TBR the original reviewers.

BUG=609127
TBR=bauerb, mmenke, blundell

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

[markusheintz_, 2016-06-15 09:02:03]

Description was changed from

==========
Reland:

Split the code for fetching images and for decoding images into two separate
classes. This will allow to use common code for image URL fetching and platform
specific implementations for image decoding.

BUG=609127

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

patch from issue 2045233002 at patchset 340001
(http://crrev.com/2045233002#ps340001)
==========

to

==========
Reland: cl/2045233002/ Split the code for fetching images and for decoding
images into two separate classes. This will all… 

Split the code for fetching images and for decoding images into two separate
classes. This will allow to use common code for image URL fetching and platform
specific implementations for image decoding.

BUG=609127

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

patch from issue 2045233002 at patchset 340001
(http://crrev.com/2045233002#ps340001)
==========

[markusheintz_, 2016-06-15 09:02:17]

markusheintz@chromium.org changed reviewers:
+ treib@chromium.org

[markusheintz_, 2016-06-15 09:02:48]

Marc can you please take a at patchset 1? This should fix the issue that caused
trouble.

[markusheintz_, 2016-06-15 09:06:54]

On 2016/06/15 09:02:48, markusheintz_ wrote:
> Marc can you please take a at patchset 1? This should fix the issue that
caused
> trouble.

Sorry I mean patchset 2

[Marc Treib, 2016-06-15 09:52:15]

On 2016/06/15 09:06:54, markusheintz_ wrote:
> On 2016/06/15 09:02:48, markusheintz_ wrote:
> > Marc can you please take a at patchset 1? This should fix the issue that
> caused
> > trouble.
> 
> Sorry I mean patchset 2

LGTM, but please update the description: Remove everything below the "BUG" line.

[Marc Treib, 2016-06-15 09:53:14]

On 2016/06/15 09:52:15, Marc Treib wrote:
> On 2016/06/15 09:06:54, markusheintz_ wrote:
> > On 2016/06/15 09:02:48, markusheintz_ wrote:
> > > Marc can you please take a at patchset 1? This should fix the issue that
> > caused
> > > trouble.
> > 
> > Sorry I mean patchset 2
> 
> LGTM, but please update the description: Remove everything below the "BUG"
line.

Ah, you'll probably also have to add a "TBR=..." line (add the reviewers from
the original CL)

[markusheintz_, 2016-06-15 09:55:22]

Description was changed from

==========
Reland: cl/2045233002/ Split the code for fetching images and for decoding
images into two separate classes. This will all… 

Split the code for fetching images and for decoding images into two separate
classes. This will allow to use common code for image URL fetching and platform
specific implementations for image decoding.

BUG=609127

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

patch from issue 2045233002 at patchset 340001
(http://crrev.com/2045233002#ps340001)
==========

to

==========
Reland: cl/2045233002/ Split the code for fetching images and for decoding
images into two separate classes. This will all… 

Split the code for fetching images and for decoding images into two separate
classes. This will allow to use common code for image URL fetching and platform
specific implementations for image decoding.

BUG=609127
==========

[markusheintz_, 2016-06-15 09:57:29]

Description was changed from

==========
Reland: cl/2045233002/ Split the code for fetching images and for decoding
images into two separate classes. This will all… 

Split the code for fetching images and for decoding images into two separate
classes. This will allow to use common code for image URL fetching and platform
specific implementations for image decoding.

BUG=609127
==========

to

==========
Reland: https://codereview.chromium.org/2045233002/ Split the code for fetching
images and for decoding images into two separate classes. This will all… 

Split the code for fetching images and for decoding images into two separate
classes. This will allow to use common code for image URL fetching and platform
specific implementations for image decoding.

BUG=609127
==========

[markusheintz_, 2016-06-15 10:01:32]

Description was changed from

==========
Reland: https://codereview.chromium.org/2045233002/ Split the code for fetching
images and for decoding images into two separate classes. This will all… 

Split the code for fetching images and for decoding images into two separate
classes. This will allow to use common code for image URL fetching and platform
specific implementations for image decoding.

BUG=609127
==========

to

==========
Reland: https://codereview.chromium.org/2045233002/ Split the code for fetching
images and for decoding images into two separate classes. This will all… 

Marc approved the small diff.

So I'll TBR the original reviewers.

BUG=609127

TBR=bauerb, mmenke, blundell
==========

[markusheintz_, 2016-06-15 10:01:45]

The CQ bit was checked by markusheintz@chromium.org

[commit-bot: I haz the power, 2016-06-15 10:01:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2064793004/20001

[commit-bot: I haz the power, 2016-06-15 10:17:37]

Description was changed from

==========
Reland: https://codereview.chromium.org/2045233002/ Split the code for fetching
images and for decoding images into two separate classes. This will all… 

Marc approved the small diff.

So I'll TBR the original reviewers.

BUG=609127

TBR=bauerb, mmenke, blundell
==========

to

==========
Reland: https://codereview.chromium.org/2045233002/ Split the code for fetching
images and for decoding images into two separate classes. This will all… 

Marc approved the small diff.

So I'll TBR the original reviewers.

BUG=609127

TBR=bauerb, mmenke, blundell
==========

[commit-bot: I haz the power, 2016-06-15 10:17:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-15 10:17:43]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-15 10:19:35]

Description was changed from

==========
Reland: https://codereview.chromium.org/2045233002/ Split the code for fetching
images and for decoding images into two separate classes. This will all… 

Marc approved the small diff.

So I'll TBR the original reviewers.

BUG=609127

TBR=bauerb, mmenke, blundell
==========

to

==========
Reland: https://codereview.chromium.org/2045233002/ Split the code for fetching
images and for decoding images into two separate classes. This will all…

Marc approved the small diff.

So I'll TBR the original reviewers.

BUG=609127

TBR=bauerb, mmenke, blundell

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

[commit-bot: I haz the power, 2016-06-15 10:19:37]

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