Files.app: Set border-outset for image-border of buttons.

Files.app: Set border-outset for image-border of buttons.

BUG=251685
R=hirono@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207792

[yoshiki, 2013-06-19 10:40:11]

@hirono: could you take a look? Thanks.

[hirono, 2013-06-19 12:34:58]

On 2013/06/19 10:40:11, yoshiki wrote:
> @hirono: could you take a look? Thanks.

It seems that the buttons became a bit larger.
https://docs.google.com/file/d/0B4oFnlAOOE2CNmxPZGhUaUIyUFE/edit?usp=sharing

Could you handle the problem?

[yoshiki, 2013-06-20 03:59:01]

On 2013/06/19 12:34:58, hirono wrote:
> On 2013/06/19 10:40:11, yoshiki wrote:
> > @hirono: could you take a look? Thanks.
> 
> It seems that the buttons became a bit larger.
> https://docs.google.com/file/d/0B4oFnlAOOE2CNmxPZGhUaUIyUFE/edit?usp=sharing
> 
> Could you handle the problem?

Done.

[hirono, 2013-06-20 04:14:59]

On 2013/06/20 03:59:01, yoshiki wrote:
> On 2013/06/19 12:34:58, hirono wrote:
> > On 2013/06/19 10:40:11, yoshiki wrote:
> > > @hirono: could you take a look? Thanks.
> > 
> > It seems that the buttons became a bit larger.
> > https://docs.google.com/file/d/0B4oFnlAOOE2CNmxPZGhUaUIyUFE/edit?usp=sharing
> > 
> > Could you handle the problem?
> 
> Done.

Thank you very much for handling it!
It seems that the width of buttons are also a bit widen.
Could you check it?

[yoshiki, 2013-06-20 04:52:52]

On 2013/06/20 04:14:59, hirono wrote:
> On 2013/06/20 03:59:01, yoshiki wrote:
> > On 2013/06/19 12:34:58, hirono wrote:
> > > On 2013/06/19 10:40:11, yoshiki wrote:
> > > > @hirono: could you take a look? Thanks.
> > > 
> > > It seems that the buttons became a bit larger.
> > >
https://docs.google.com/file/d/0B4oFnlAOOE2CNmxPZGhUaUIyUFE/edit?usp=sharing
> > > 
> > > Could you handle the problem?
> > 
> > Done.
> 
> Thank you very much for handling it!
> It seems that the width of buttons are also a bit widen.
> Could you check it?

Done. But the width of buttons are not specified by the spec (except square
ones), so we doesn't much care about it.

[hirono, 2013-06-20 05:54:10]

On 2013/06/20 04:52:52, yoshiki wrote:
> On 2013/06/20 04:14:59, hirono wrote:
> > On 2013/06/20 03:59:01, yoshiki wrote:
> > > On 2013/06/19 12:34:58, hirono wrote:
> > > > On 2013/06/19 10:40:11, yoshiki wrote:
> > > > > @hirono: could you take a look? Thanks.
> > > > 
> > > > It seems that the buttons became a bit larger.
> > > >
> https://docs.google.com/file/d/0B4oFnlAOOE2CNmxPZGhUaUIyUFE/edit?usp=sharing
> > > > 
> > > > Could you handle the problem?
> > > 
> > > Done.
> > 
> > Thank you very much for handling it!
> > It seems that the width of buttons are also a bit widen.
> > Could you check it?
> 
> Done. But the width of buttons are not specified by the spec (except square
> ones), so we doesn't much care about it.

Thank you very much and I see about the width of buttons!
Could you check two more things?

* During pressing buttons, we see the unexpected inner shadow.
  Could you remove it?

* This is not due to this CL,
  but could you please fix the position of trash button so that is matches the
spec?

https://docs.google.com/file/d/0B4oFnlAOOE2CQWVVc0lKRUFxWFU/edit?usp=sharing

Thank you very much!

[yoshiki, 2013-06-20 06:27:40]

On 2013/06/20 05:54:10, hirono wrote:
> On 2013/06/20 04:52:52, yoshiki wrote:
> > On 2013/06/20 04:14:59, hirono wrote:
> > > On 2013/06/20 03:59:01, yoshiki wrote:
> > > > On 2013/06/19 12:34:58, hirono wrote:
> > > > > On 2013/06/19 10:40:11, yoshiki wrote:
> > > > > > @hirono: could you take a look? Thanks.
> > > > > 
> > > > > It seems that the buttons became a bit larger.
> > > > >
> > https://docs.google.com/file/d/0B4oFnlAOOE2CNmxPZGhUaUIyUFE/edit?usp=sharing
> > > > > 
> > > > > Could you handle the problem?
> > > > 
> > > > Done.
> > > 
> > > Thank you very much for handling it!
> > > It seems that the width of buttons are also a bit widen.
> > > Could you check it?
> > 
> > Done. But the width of buttons are not specified by the spec (except square
> > ones), so we doesn't much care about it.
> 
> Thank you very much and I see about the width of buttons!
> Could you check two more things?
> 
> * During pressing buttons, we see the unexpected inner shadow.
>   Could you remove it?

Done.

> * This is not due to this CL,
>   but could you please fix the position of trash button so that is matches the
> spec?

Done.

> https://docs.google.com/file/d/0B4oFnlAOOE2CQWVVc0lKRUFxWFU/edit?usp=sharing
> 
> Thank you very much!

[hirono, 2013-06-20 07:02:40]

On 2013/06/20 06:27:40, yoshiki wrote:
> On 2013/06/20 05:54:10, hirono wrote:
> > On 2013/06/20 04:52:52, yoshiki wrote:
> > > On 2013/06/20 04:14:59, hirono wrote:
> > > > On 2013/06/20 03:59:01, yoshiki wrote:
> > > > > On 2013/06/19 12:34:58, hirono wrote:
> > > > > > On 2013/06/19 10:40:11, yoshiki wrote:
> > > > > > > @hirono: could you take a look? Thanks.
> > > > > > 
> > > > > > It seems that the buttons became a bit larger.
> > > > > >
> > >
https://docs.google.com/file/d/0B4oFnlAOOE2CNmxPZGhUaUIyUFE/edit?usp=sharing
> > > > > > 
> > > > > > Could you handle the problem?
> > > > > 
> > > > > Done.
> > > > 
> > > > Thank you very much for handling it!
> > > > It seems that the width of buttons are also a bit widen.
> > > > Could you check it?
> > > 
> > > Done. But the width of buttons are not specified by the spec (except
square
> > > ones), so we doesn't much care about it.
> > 
> > Thank you very much and I see about the width of buttons!
> > Could you check two more things?
> > 
> > * During pressing buttons, we see the unexpected inner shadow.
> >   Could you remove it?
> 
> Done.
> 
> > * This is not due to this CL,
> >   but could you please fix the position of trash button so that is matches
the
> > spec?
> 
> Done.
> 
> > https://docs.google.com/file/d/0B4oFnlAOOE2CQWVVc0lKRUFxWFU/edit?usp=sharing
> > 
> > Thank you very much!

lgtm with nit.
Thank you for many times handling!

Could you remove the white line?
https://docs.google.com/file/d/0B4oFnlAOOE2Cb0dtd3FHbDhyTk0/edit?usp=sharing

[yoshiki, 2013-06-21 04:52:53]

thanks!

[commit-bot: I haz the power, 2013-06-21 04:53:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoshiki@chromium.org/17457002/22001

[yoshiki, 2013-06-21 10:29:21]

Committed patchset #6 manually as r207792 (presubmit successful).