Move NaClModulesHandler to src/extensions

Move NaClModulesHandler to src/extensions

This is a first, very small step towards making NaCl work in app_shell.

BUG=400577
TEST=compiles
TBR=brettw@chromium.org for moving a few lines out of chrome/common/BUILD.gn into extensions/common/BUILD.gn

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288136

[James Cook, 2014-08-04 23:34:00]

rockot, PTAL

[Ken Rockot(use gerrit already), 2014-08-04 23:41:57]

lgtm

https://codereview.chromium.org/435343002/diff/1/chrome/common/extensions/chr...
File chrome/common/extensions/chrome_manifest_handlers.cc (right):

https://codereview.chromium.org/435343002/diff/1/chrome/common/extensions/chr...
chrome/common/extensions/chrome_manifest_handlers.cc:65: (new
NaClModulesHandler)->Register();
Any harm in moving this to //extensions/common/common_manifest_handlers.cc?

[James Cook, 2014-08-05 02:02:20]

https://codereview.chromium.org/435343002/diff/1/chrome/common/extensions/chr...
File chrome/common/extensions/chrome_manifest_handlers.cc (right):

https://codereview.chromium.org/435343002/diff/1/chrome/common/extensions/chr...
chrome/common/extensions/chrome_manifest_handlers.cc:65: (new
NaClModulesHandler)->Register();
On 2014/08/04 23:41:56, Ken Rockot wrote:
> Any harm in moving this to //extensions/common/common_manifest_handlers.cc?

Good catch. Done.

[James Cook, 2014-08-05 02:02:58]

The CQ bit was checked by jamescook@chromium.org

[commit-bot: I haz the power, 2014-08-05 02:04:27]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jamescook@chromium.org/435343002/20001

[commit-bot: I haz the power, 2014-08-05 06:19:47]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)
  mac_chromium_rel on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel/...)

[commit-bot: I haz the power, 2014-08-05 06:44:05]

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

[commit-bot: I haz the power, 2014-08-05 06:44:06]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)

[James Cook, 2014-08-06 17:30:42]

The CQ bit was checked by jamescook@chromium.org

[commit-bot: I haz the power, 2014-08-06 17:32:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jamescook@chromium.org/435343002/20001

[commit-bot: I haz the power, 2014-08-06 17:35:33]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)

[commit-bot: I haz the power, 2014-08-06 17:38:33]

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

[commit-bot: I haz the power, 2014-08-06 17:38:34]

Try jobs failed on following builders:
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)

[James Cook, 2014-08-06 21:11:34]

The CQ bit was checked by jamescook@chromium.org

[commit-bot: I haz the power, 2014-08-06 21:13:32]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jamescook@chromium.org/435343002/40001

[commit-bot: I haz the power, 2014-08-07 01:44:28]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/43482)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-07 02:09:47]

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

[commit-bot: I haz the power, 2014-08-07 02:09:50]

Try jobs failed on following builders:
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)

[James Cook, 2014-08-07 16:44:16]

The CQ bit was checked by jamescook@chromium.org

[commit-bot: I haz the power, 2014-08-07 16:46:48]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jamescook@chromium.org/435343002/60001

[commit-bot: I haz the power, 2014-08-07 20:40:14]

Change committed as 288136