Use apps::IsAppShimsEnabled instead of kEnableAppShims

Use apps::IsAppShimsEnabled instead of kEnableAppShims

Currently the main menu is only modified for app windows when
--enable-app-shims is enabled. This change enables modifying
the main menu whenever app shims are enabled (i.e. when the app
launcher is enabled).

BUG=168080, 276052
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221974

[jackhou1, 2013-09-06 05:43:29]

thakis, please review for OWNERS

[tapted, 2013-09-06 05:50:31]

lgtm

[Nico, 2013-09-08 20:09:11]

Code lgtm (go ahead and land), but I don't understand the CL description. How
does this enable modifying the main menu? Does the main menu modification code
also check this function? Does the IsAppShimsEnabled() do menu
enabling/disabling (probably not)?

[jackhou1, 2013-09-08 23:49:12]

On 2013/09/08 20:09:11, Nico wrote:
> Code lgtm (go ahead and land), but I don't understand the CL description. How
> does this enable modifying the main menu? Does the main menu modification code
> also check this function? Does the IsAppShimsEnabled() do menu
> enabling/disabling (probably not)?

Updated the description. The AppShimMenuController does the main menu
modification, it's only created if IsAppShimsEnabled().

[commit-bot: I haz the power, 2013-09-08 23:49:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jackhou@chromium.org/23558007/1

[commit-bot: I haz the power, 2013-09-09 04:13:26]

Change committed as 221974