Couple of tweaks for tables:

Couple of tweaks for tables:
. Expose TableHeader::kHorizontalPadding so that it can be used in
  calculating preferred width of a column.
. Make TableHeader::OnMousePressed return false if not the left mouse
  button. This lets context menus on an ancestor work.
. Exposes couple of methods on TableView used by TaskManager.
. Fixes sizes in TableView.
. Turns on grouping in table example.

BUG=none
TEST=none
R=sadrul@chromium.org


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

[tfarina, 2013-01-08 23:00:19]

On Tue, Jan 8, 2013 at 8:44 PM,  <sky@chromium.org> wrote:
> Reviewers: sadrul,
>
> Description:
> Couple of tweaks for tables:
> . Expose TableHeader::kHorizontalPadding so that it can be used in
>   calculating preferred width of a column.
> . Make TableHeader::OnMousePressed return false if not the left mouse
>   button. This lets contenxt menus on an ancestor work.
s/contenxt/context

> Please review this at https://codereview.chromium.org/11827007/
>

-- 
Thiago

[tfarina, 2013-01-08 23:01:24]

https://codereview.chromium.org/11827007/diff/1007/ui/views/examples/table_ex...
File ui/views/examples/table_example.h (right):

https://codereview.chromium.org/11827007/diff/1007/ui/views/examples/table_ex...
ui/views/examples/table_example.h:34: public views::TableGrouper,
nit: you shouldn't need views:: here, as TableExample is under views::examples
namespace.

[sky, 2013-01-08 23:17:15]

nits fixed

[sadrul, 2013-01-09 17:52:48]

LGTM

[commit-bot: I haz the power, 2013-01-09 17:58:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/11827007/5001

[commit-bot: I haz the power, 2013-01-09 18:54:24]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-01-09 19:00:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/11827007/10002

[commit-bot: I haz the power, 2013-01-09 23:13:39]

Change committed as 175912