[Extensions] Sync the 'allowed scripting on all urls' preference

chrome/browser/extensions/extension_sync_data.h

Index: chrome/browser/extensions/extension_sync_data.h
diff --git a/chrome/browser/extensions/extension_sync_data.h b/chrome/browser/extensions/extension_sync_data.h
index 1e924ddf2c443de6f5ad392af98ff1d62a11e35a..7421fb0c40f571f900546f11ef2b1f5991b09413 100644
--- a/chrome/browser/extensions/extension_sync_data.h
+++ b/chrome/browser/extensions/extension_sync_data.h
@@ -26,13 +26,20 @@ class Extension;
 // A class that encapsulates the synced properties of an Extension.
 class ExtensionSyncData {
  public:
+  enum PrefValue {

[not at google - send to devlin, 2015/02/11 01:34:51]

This "PrefValue" name is freaking me out. Can you call it OptionalBoolean,
literally what it is.

[Devlin, 2015/02/11 17:52:21]

Fair. Done.

+    PREF_UNSET,
+    PREF_TRUE,
+    PREF_FALSE
+  };
+
   ExtensionSyncData();
   explicit ExtensionSyncData(const syncer::SyncData& sync_data);
   explicit ExtensionSyncData(const syncer::SyncChange& sync_change);
   ExtensionSyncData(const Extension& extension,
                     bool enabled,
                     bool incognito_enabled,
-                    bool remote_install);
+                    bool remote_install,
+                    PrefValue all_urls_enabled);
   ~ExtensionSyncData();
 
   // Retrieve sync data from this class.
@@ -58,6 +65,7 @@ class ExtensionSyncData {
   bool enabled() const { return enabled_; }
   bool incognito_enabled() const { return incognito_enabled_; }
   bool remote_install() const { return remote_install_; }
+  PrefValue all_urls_enabled() const { return all_urls_enabled_; }
   bool installed_by_custodian() const { return installed_by_custodian_; }
 
   // Version-dependent properties (i.e., should be used only when the
@@ -76,6 +84,7 @@ class ExtensionSyncData {
   bool enabled_;
   bool incognito_enabled_;
   bool remote_install_;
+  PrefValue all_urls_enabled_;
   bool installed_by_custodian_;
   Version version_;
   GURL update_url_;