Introduces an additional extension loader that load extra extensions based on per-extension json fi…

chrome/browser/extensions/external_extension_util.h

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_EXTERNAL_EXTENSION_UTIL_H__
+#define CHROME_BROWSER_EXTENSIONS_EXTERNAL_EXTENSION_UTIL_H__
+#pragma once
+
+namespace base {
+class DictionaryValue;
+class ListValue;

[Sam Kerner (Chrome), 2012/04/13 19:35:13]

I think you don't use DictionaryValue or ListValue.  If so, please remove.

[Alexandre Abreu, 2012/04/13 21:52:38]

Done.

+class FilePath;
+class ValueSerializer;
+}
+class FilePath;

[Sam Kerner (Chrome), 2012/04/13 19:35:13]

File path is declared both inside namespace base and outside of any namespace. 
Looking at other files (grep 'class FilePath'
~/cr/gitMac/src/chrome/browser/extensions/ ), I think the convention is to use
FilePath over base::FilePath.

[Alexandre Abreu, 2012/04/13 21:52:38]

yes, from what I see

+
+// Provides various utility functions for external extension manipulation.
+class ExternalExtensionUtil {
+ public:
+  static base::DictionaryValue* ExtractPrefs(const FilePath& path
+                                         , base::ValueSerializer* serializer);

[Sam Kerner (Chrome), 2012/04/13 19:35:13]

Style nit: comma belongs on the previous line, and indentation should make
"const" and "base::" start in the same column.

[Alexandre Abreu, 2012/04/13 21:52:38]

Done.

+};
+
+#endif  // CHROME_BROWSER_EXTENSIONS_EXTERNAL_EXTENSION_UTIL_H__
+