Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4745)

Unified Diff: chrome/browser/extensions/api/runtime/runtime_api.cc

Issue 67923002: Fixes for -Wunused-function on Linux, Android and ChromeOS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/runtime/runtime_api.cc
diff --git a/chrome/browser/extensions/api/runtime/runtime_api.cc b/chrome/browser/extensions/api/runtime/runtime_api.cc
index 1e478f8f7395ebd036fb2e47cab6278664316f57..0795e37b8eb2b259c804689df20baad58255566d 100644
--- a/chrome/browser/extensions/api/runtime/runtime_api.cc
+++ b/chrome/browser/extensions/api/runtime/runtime_api.cc
@@ -121,12 +121,14 @@ void SetUninstallUrl(ExtensionPrefs* prefs,
new base::StringValue(url_string));
}
+#if defined(ENABLE_EXTENSIONS)
std::string GetUninstallUrl(ExtensionPrefs* prefs,
const std::string& extension_id) {
std::string url_string;
prefs->ReadPrefAsString(extension_id, kUninstallUrl, &url_string);
return url_string;
}
+#endif // if defined(ENABLE_EXTENSIONS)
Nico 2013/11/10 05:07:09 No "if", just " // defined(ENABLE_EXTENSIONS)"
hans 2013/11/11 19:29:51 Done. Seems my vim was getting sloppy :)
} // namespace

Powered by Google App Engine
This is Rietveld 408576698