Files.app: Update toolbar layout and icons on it.

Files.app: Update toolbar layout and icons on it.

- Use pixel-perfect PNGs for icons on toolbar, instead of core-icons(SVG).
- Update hover/focus color to be visible on white and blue background.
- Change the shape of hover/focus background from square to circle.
- Some minor layout change to comply with the latest spec.

BUG=402344
TEST=manually compared with the mock.

Committed: https://crrev.com/317f49316a9cf1971caca44c8e9cf85d4d64e7cc
Cr-Commit-Position: refs/heads/master@{#316555}

[fukino, 2015-02-17 09:03:04]

fukino@chromium.org changed reviewers:
+ yawano@chromium.org

[fukino, 2015-02-17 09:03:46]

Could you take a look? Thanks!

[yawano, 2015-02-17 09:36:04]

https://codereview.chromium.org/928353002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/css/file_manager.css (right):

https://codereview.chromium.org/928353002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/css/file_manager.css:516: display: none
Order should be display < margin.

https://codereview.chromium.org/928353002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/main.html (right):

https://codereview.chromium.org/928353002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/main.html:364: <div class="icon"></core-icon>
endTag does not match with startTag.

The background-image is set directly to #view-button. Is this div element still
necessary?

[fukino, 2015-02-17 09:57:25]

Thanks!

https://codereview.chromium.org/928353002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/foreground/css/file_manager.css (right):

https://codereview.chromium.org/928353002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/foreground/css/file_manager.css:516: display: none
On 2015/02/17 09:36:04, yawano wrote:
> Order should be display < margin.

Done.

https://codereview.chromium.org/928353002/diff/20001/ui/file_manager/file_man...
File ui/file_manager/file_manager/main.html (right):

https://codereview.chromium.org/928353002/diff/20001/ui/file_manager/file_man...
ui/file_manager/file_manager/main.html:364: <div class="icon"></core-icon>
On 2015/02/17 09:36:04, yawano wrote:
> endTag does not match with startTag.
> 
> The background-image is set directly to #view-button. Is this div element
still
> necessary? 

Sorry, this tag remained unintentionally.
Thank you for pointing this out!

[yawano, 2015-02-17 09:58:28]

lgtm. Thank you!

[fukino, 2015-02-17 09:59:42]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2015-02-17 10:00:41]

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

[commit-bot: I haz the power, 2015-02-17 10:33:01]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-17 10:33:46]

Patchset 3 (id:??) landed as
https://crrev.com/317f49316a9cf1971caca44c8e9cf85d4d64e7cc
Cr-Commit-Position: refs/heads/master@{#316555}