chromeos: Cats avatar added to defaults set.

chromeos: Cats avatar added to defaults set.

BUG=151287
Committed: https://crrev.com/b3039edc49734f17374b1d9d3e2cf2435092a9b4
Cr-Commit-Position: refs/heads/master@{#316249}

[merkulova, 2015-02-12 14:06:36]

merkulova@chromium.org changed reviewers:
+ derat@chromium.org, dzhioev@chromium.org

[merkulova, 2015-02-12 14:07:15]

Dan as resources owner.
Pavel as login/* owner.

[Daniel Erat, 2015-02-12 14:18:10]

lgtm with nits. please also update the changelist description to begin with
"chromeos: " so it's easy for people looking at descriptions in the commit
history to see which system this change affected.

https://codereview.chromium.org/920813002/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/login/oobe_screen_user_image.js (right):

https://codereview.chromium.org/920813002/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/login/oobe_screen_user_image.js:306: 
nit: delete extra blank line

https://codereview.chromium.org/920813002/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/user_images_grid.js (right):

https://codereview.chromium.org/920813002/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/user_images_grid.js:642: * @param
{Array.<{url: string, author: string, website: string}>}
this comment doesn't mention a "title" property, but you appear to be using one
below

[merkulova, 2015-02-12 14:32:04]

New patchsets have been uploaded after l-g-t-m from derat@chromium.org

[merkulova, 2015-02-12 14:32:32]

merkulova@chromium.org changed reviewers:
+ nkostylev@chromium.org

[merkulova, 2015-02-12 14:33:06]

https://codereview.chromium.org/920813002/diff/1/chrome/browser/resources/chr...
File chrome/browser/resources/chromeos/login/oobe_screen_user_image.js (right):

https://codereview.chromium.org/920813002/diff/1/chrome/browser/resources/chr...
chrome/browser/resources/chromeos/login/oobe_screen_user_image.js:306: 
On 2015/02/12 14:18:10, Daniel Erat wrote:
> nit: delete extra blank line

Done.

[dzhioev (left Google), 2015-02-13 12:38:09]

On 2015/02/12 14:07:15, merkulova wrote:
> Dan as resources owner.
> Pavel as login/* owner.

LGTM

[Nikita (slow), 2015-02-13 17:53:21]

The CQ bit was checked by nkostylev@chromium.org

[Nikita (slow), 2015-02-13 17:53:21]

lgtm

[commit-bot: I haz the power, 2015-02-13 17:54:21]

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

[commit-bot: I haz the power, 2015-02-13 18:56:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-13 18:57:52]

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