Cleanup pass from Cloud Print Proxy policy implementation.

chrome/browser/printing/cloud_print/cloud_print_proxy_service_unittest.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 <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/printing/cloud_print/cloud_print_proxy_service.h"
 #include "chrome/browser/service/service_process_control.h"
 #include "chrome/common/cloud_print/cloud_print_proxy_info.h"
@@ skipping 16 matching lines @@
 using ::testing::ReturnPointee;
 using ::testing::WithArgs;
 using ::testing::_;
 
 class MockServiceProcessControl : public ServiceProcessControl {
  public:
   static std::string EnabledUserId();
 
   MockServiceProcessControl() : connected_(false) { }
 
-  MOCK_CONST_METHOD0(is_connected, bool());
+  MOCK_CONST_METHOD0(IsConnected, bool());
 
   MOCK_METHOD2(Launch, void(const base::Closure&, const base::Closure&));
   MOCK_METHOD0(Disconnect, void());
 
   MOCK_METHOD1(OnMessageReceived, bool(const IPC::Message&));
   MOCK_METHOD1(OnChannelConnected, void(int32 peer_pid));
   MOCK_METHOD0(OnChannelError, void());
 
   MOCK_METHOD1(Send, bool(IPC::Message*));
 
@@ skipping 23 matching lines @@
   return std::string("dorothy@somewhere.otr");
 }
 
 void CallTask(const base::Closure& task) {
   if (!task.is_null())
     task.Run();
 }
 
 void MockServiceProcessControl::SetConnectSuccessMockExpectations(
     ServiceState service_state) {
-  EXPECT_CALL(*this, is_connected()).WillRepeatedly(ReturnPointee(&connected_));
+  EXPECT_CALL(*this, IsConnected()).WillRepeatedly(ReturnPointee(&connected_));
 
   EXPECT_CALL(*this, Launch(_, _))
       .WillRepeatedly(
           DoAll(Assign(&connected_, true), WithArgs<0>(Invoke(CallTask))));
   EXPECT_CALL(*this, Disconnect()).Times(AtMost(1))
       .WillRepeatedly(Assign(&connected_, false));
 
   EXPECT_CALL(*this, Send(_)).Times(0);
 
   if (service_state == kServiceStateEnabled)
@@ skipping 82 matching lines @@
   MessageLoopForUI message_loop_;
   content::TestBrowserThread ui_thread_;
   TestingProfile profile_;
 };
 
 TEST_F(CloudPrintProxyPolicyTest, VerifyExpectations) {
   MockServiceProcessControl mock_control;
   mock_control.SetConnectSuccessMockExpectations(
       MockServiceProcessControl::kServiceStateNone);
 
-  EXPECT_FALSE(mock_control.is_connected());
+  EXPECT_FALSE(mock_control.IsConnected());
   mock_control.Launch(base::Closure(), base::Closure());
-  EXPECT_TRUE(mock_control.is_connected());
+  EXPECT_TRUE(mock_control.IsConnected());
   mock_control.Launch(base::Closure(), base::Closure());
-  EXPECT_TRUE(mock_control.is_connected());
+  EXPECT_TRUE(mock_control.IsConnected());
   mock_control.Disconnect();
-  EXPECT_FALSE(mock_control.is_connected());
+  EXPECT_FALSE(mock_control.IsConnected());
 }
 
 TEST_F(CloudPrintProxyPolicyTest, StartWithNoPolicyProxyDisabled) {
   TestCloudPrintProxyService service(&profile_);
 
   service.GetMockServiceProcessControl()->SetConnectSuccessMockExpectations(
       MockServiceProcessControl::kServiceStateDisabled);
 
   TestingPrefService* prefs = profile_.GetTestingPrefService();
   prefs->SetUserPref(prefs::kCloudPrintEmail,
@@ skipping 188 matching lines @@
   prefs->RemoveManagedPref(prefs::kCloudPrintProxyEnabled);
   EXPECT_EQ(std::string(), prefs->GetString(prefs::kCloudPrintEmail));
 
   service.GetMockServiceProcessControl()->SetWillBeEnabledExpectations();
   service.EnableForUser(std::string(),
                         MockServiceProcessControl::EnabledUserId());
 
   EXPECT_EQ(MockServiceProcessControl::EnabledUserId(),
             prefs->GetString(prefs::kCloudPrintEmail));
 }