Allow whitelisted prefs to be displayed in ChromeOS in chrome://local-state.

chrome/browser/ui/webui/local_state/local_state_ui.h

Index: chrome/browser/ui/webui/local_state/local_state_ui.h
diff --git a/chrome/browser/ui/webui/local_state/local_state_ui.h b/chrome/browser/ui/webui/local_state/local_state_ui.h
index e845dae0958c9da4541f153760800e440588fe00..5ae6fdb5c3a3b3b5e5232a7544056d8aadb56a97 100644
--- a/chrome/browser/ui/webui/local_state/local_state_ui.h
+++ b/chrome/browser/ui/webui/local_state/local_state_ui.h
@@ -5,9 +5,26 @@
 #ifndef CHROME_BROWSER_UI_WEBUI_LOCAL_STATE_LOCAL_STATE_UI_H_
 #define CHROME_BROWSER_UI_WEBUI_LOCAL_STATE_LOCAL_STATE_UI_H_
 
+#include <string>
+#include <vector>
+
 #include "base/macros.h"
 #include "content/public/browser/web_ui_controller.h"
 
+namespace base {
+
+class DictionaryValue;
+
+}  // namespace base

[Alexei Svitkine (slow), 2016/04/26 15:33:35]

Nit: No need for this end comment for a forward decl. However, please add one
for the internal namespace where you declare a function below.

Same comment about the empty lines around namespace - no need for the fwd decl,
but add them below.

[hamelphi1, 2016/04/26 17:42:40]

Done.

+
+// Namespace for exposing the method for unittests.

[Alexei Svitkine (slow), 2016/04/26 15:33:35]

Nit: unittests -> unit tests

[hamelphi1, 2016/04/26 17:42:40]

Done.

+namespace internal {
+// Removes elements from |prefs| where the key does not match any of the
+// prefixes in |valid_prefixes|.
+void FilterPrefs(const std::vector<std::string>& valid_prefixes,
+                 base::DictionaryValue* prefs);
+}
+
 // Controller for chrome://local-state/ page.
 class LocalStateUI : public content::WebUIController {
  public: