Simplify AuthenticationMethod type and PIN hash handling.

remoting/test/test_chromoting_client_unittest.cc

 // Copyright 2015 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/macros.h"
 #include "base/message_loop/message_loop.h"
 #include "remoting/protocol/fake_connection_to_host.h"
 #include "remoting/signaling/fake_signal_strategy.h"
@@ skipping 55 matching lines @@
   // keep the ptr around so we can use it to simulate state changes.  It will
   // remain valid until |test_chromoting_client_| is destroyed.
   fake_connection_to_host_ = new FakeConnectionToHost();
   test_chromoting_client_->SetSignalStrategyForTests(make_scoped_ptr(
       new FakeSignalStrategy("test_user@faux_address.com/123")));
   test_chromoting_client_->SetConnectionToHostForTests(
       make_scoped_ptr(fake_connection_to_host_));
 
   connection_setup_info_.host_jid = "test_host@faux_address.com/321";
   connection_setup_info_.auth_methods.push_back(
-      protocol::AuthenticationMethod::ThirdParty());
+      protocol::AuthenticationMethod::THIRD_PARTY);
 }
 
 void TestChromotingClientTest::TearDown() {
   test_chromoting_client_->RemoveRemoteConnectionObserver(this);
   fake_connection_to_host_ = nullptr;
 
   // The chromoting instance must be destroyed before the message loop.
   test_chromoting_client_.reset();
 
   // The IceTransportFactory destroys the PortAllocator via a DeleteSoon
@@ skipping 108 matching lines @@
   // Close the connection via the TestChromotingClient and verify the error
   // state is persisted.
   test_chromoting_client_->EndConnection();
   EXPECT_EQ(protocol::ConnectionToHost::State::FAILED, connection_state_);
   EXPECT_EQ(protocol::ErrorCode::UNKNOWN_ERROR, error_code_);
   EXPECT_FALSE(is_connected_to_host_);
 }
 
 }  // namespace test
 }  // namespace remoting