Remove ScheduleAllView class from wrench_menu.cc

Remove ScheduleAllView class from wrench_menu.cc

Now that it is only used in one place, this class needlessly increases inheritance complexity

BUG=none


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

[Evan Stade, 2013-01-04 00:30:41]

tangential question, Scott: I noticed this while looking into adding button
items to a different menu. Is there a reason why button items are not supported
at a more generic layer than wrench_menu.cc? Was it a matter of simply not
expecting them to be used elsewhere, or is there something that makes supporting
them in MenuItemView particularly difficult?

[tfarina, 2013-01-04 19:15:34]

https://codereview.chromium.org/11753018/diff/1/chrome/browser/ui/views/wrenc...
File chrome/browser/ui/views/wrench_menu.cc (right):

https://codereview.chromium.org/11753018/diff/1/chrome/browser/ui/views/wrenc...
chrome/browser/ui/views/wrench_menu.cc:328:
View::SchedulePaintInRect(gfx::Rect(0, 0, width(), height()));
nit: gfx::Rect(size())

[sky, 2013-01-07 16:28:35]

LGTM

Elliot added support for buttons in the menu model after this code was added. No
one has tried to convert WrenchMenu to support buttons from the model.

[commit-bot: I haz the power, 2013-01-07 20:36:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/11753018/13001

[commit-bot: I haz the power, 2013-01-07 22:47:35]

Retried try job too often on win_aura for step(s) interactive_ui_tests

[commit-bot: I haz the power, 2013-01-08 02:07:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/estade@chromium.org/11753018/13001

[commit-bot: I haz the power, 2013-01-08 03:20:59]

Change committed as 175484