Add a preference to clear Flash LSO data in the Clear Browsing Data dialog.

chrome/browser/plugin_data_remover.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/plugin_data_remover.h"
 
 #include "base/message_loop_proxy.h"
+#include "base/version.h"
 #include "chrome/browser/browser_thread.h"
 #include "chrome/browser/plugin_service.h"
 #include "chrome/common/plugin_messages.h"
 #include "ipc/ipc_channel.h"
+#include "webkit/glue/plugins/plugin_group.h"
+#include "webkit/glue/plugins/plugin_list.h"
 
 #if defined(OS_POSIX)
 #include "ipc/ipc_channel_posix.h"
 #endif
 
 namespace {
 const std::string flash_mime_type = "application/x-shockwave-flash";
+const std::string min_flash_version = "10.2";

[jam, 2010/11/29 17:14:08]

these globals don't seem to match our global naming convention, i.e. they look
like local variables.  I suggest prefixing them with "g_".

we also avoid having any globals that use types with constructors/destructors. 
the reasons are that it slows down startup/shutdown slightly (see davemoore's
crusade to remove them for chrome OS perf reasons, and also why we have
base::LazyInstance), and also there's a limit to how many objects the CRT can
keep track of. can you make the strings just be const char*?  

 const int64 timeout_ms = 10000;
 }
 
 PluginDataRemover::PluginDataRemover()
     : channel_(NULL),
       method_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
 }
 
 PluginDataRemover::~PluginDataRemover() {
   DCHECK(!done_task_.get());
   if (!BrowserThread::DeleteSoon(BrowserThread::IO, FROM_HERE, channel_))
     delete channel_;
 }
 
 void PluginDataRemover::StartRemoving(base::Time begin_time, Task* done_task) {
   DCHECK(!done_task_.get());
   begin_time_ = begin_time;
 
   message_loop_ = base::MessageLoopProxy::CreateForCurrentThread();
   done_task_.reset(done_task);
 
   PluginService::GetInstance()->OpenChannelToPlugin(
       GURL(), flash_mime_type, this);
 
-  BrowserThread::PostDelayedTask(
-      BrowserThread::IO,
+  MessageLoop::current()->PostDelayedTask(
       FROM_HERE,
       method_factory_.NewRunnableMethod(&PluginDataRemover::OnTimeout),
       timeout_ms);
 }
 
 int PluginDataRemover::ID() {
   // Generate an ID for the browser process.
   return ChildProcessInfo::GenerateChildProcessUniqueId();
 }
 
@@ skipping 48 matching lines @@
     IPC_MESSAGE_UNHANDLED_ERROR()
   IPC_END_MESSAGE_MAP()
 }
 
 void PluginDataRemover::OnChannelError() {
   NOTREACHED() << "Channel error";
   SignalDone();
 }
 
 void PluginDataRemover::SignalDone() {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+  // This method could be called from the timeout callback while in another
+  // callback, so we keep a lock while releasing |done_task_|.
+  AutoLock lock(task_lock_);
   if (!done_task_.get())
     return;
   message_loop_->PostTask(FROM_HERE, done_task_.release());
   message_loop_ = NULL;
 }
+
+// static
+bool PluginDataRemover::IsSupported() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
+  bool allow_wildcard = false;
+  WebPluginInfo plugin;
+  std::string mime_type;
+  if (!NPAPI::PluginList::Singleton()->GetPluginInfo(GURL(),
+                                                     flash_mime_type,
+                                                     allow_wildcard,
+                                                     &plugin,
+                                                     &mime_type))
+    return false;
+  scoped_ptr<Version> version(
+      PluginGroup::CreateVersionFromString(plugin.version));
+  scoped_ptr<Version> min_version(
+      Version::GetVersionFromString(min_flash_version));
+  return plugin.enabled &&
+         version.get()
+         && min_version->CompareTo(*version) == -1;
+}