Correct ENABLE_MEDIA_ROUTER defines to fix compilation without it.

Correct ENABLE_MEDIA_ROUTER defines to fix compilation without it.

Committed: https://crrev.com/f6d0a228e210cbcd96b6ef494cadbf12c09b62ea
Cr-Commit-Position: refs/heads/master@{#334134}

[mharanczyk, 2015-06-11 16:21:43]

mharanczyk@opera.com changed reviewers:
+ brettw@chromium.org, finnur@chromium.org, rdevlin.cronin@chromium.org

[mharanczyk, 2015-06-11 16:24:08]

Simple addition/move of define ENABLE_MEDIA_ROUTER where needed, please take a
look.

[Devlin, 2015-06-11 16:42:03]

lgtm.  +Jennifer (media router action) for FYI.

https://codereview.chromium.org/1177973005/diff/1/chrome/browser/extensions/c...
File chrome/browser/extensions/chrome_mojo_service_registration.cc (right):

https://codereview.chromium.org/1177973005/diff/1/chrome/browser/extensions/c...
chrome/browser/extensions/chrome_mojo_service_registration.cc:31:
content::ServiceRegistry* service_registry =
nit: Since you're here, mind just inlining this whole thing on line 38?

render_frame_host->GetServiceRegistry()->AddService(...)

[mharanczyk, 2015-06-12 08:11:06]

The CQ bit was checked by mharanczyk@opera.com

[mharanczyk, 2015-06-12 08:11:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
rdevlin.cronin@chromium.org
Link to the patchset: https://codereview.chromium.org/1177973005/#ps20001
(title: "Nit fix.")

[commit-bot: I haz the power, 2015-06-12 08:11:14]

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

[commit-bot: I haz the power, 2015-06-12 08:18:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-12 08:18:04]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[mharanczyk, 2015-06-12 08:25:53]

mharanczyk@opera.com changed reviewers:
+ jochen@chromium.org

[mharanczyk, 2015-06-12 08:26:43]

On 2015/06/12 08:18:04, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

Still need whitelist.cc review, sorry.

[jochen (gone - plz use gerrit), 2015-06-12 08:28:59]

rationale: restricting an existing component extension is fine

LGTM

[mharanczyk, 2015-06-12 08:32:31]

The CQ bit was checked by mharanczyk@opera.com

[commit-bot: I haz the power, 2015-06-12 08:32:45]

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

[commit-bot: I haz the power, 2015-06-12 09:11:51]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-12 09:12:36]

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