MD Settings / User Manager: fix blurry avatar icons

MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true

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

[Dan Beam, 2016-08-12 01:26:23]

Description was changed from

==========
MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
==========

to

==========
MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-08-12 01:29:54]

https://codereview.chromium.org/2241563003/diff/1/ui/webui/resources/cr_eleme...
File
ui/webui/resources/cr_elements/cr_profile_avatar_selector/cr_profile_avatar_selector.html
(right):

https://codereview.chromium.org/2241563003/diff/1/ui/webui/resources/cr_eleme...
ui/webui/resources/cr_elements/cr_profile_avatar_selector/cr_profile_avatar_selector.html:22:
#selector .avatar {
needed to override .keyboard-focus

[Dan Beam, 2016-08-12 01:30:47]

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

[commit-bot: I haz the power, 2016-08-12 01:31:04]

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

[Dan Beam, 2016-08-12 01:34:28]

proof i actually tried it ;)
https://bugs.chromium.org/p/chromium/issues/detail?id=637156#c2

[commit-bot: I haz the power, 2016-08-12 02:39:03]

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

[commit-bot: I haz the power, 2016-08-12 02:39:03]

Dry run: This issue passed the CQ dry run.

[noms (inactive), 2016-08-12 04:50:46]

On 2016/08/12 01:34:28, Dan Beam wrote:
> proof i actually tried it ;)
> https://bugs.chromium.org/p/chromium/issues/detail?id=637156#c2

I thiiiiiink you also need to do it on `people_page.html` -- the profile icon
there also uses `src="[[profileIconUrl_]]"`, which is the fuzzy 1x version

[Dan Beam, 2016-08-12 21:07:01]

On 2016/08/12 04:50:46, noms (inactive) wrote:
> On 2016/08/12 01:34:28, Dan Beam wrote:
> > proof i actually tried it ;)
> > https://bugs.chromium.org/p/chromium/issues/detail?id=637156#c2
> 
> I thiiiiiink you also need to do it on `people_page.html` -- the profile icon
> there also uses `src="[[profileIconUrl_]]"`, which is the fuzzy 1x version

Done.

[noms (inactive), 2016-08-12 21:11:22]

On 2016/08/12 21:07:01, Dan Beam wrote:
> On 2016/08/12 04:50:46, noms (inactive) wrote:
> > On 2016/08/12 01:34:28, Dan Beam wrote:
> > > proof i actually tried it ;)
> > > https://bugs.chromium.org/p/chromium/issues/detail?id=637156#c2
> > 
> > I thiiiiiink you also need to do it on `people_page.html` -- the profile
icon
> > there also uses `src="[[profileIconUrl_]]"`, which is the fuzzy 1x version
> 
> Done.

 TIL y'all use @private and not weird underscore prefixes. LGTM!

[Dan Beam, 2016-08-12 21:16:05]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-08-12 21:16:05]

The patchset sent to the CQ was uploaded after l-g-t-m from noms@chromium.org
Link to the patchset: https://codereview.chromium.org/2241563003/#ps60001
(title: "sorry, i spaced")

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

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-12 21:51:55]

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

[commit-bot: I haz the power, 2016-08-12 21:51:55]

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

[Dan Beam, 2016-08-12 22:01:40]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-08-12 22:01:40]

The patchset sent to the CQ was uploaded after l-g-t-m from noms@chromium.org
Link to the patchset: https://codereview.chromium.org/2241563003/#ps80001
(title: "fix tests")

[commit-bot: I haz the power, 2016-08-12 22:02:31]

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-13 00:33:35]

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

[commit-bot: I haz the power, 2016-08-13 00:33:36]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Dan Beam, 2016-08-15 16:42:20]

Description was changed from

==========
MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true
==========

[Dan Beam, 2016-08-15 16:42:28]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-08-15 16:42:57]

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-15 16:47:21]

Description was changed from

==========
MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true
==========

to

==========
MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true
==========

[commit-bot: I haz the power, 2016-08-15 16:47:22]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-15 16:49:00]

Description was changed from

==========
MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true
==========

to

==========
MD Settings / User Manager: fix blurry avatar icons

R=noms@chromium.org
BUG=637156
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOTRY=true

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

[commit-bot: I haz the power, 2016-08-15 16:49:01]

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