Move [Get|Set]UpdateURLData from ExtensionPrefs to ExtensionAPI (aka Module)

chrome/browser/extensions/updater/extension_updater.cc

Index: chrome/browser/extensions/updater/extension_updater.cc
diff --git a/chrome/browser/extensions/updater/extension_updater.cc b/chrome/browser/extensions/updater/extension_updater.cc
index 4db6feb3be44a05167e4d325ed415e02fa310166..7ca8f13220080388ccee7d423ed6309f69336a7b 100644
--- a/chrome/browser/extensions/updater/extension_updater.cc
+++ b/chrome/browser/extensions/updater/extension_updater.cc
@@ -16,6 +16,7 @@
 #include "base/stl_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
+#include "chrome/browser/extensions/api/module/module.h"
 #include "chrome/browser/extensions/blacklist.h"
 #include "chrome/browser/extensions/crx_installer.h"
 #include "chrome/browser/extensions/extension_service.h"
@@ -544,7 +545,9 @@ bool ExtensionUpdater::GetPingDataForExtension(
 
 std::string ExtensionUpdater::GetUpdateUrlData(const std::string& id) {
   DCHECK(alive_);
-  return extension_prefs_->GetUpdateUrlData(id);
+  std::string data;
+  extension_prefs_->ReadPrefAsString(id, extension::kUpdateURLData, &data);

[Yoyo Zhou, 2013/05/15 18:49:49]

I think it'd be better to have an accessor in module.h than to expose the pref
name here. What do you think?

[Devlin, 2013/05/15 23:12:29]

Agreed, and done.

+  return data;
 }
 
 bool ExtensionUpdater::IsExtensionPending(const std::string& id) {