Move Feature and Manifest to top-level extensions.

Move Feature and Manifest to top-level extensions.

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

[Yoyo Zhou, 2013-08-16 01:18:27]

mpcomplete: please review.
garykac: please review remoting/test/DEPS.
jam: please review chrome/utility/DEPS.

I believe these DEPS changes to be ok because they're already transitively
allowed, via chrome/common.

[Matt Perry, 2013-08-16 01:26:23]

lgtm

[garykac, 2013-08-19 18:03:22]

remoting/tests LGTM

[Yoyo Zhou, 2013-08-20 19:05:47]

Changed jam to ben since jam is OOO. ben, can you review chrome/utility/DEPS?

[Yoyo Zhou, 2013-08-21 02:01:16]

Ok, ben is OOO too... How about sky? Scott, can you review chrome/utility/DEPS?

[sky, 2013-08-21 14:17:22]

This is the first I've seen of chrome/utility, so I'm not sure what the
expectations are for its DEPS. How about adding an OWNERS file to chrome/utility
and getting that person to review this change?

[Yoyo Zhou, 2013-08-21 15:57:05]

+thestig
I'm not sure who should go in a chrome/utility/OWNERS file either.

[Lei Zhang, 2013-08-21 19:52:49]

On 2013/08/21 15:57:05, Yoyo Zhou wrote:
> +thestig
> I'm not sure who should go in a chrome/utility/OWNERS file either.

All the chrome/utility/* sub-folders have OWNERS files. Doesn't chrome/utility
itself, which only has a few files, just fall back to the top level chrome/
OWNERS? I"m fine with that.

chrome/utility LGTM

[commit-bot: I haz the power, 2013-08-21 23:39:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/23257005/5001

[commit-bot: I haz the power, 2013-08-21 23:41:11]

Failed to apply patch for remoting/test/remote_desktop_browsertest.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file remoting/test/remote_desktop_browsertest.cc
  Hunk #1 FAILED at 14.
  Hunk #2 FAILED at 25.
  2 out of 2 hunks FAILED -- saving rejects to file
remoting/test/remote_desktop_browsertest.cc.rej

Patch:       remoting/test/remote_desktop_browsertest.cc
Index: remoting/test/remote_desktop_browsertest.cc
diff --git a/remoting/test/remote_desktop_browsertest.cc
b/remoting/test/remote_desktop_browsertest.cc
index
288fe664947865b55ca7e75dda242bd3b5a1b0e6..5156f55e4cd61f2844c50d64d2db5b3886a2492b
100644
--- a/remoting/test/remote_desktop_browsertest.cc
+++ b/remoting/test/remote_desktop_browsertest.cc
@@ -14,7 +14,6 @@
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_file_util.h"
-#include "chrome/common/extensions/manifest.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
@@ -25,6 +24,7 @@
 #include "content/public/browser/web_contents.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/test_utils.h"
+#include "extensions/common/manifest.h"
 #include "net/dns/mock_host_resolver.h"
 
 using extensions::Extension;

[commit-bot: I haz the power, 2013-08-22 00:32:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/23257005/18001

[commit-bot: I haz the power, 2013-08-22 01:08:03]

Retried try job too often on linux_chromeos_clang for step(s) compile
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2013-08-22 16:47:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yoz@chromium.org/23257005/36001

[commit-bot: I haz the power, 2013-08-22 19:36:38]

Change committed as 219075