Multi-Profiles: New Icon drop

Multi-Profiles: New Icon drop

BUG=None
TEST=None

[Miranda Callahan, 2011-07-29 19:34:42]

On 2011/07/29 18:19:16, sail wrote:

Sailesh, when I did this image drop, because the images are a different size
(38x31 instead of 32x32), I saw lots of squishy distortion in fields that I had
to hack around. If it hadn't been for that, I would have made the drop myself
yesterday. Do you see this at all?

[sail, 2011-07-29 20:45:06]

On 2011/07/29 19:34:42, Miranda Callahan wrote:
> On 2011/07/29 18:19:16, sail wrote:
> 
> Sailesh, when I did this image drop, because the images are a different size
> (38x31 instead of 32x32), I saw lots of squishy distortion in fields that I
had
> to hack around.

Is this on Windows? If so what fields? The avatar button in the title bar on
Windows should display 38x31 icons just fine. We already use it to draw the
incognito icon which is also 38x31.

If this is not on Windows then I'm not too worried. As the UI changes I think
it's ok for the Mac/Linux UI to degrade temporarily.

[mirandac, 2011-07-29 22:25:22]

On Fri, Jul 29, 2011 at 16:45, <sail@chromium.org> wrote:

> On 2011/07/29 19:34:42, Miranda Callahan wrote:
>
>> On 2011/07/29 18:19:16, sail wrote:
>>
>
>  Sailesh, when I did this image drop, because the images are a different
>> size
>> (38x31 instead of 32x32), I saw lots of squishy distortion in fields that
>> I
>>
> had
>
>> to hack around.
>>
>
> Is this on Windows? If so what fields? The avatar button in the title bar
> on
> Windows should display 38x31 icons just fine. We already use it to draw the
> incognito icon which is also 38x31.
>
> If this is not on Windows then I'm not too worried. As the UI changes I
> think
> it's ok for the Mac/Linux UI to degrade temporarily.


I saw it on Windows, alas -- saw it in the UI you get to choose a different
profile avatar, specifically.  Also saw it on the faces in the corner of the
frame -- if you look carefully, they are a wee bit squeezed. But in the UI
for choosing a new profile avatar, it's obvious...


>
>
>
http://codereview.chromium.**org/7493082/<http://codereview.chromium.org/7493...
>

[Elliot Glaysher, 2011-08-03 19:41:40]

This looks trivial. LGTM.