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

Unified Diff: chrome/common/pepper_flash.cc

Issue 2815923004: Remove references to WinFlapper. (Closed)
Patch Set: Created 3 years, 8 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/pepper_flash.cc
diff --git a/chrome/common/pepper_flash.cc b/chrome/common/pepper_flash.cc
index 0eb72dd49c1f25547f2a7a399c58ac2cab481f62..86d9c6f591ba0ce18fe634c8d241df6ee0a52cb2 100644
--- a/chrome/common/pepper_flash.cc
+++ b/chrome/common/pepper_flash.cc
@@ -102,10 +102,7 @@ bool CheckPepperFlashManifest(const base::DictionaryValue& manifest,
base::Version* version_out) {
std::string name;
manifest.GetStringASCII("name", &name);
- // TODO(viettrungluu): Support WinFlapper for now, while we change the format
- // of the manifest. (Should be safe to remove checks for "WinFlapper" in, say,
- // Nov. 2011.) crbug.com/98458
- if (name != kPepperFlashManifestName && name != "WinFlapper")
+ if (name != kPepperFlashManifestName)
return false;
std::string proposed_version;
@@ -117,12 +114,6 @@ bool CheckPepperFlashManifest(const base::DictionaryValue& manifest,
if (!CheckPepperFlashInterfaces(manifest))
return false;
- // TODO(viettrungluu): See above TODO.
- if (name == "WinFlapper") {
- *version_out = version;
- return true;
- }
-
std::string os;
manifest.GetStringASCII("x-ppapi-os", &os);
if (os != kPepperFlashOperatingSystem)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698