App list: The line under folder headings has a fixed margin.

App list: The line under folder headings has a fixed margin.

Previously, it had a fixed width, causing it to be very short on the
wider app list window style. Now it extends (almost) all the way to the
edge regardless of the app list's width.

BUG=415461
Committed: https://crrev.com/2cf06c31177287c9a1aecdf5b5ac6e2148215bf2
Cr-Commit-Position: refs/heads/master@{#295697}

[Matt Giuca, 2014-09-18 07:19:35]

mgiuca@chromium.org changed reviewers:
+ calamity@chromium.org

[calamity, 2014-09-19 09:56:17]

lgtm

https://codereview.chromium.org/581853003/diff/1/ui/app_list/views/folder_hea...
File ui/app_list/views/folder_header_view.cc (right):

https://codereview.chromium.org/581853003/diff/1/ui/app_list/views/folder_hea...
ui/app_list/views/folder_header_view.cc:197: rect.set_width(rect.width() -
kBottomSeparatorPadding * 2);
Use Rect::Inset().

[Matt Giuca, 2014-09-19 09:58:23]

https://codereview.chromium.org/581853003/diff/1/ui/app_list/views/folder_hea...
File ui/app_list/views/folder_header_view.cc (right):

https://codereview.chromium.org/581853003/diff/1/ui/app_list/views/folder_hea...
ui/app_list/views/folder_header_view.cc:197: rect.set_width(rect.width() -
kBottomSeparatorPadding * 2);
On 2014/09/19 09:56:17, calamity wrote:
> Use Rect::Inset().

Done.

[Matt Giuca, 2014-09-19 09:58:35]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2014-09-19 09:59:31]

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

[commit-bot: I haz the power, 2014-09-19 11:11:24]

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

[commit-bot: I haz the power, 2014-09-19 11:11:25]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_swarming on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Matt Giuca, 2014-09-19 13:13:19]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2014-09-19 13:14:18]

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

[commit-bot: I haz the power, 2014-09-19 13:45:05]

Committed patchset #2 (id:20001) as a390d5455d34439a60c187f5d470d29e523e5327

[commit-bot: I haz the power, 2014-09-19 13:45:39]

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