OLD | NEW |
---|---|
(Empty) | |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef CHROME_BROWSER_EXTENSIONS_EXTERNAL_EXTENSION_UTIL_H_ | |
6 #define CHROME_BROWSER_EXTENSIONS_EXTERNAL_EXTENSION_UTIL_H_ | |
7 #pragma once | |
8 | |
9 namespace base { | |
10 class DictionaryValue; | |
11 class ValueSerializer; | |
12 } | |
13 class FilePath; | |
14 | |
15 // Provides various utility functions for external extension manipulation. | |
16 class ExternalExtensionUtil { | |
Finnur
2012/04/25 13:33:45
I still don't see a reason why this is a class and
Alexandre Abreu
2012/04/25 15:31:05
Agree, as in many other places, I tried to follow
| |
17 public: | |
18 // Extracts/expect a file content in json format. | |
19 // Used by the external extension facilities/providers | |
20 // to parse their extension manifests designated by |path|. | |
21 // An empty dictionary is returned in case of failure (e.g. invalid | |
22 // path or json content). | |
23 // Caller takes ownership of the returned dictionary. | |
24 static base::DictionaryValue* ExtractExtensionPrefs(const FilePath& path, | |
25 base::ValueSerializer* serializer); | |
Finnur
2012/04/25 13:33:45
Can you add WARN_UNUSED_RESULT to the end (before
Alexandre Abreu
2012/04/25 15:31:05
Sorry missed that.
Done.
| |
26 }; | |
27 | |
28 #endif // CHROME_BROWSER_EXTENSIONS_EXTERNAL_EXTENSION_UTIL_H_ | |
OLD | NEW |