Expose chromecast extension ids for usage by cast system tray integration

Expose chromecast extension ids for usage by cast system tray integration

For additional context, see:
- https://codereview.chromium.org/1118613003/
- https://codereview.chromium.org/1115083002/

BUG=433140
Committed: https://crrev.com/a666b56d05b0722ff1a89d22a6f8885fe7a202b8
Cr-Commit-Position: refs/heads/master@{#329451}

[jdufault, 2015-05-07 19:54:57]

jdufault@chromium.org changed reviewers:
+ achuith@chromium.org

[achuithb, 2015-05-08 04:44:43]

https://codereview.chromium.org/1129303002/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/tab_capture/tab_capture_api.cc (right):

https://codereview.chromium.org/1129303002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/tab_capture/tab_capture_api.cc:38: // Whitelisted
extensions that do not check for a browser action grant because
I'd move this code to below the anonymous namespace for consistency with the
rest of the codebase.

[achuithb, 2015-05-08 04:48:31]

achuith@chromium.org changed reviewers:
+ justinlin@chromium.org

[achuithb, 2015-05-08 04:48:31]

Justin, could you please take a look

[achuithb, 2015-05-08 04:52:01]

achuith@chromium.org changed reviewers:
+ hubbe@chromium.org
- justinlin@chromium.org

[achuithb, 2015-05-08 04:52:01]

Looks like Justin is OOO. Hubbe, could you please take a look.

[hubbe, 2015-05-12 17:25:42]

lgtm

https://codereview.chromium.org/1129303002/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/tab_capture/tab_capture_api.cc (right):

https://codereview.chromium.org/1129303002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/tab_capture/tab_capture_api.cc:38: // Whitelisted
extensions that do not check for a browser action grant because
On 2015/05/08 04:44:43, achuithb wrote:
> I'd move this code to below the anonymous namespace for consistency with the
> rest of the codebase.

I also think this should be moved.

[jdufault, 2015-05-12 18:08:54]

https://codereview.chromium.org/1129303002/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/tab_capture/tab_capture_api.cc (right):

https://codereview.chromium.org/1129303002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/tab_capture/tab_capture_api.cc:38: // Whitelisted
extensions that do not check for a browser action grant because
On 2015/05/12 17:25:41, hubbe wrote:
> On 2015/05/08 04:44:43, achuithb wrote:
> > I'd move this code to below the anonymous namespace for consistency with the
> > rest of the codebase.
> 
> I also think this should be moved.

Done.

[jdufault, 2015-05-12 18:16:38]

https://codereview.chromium.org/1129303002/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/api/tab_capture/tab_capture_api.cc (right):

https://codereview.chromium.org/1129303002/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/api/tab_capture/tab_capture_api.cc:38: // Whitelisted
extensions that do not check for a browser action grant because
On 2015/05/12 18:08:54, jdufault wrote:
> On 2015/05/12 17:25:41, hubbe wrote:
> > On 2015/05/08 04:44:43, achuithb wrote:
> > > I'd move this code to below the anonymous namespace for consistency with
the
> > > rest of the codebase.
> > 
> > I also think this should be moved.
> 
> Done.

This broke the build (since kChromecastExtensionIds needs to refer to
extensions::kChromecastExtensionIds), so I moved it to be below the anonymous
namespace but within the extensions namespace.

[jdufault, 2015-05-12 18:17:16]

The CQ bit was checked by jdufault@chromium.org

[jdufault, 2015-05-12 18:17:16]

The patchset sent to the CQ was uploaded after l-g-t-m from hubbe@chromium.org
Link to the patchset: https://codereview.chromium.org/1129303002/#ps40001
(title: "Fix build")

[commit-bot: I haz the power, 2015-05-12 18:18:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1129303002/40001

[commit-bot: I haz the power, 2015-05-12 19:22:43]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-12 19:23:27]

Patchset 3 (id:??) landed as
https://crrev.com/a666b56d05b0722ff1a89d22a6f8885fe7a202b8
Cr-Commit-Position: refs/heads/master@{#329451}