Disable extension Panels on Dev and Canary, except for the whitelist

Disable extension Panels on Dev and Canary, except for the whitelist

They're already disabled on Beta and Stable channels. Leaving them
enabled in Dev/Canary confuses users and makes people suspect
regressions.

After this, panels are only enabled when --enable-panels is set in
chrome://flags and for the whitelist (Hangouts).

CL also tries to consolidate all the Hangouts extension IDs scattered
around the codebase.

BUG=467808, 526708
Committed: https://crrev.com/841c14b4ca8818f08890157ba159e331a1995453
Cr-Commit-Position: refs/heads/master@{#347847}

[tapted, 2015-09-07 04:23:48]

tapted@chromium.org changed reviewers:
+ benwells@chromium.org

[tapted, 2015-09-07 05:30:27]

Hi Ben, please take a look

https://codereview.chromium.org/1329033002/diff/20001/chrome/browser/ui/app_l...
File chrome/browser/ui/app_list/search/people/people_result.cc (left):

https://codereview.chromium.org/1329033002/diff/20001/chrome/browser/ui/app_l...
chrome/browser/ui/app_list/search/people/people_result.cc:192: // See
crbug.com/306672
This has been closed WontFix, so I don't think this TODO will even happen..

https://codereview.chromium.org/1329033002/diff/20001/chrome/browser/ui/panel...
File chrome/browser/ui/panels/panel_manager.cc (right):

https://codereview.chromium.org/1329033002/diff/20001/chrome/browser/ui/panel...
chrome/browser/ui/panels/panel_manager.cc:130: return true;
To keep things simple, this would now return true for the Hangouts packaged
apps, but ShouldUsePanels() is never called for packaged apps, so it shouldn't
matter.

[benwells, 2015-09-07 23:22:32]

lgtm

[Dmitry Titov, 2015-09-08 04:39:17]

thanks for doing that :)

[tapted, 2015-09-08 04:41:07]

tapted@chromium.org changed reviewers:
+ dimich@chromium.org

[tapted, 2015-09-08 04:41:07]

oh hi! I actually need an OWNER for the
chrome/browser/ui/panels/panel_manager.cc - dimich - could you take a peek?
Thanks!

[Dmitry Titov, 2015-09-08 17:16:28]

LGTM for panel_manager.cc

[tapted, 2015-09-08 23:38:29]

The CQ bit was checked by tapted@chromium.org

[commit-bot: I haz the power, 2015-09-08 23:38:53]

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

[commit-bot: I haz the power, 2015-09-09 00:34:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-09 00:34:46]

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