Fixed the WallpaperResizer to ensure the image isn't destroyed prematurely.

Fixed the WallpaperResizer to ensure the image isn't destroyed prematurely.

This CL ensures the original wallpaper image isn't prematurely destroyed during
the resize task.

BUG=NONE

Review-Url: https://codereview.chromium.org/2859543002
Cr-Commit-Position: refs/heads/master@{#468735}
Committed: https://chromium.googlesource.com/chromium/src/+/0e4f55fe3a84ef27f58e16f9071ce4ef9f7bce93

[bruthig, 2017-05-02 17:51:30]

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

[commit-bot: I haz the power, 2017-05-02 17:51:48]

Dry run: CQ is trying da patch.

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

[bruthig, 2017-05-02 17:52:26]

bruthig@chromium.org changed reviewers:
+ xdai@chromium.org

[bruthig, 2017-05-02 17:52:27]

xdai@, can you PTAL?

[xdai1, 2017-05-02 18:23:07]

lgtm

https://codereview.chromium.org/2859543002/diff/1/components/wallpaper/wallpa...
File components/wallpaper/wallpaper_resizer.cc (right):

https://codereview.chromium.org/2859543002/diff/1/components/wallpaper/wallpa...
components/wallpaper/wallpaper_resizer.cc:27: // Resizes |orig_bitmap| to
|target_size| using |layout| and stores the
nit: update |orig_bitmap| to |image|

[bruthig, 2017-05-02 18:29:56]

Thx!

Sending to CQ...

https://codereview.chromium.org/2859543002/diff/1/components/wallpaper/wallpa...
File components/wallpaper/wallpaper_resizer.cc (right):

https://codereview.chromium.org/2859543002/diff/1/components/wallpaper/wallpa...
components/wallpaper/wallpaper_resizer.cc:27: // Resizes |orig_bitmap| to
|target_size| using |layout| and stores the
On 2017/05/02 18:23:07, xdai1 wrote:
> nit: update |orig_bitmap| to |image|

Done.

[bruthig, 2017-05-02 18:29:59]

The CQ bit was checked by bruthig@chromium.org

[bruthig, 2017-05-02 18:30:00]

The patchset sent to the CQ was uploaded after l-g-t-m from xdai@chromium.org
Link to the patchset: https://codereview.chromium.org/2859543002/#ps20001
(title: "Addressed patch set 1 comments.")

[commit-bot: I haz the power, 2017-05-02 18:30:46]

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, 2017-05-02 19:12:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493749799772400,
"parent_rev": "1dc0db81fb37243afa445082e0f402b3100863b9", "commit_rev":
"0e4f55fe3a84ef27f58e16f9071ce4ef9f7bce93"}

[commit-bot: I haz the power, 2017-05-02 19:12:36]

Description was changed from

==========
Fixed the WallpaperResizer to ensure the image isn't destroyed prematurely.

This CL ensures the original wallpaper image isn't prematurely destroyed during
the resize task.

BUG=NONE
==========

to

==========
Fixed the WallpaperResizer to ensure the image isn't destroyed prematurely.

This CL ensures the original wallpaper image isn't prematurely destroyed during
the resize task.

BUG=NONE

Review-Url: https://codereview.chromium.org/2859543002
Cr-Commit-Position: refs/heads/master@{#468735}
Committed:
https://chromium.googlesource.com/chromium/src/+/0e4f55fe3a84ef27f58e16f9071c...
==========

[commit-bot: I haz the power, 2017-05-02 19:12:38]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/0e4f55fe3a84ef27f58e16f9071c...