Revert icon to unmodified state

Revert icon to unmodified state

BUG=701363
Review-Url: https://codereview.chromium.org/2755483005
Cr-Commit-Position: refs/heads/master@{#458430}
Committed: https://chromium.googlesource.com/chromium/src/+/30d9c1420d887d155f3c4fcbf7011c1e56b922a8

[jasonkliu, 2017-03-15 00:54:42]

jasonkliu@chromium.org changed reviewers:
+ estade@chromium.org, mrefaat@chromium.org

[jasonkliu, 2017-03-15 00:54:42]

PTAL, thanks!

Here is the result of the icon change:
https://drive.google.com/open?id=0B3dPCXKQYa2ddlBjZDEzbHRTek0

The top image is what will be in M58 stable, as crrev/2732493002 didn't land
until after branch point.  The middle image is what is currently live in Canary.
 The bottom image is with this fix.

[Evan Stade, 2017-03-15 15:37:40]

On 2017/03/15 00:54:42, jasonkliu1 wrote:
> PTAL, thanks!
> 
> Here is the result of the icon change:
> https://drive.google.com/open?id=0B3dPCXKQYa2ddlBjZDEzbHRTek0
> 
> The top image is what will be in M58 stable, as crrev/2732493002 didn't land
> until after branch point.  The middle image is what is currently live in
Canary.
>  The bottom image is with this fix.

lgtm, thanks.

[mrefaat, 2017-03-15 15:39:04]

lgtm

[Evan Stade, 2017-03-15 15:39:28]

On 2017/03/15 15:37:40, Evan Stade wrote:
> On 2017/03/15 00:54:42, jasonkliu1 wrote:
> > PTAL, thanks!
> > 
> > Here is the result of the icon change:
> > https://drive.google.com/open?id=0B3dPCXKQYa2ddlBjZDEzbHRTek0
> > 
> > The top image is what will be in M58 stable, as crrev/2732493002 didn't land
> > until after branch point.  The middle image is what is currently live in
> Canary.
> >  The bottom image is with this fix.
> 
> lgtm, thanks.

btw, can you move this icon to chrome/app/vector_icons/ and make it win-only
(appears to only be needed on win?)

[jasonkliu, 2017-03-21 14:34:38]

On 2017/03/15 15:39:28, Evan Stade wrote:
> On 2017/03/15 15:37:40, Evan Stade wrote:
> > On 2017/03/15 00:54:42, jasonkliu1 wrote:
> > > PTAL, thanks!
> > > 
> > > Here is the result of the icon change:
> > > https://drive.google.com/open?id=0B3dPCXKQYa2ddlBjZDEzbHRTek0
> > > 
> > > The top image is what will be in M58 stable, as crrev/2732493002 didn't
land
> > > until after branch point.  The middle image is what is currently live in
> > Canary.
> > >  The bottom image is with this fix.
> > 
> > lgtm, thanks.
> 
> btw, can you move this icon to chrome/app/vector_icons/ and make it win-only
> (appears to only be needed on win?)

The icon is now win-only.  I think it's already in chrome/app/vector_icons/?
PTAL, thanks.

[Evan Stade, 2017-03-21 15:01:40]

lgtm

[jasonkliu, 2017-03-21 15:14:55]

The CQ bit was checked by jasonkliu@chromium.org

[jasonkliu, 2017-03-21 15:14:56]

The patchset sent to the CQ was uploaded after l-g-t-m from mrefaat@chromium.org
Link to the patchset: https://codereview.chromium.org/2755483005/#ps20001
(title: "Icon is for Windows only")

[commit-bot: I haz the power, 2017-03-21 15:15:22]

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-03-21 16:13:03]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490109295762370,
"parent_rev": "de737054e108326b4f591a2313f904a51cfe2676", "commit_rev":
"30d9c1420d887d155f3c4fcbf7011c1e56b922a8"}

[commit-bot: I haz the power, 2017-03-21 16:13:50]

Description was changed from

==========
Revert icon to unmodified state

BUG=701363
==========

to

==========
Revert icon to unmodified state

BUG=701363

Review-Url: https://codereview.chromium.org/2755483005
Cr-Commit-Position: refs/heads/master@{#458430}
Committed:
https://chromium.googlesource.com/chromium/src/+/30d9c1420d887d155f3c4fcbf701...
==========

[commit-bot: I haz the power, 2017-03-21 16:13:53]

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

[jasonkliu, 2017-03-27 23:21:39]

Patchset #3 (id:40001) has been deleted