Rename PluginService to PluginServiceImpl.

content/browser/plugin_service_impl_browsertest.cc

 // Copyright (c) 2011 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 "content/browser/plugin_service.h"
-
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/path_service.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/testing_profile.h"
 #include "chrome/test/base/ui_test_utils.h"
+#include "content/browser/plugin_service_impl.h"

[Nico, 2011/12/21 03:38:39]

Nit: the style guide says that the header for a test file should be at the top:
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Names_...
I guess that applies to _browsertest files as well.

[jam, 2011/12/21 04:56:50]

oh, that's new to me. I just checked and it got updated a month ago. i don't
know why this was done, since it seems redundant to enforce this for the test
file. but i'll revert this change then.

 #include "content/browser/resource_context.h"
 #include "content/public/common/content_switches.h"
 #include "content/test/test_browser_thread.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "webkit/plugins/npapi/plugin_list.h"
 
 using content::BrowserThread;
 
 namespace {
 
@@ skipping 278 matching lines @@
       BrowserThread::IO, FROM_HERE,
       base::Bind(&OpenChannel, &mock_client));
   ui_test_utils::RunMessageLoop();
   EXPECT_TRUE(mock_client.get_resource_context_called());
   EXPECT_TRUE(mock_client.set_plugin_info_called());
   EXPECT_TRUE(mock_client.on_found_plugin_process_host_called());
   EXPECT_TRUE(mock_client.on_sent_plugin_channel_request_called());
 }
 
 }  // namespace