Cleanup: Put more media and profile extensions code behind ENABLE_EXTENSIONS ifdefs.

Cleanup: Put more media and profile extensions code behind ENABLE_EXTENSIONS ifdefs.

BUG=424746
Committed: https://crrev.com/2bad4ee392b58240220a56a1a40778faeb67615c
Cr-Commit-Position: refs/heads/master@{#300735}

[Lei Zhang, 2014-10-17 18:55:27]

thestig@chromium.org changed reviewers:
+ vrk@chromium.org

[vrk (LEFT CHROMIUM), 2014-10-17 19:56:44]

Could you add a BUG= link to the CL?

https://codereview.chromium.org/661053002/diff/1/chrome/browser/media/media_c...
File chrome/browser/media/media_capture_devices_dispatcher.cc (right):

https://codereview.chromium.org/661053002/diff/1/chrome/browser/media/media_c...
chrome/browser/media/media_capture_devices_dispatcher.cc:726: bool audio_allowed
= false;
nit: looks like these two "audio_allowed"/ "video_allowed" edits can be
reverted?

[Lei Zhang, 2014-10-17 21:44:51]

I filed a bug.

https://codereview.chromium.org/661053002/diff/1/chrome/browser/media/media_c...
File chrome/browser/media/media_capture_devices_dispatcher.cc (right):

https://codereview.chromium.org/661053002/diff/1/chrome/browser/media/media_c...
chrome/browser/media/media_capture_devices_dispatcher.cc:726: bool audio_allowed
= false;
On 2014/10/17 19:56:44, vrk wrote:
> nit: looks like these two "audio_allowed"/ "video_allowed" edits can be
> reverted?

indeed, I had the ifdef inside the function before.

[vrk (LEFT CHROMIUM), 2014-10-22 14:38:50]

LGTM

d'oh, sorry I forgot about this. Thanks for the cleanup!

[vrk (LEFT CHROMIUM), 2014-10-22 14:38:52]

The CQ bit was checked by vrk@chromium.org

[commit-bot: I haz the power, 2014-10-22 14:41:01]

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

[commit-bot: I haz the power, 2014-10-22 16:48:26]

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

[commit-bot: I haz the power, 2014-10-22 16:48:27]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Lei Zhang, 2014-10-22 18:49:09]

On 2014/10/22 14:38:50, vrk wrote:
> LGTM
> 
> d'oh, sorry I forgot about this. Thanks for the cleanup!

Thanks. No rush.

[Lei Zhang, 2014-10-22 18:49:13]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-10-22 18:50:38]

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

[commit-bot: I haz the power, 2014-10-22 20:15:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-22 20:16:43]

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