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

Unified Diff: chrome/browser/component_updater/pepper_flash_component_installer.cc

Issue 10977073: Delete some unused code found by -Wunused-function (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: leiz Created 8 years, 3 months 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/component_updater/pepper_flash_component_installer.cc
diff --git a/chrome/browser/component_updater/pepper_flash_component_installer.cc b/chrome/browser/component_updater/pepper_flash_component_installer.cc
index 05ac5dd2d68cb0c2285ac8bb1b59be32a7f16564..77f678c94a51b87bc4dc46013ec9341fc69017cc 100644
--- a/chrome/browser/component_updater/pepper_flash_component_installer.cc
+++ b/chrome/browser/component_updater/pepper_flash_component_installer.cc
@@ -81,6 +81,7 @@ FilePath GetPepperFlashBaseDirectory() {
return result;
}
+#if defined(GOOGLE_CHROME_BUILD)
// Pepper Flash plugins have the version encoded in the path itself
// so we need to enumerate the directories to find the full path.
// On success, |latest_dir| returns something like:
@@ -115,6 +116,7 @@ bool GetPepperFlashDirectory(FilePath* latest_dir,
}
return found;
}
+#endif
// Returns true if the Pepper |interface_name| is implemented by this browser.
// It does not check if the interface is proxied.
@@ -300,6 +302,7 @@ bool CheckPepperFlashManifest(base::DictionaryValue* manifest,
namespace {
+#if defined(GOOGLE_CHROME_BUILD)
void FinishPepperFlashUpdateRegistration(ComponentUpdateService* cus,
const Version& version) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
@@ -346,6 +349,7 @@ void StartPepperFlashUpdateRegistration(ComponentUpdateService* cus) {
file_util::Delete(*iter, true);
}
}
+#endif // defined(GOOGLE_CHROME_BUILD)
} // namespace
« no previous file with comments | « chrome/browser/chrome_browser_main.cc ('k') | chrome/browser/content_settings/content_settings_pref_provider_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698