reland "views: change WrenchMenu to use each model's command ID's when creating MenuItemView's"

reland "views: change WrenchMenu to use each model's command ID's when creating MenuItemView's"

the original cl (hhttps://codereview.chromium.org/25667005/) was reverted because of crash in bug 304756, this cl relands it with the fixes. 

this is in preparation for dynamic recent tabs model and menu: 

* WrenchMenu: 
- for each menu item, MenuItemView::GetCommand() used to be separate and different from MenuModel::GetCommandIdAt(model_index); they are the same now. 
- different range of command id's are reserved for recent tabs and bookmarks. 
- command id's of all items in the wrench menu and all its submenus can't clash, except for separator and IDC_SHOW_HISTORY, the latter of which is in both wrench menu and recent tabs submenu. 

* RecentTabsSubMenuModel: 
- make all command ids within range, use different ids among device name headers. 

* BookmarkMenuDelegate 
- makes sure only maximum number of menu items allowed is created. 
- add test. 

BUG=256750
TEST=wrench menu and all its submenus still work.

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

[kuan, 2013-10-08 16:38:44]

hi scott,

i've created a new cl for easier review.  1st patch set is the exact cl that was
reverted.  2nd patch set contains the changes to fix the crash w/ too many bmb
items.  so u just need to review the diff between 1st and 2nd patch sets.  btw,
i've also addressed ur comments from https://codereview.chromium.org/26350003/
(which i've now closed to avoid confusion).

ptal.  thx.

[sky, 2013-10-08 20:51:18]

LGTM

[commit-bot: I haz the power, 2013-10-08 20:56:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kuan@chromium.org/26455004/3001

[commit-bot: I haz the power, 2013-10-09 02:29:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kuan@chromium.org/26455004/3001

[commit-bot: I haz the power, 2013-10-09 05:01:55]

Change committed as 227675