Policy: Re-enabled plugin still disabled

chrome/browser/chromeos/gview_request_interceptor_unittest.cc

 // Copyright (c) 2009 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 <string>
 #include "base/message_loop.h"
 #include "chrome/browser/chromeos/gview_request_interceptor.h"
 #include "chrome/browser/plugin_service.h"
 #include "chrome/common/chrome_paths.h"
 #include "net/base/load_flags.h"
@@ skipping 127 matching lines @@
   EXPECT_EQ(0, test_delegate_.received_redirect_count());
   EXPECT_EQ(GURL("http://foo.com/file.pdf"), request.url());
 }
 
 TEST_F(GViewRequestInterceptorTest, InterceptPdfWhenDisabled) {
   bool enabled;
   SetPDFPluginLoadedState(true, &enabled);
 
   if (enabled) {
     bool pdf_plugin_disabled =
-        NPAPI::PluginList::Singleton()->DisablePlugin(pdf_path_);
+        NPAPI::PluginList::Singleton()->DisablePlugin(pdf_path_, true);

[danno, 2010/12/14 09:43:39]

Why is this true? It seems to be unrelated to policy.

[pastarmovj, 2010/12/15 14:44:51]

Done. It doesn't matter in that case really but yes maybe it's better to be
false.

     EXPECT_TRUE(pdf_plugin_disabled);
   }
 
   net::URLRequest request(GURL("http://foo.com/file.pdf"), &test_delegate_);
   request.Start();
   MessageLoop::current()->Run();
   EXPECT_EQ(1, test_delegate_.received_redirect_count());
   EXPECT_EQ(
       GURL("http://docs.google.com/gview?url=http%3A//foo.com/file.pdf"),
       request.url());
@@ skipping 14 matching lines @@
 TEST_F(GViewRequestInterceptorTest, InterceptPowerpoint) {
   net::URLRequest request(GURL("http://foo.com/file.ppt"), &test_delegate_);
   request.Start();
   MessageLoop::current()->Run();
   EXPECT_EQ(1, test_delegate_.received_redirect_count());
   EXPECT_EQ(GURL("http://docs.google.com/gview?url=http%3A//foo.com/file.ppt"),
                  request.url());
 }
 
 }  // namespace chromeos