Remove old Authentication code that we don't use or need.

remoting/client/plugin/pepper_view.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 "remoting/client/plugin/pepper_view.h"
 
 #include "base/message_loop.h"
 #include "base/string_util.h"
 #include "ppapi/cpp/graphics_2d.h"
 #include "ppapi/cpp/image_data.h"
@@ skipping 218 matching lines @@
   // TODO(hclam): Re-consider the way we communicate with Javascript.
   ChromotingScriptableObject* scriptable_obj = instance_->GetScriptableObject();
   switch (state) {
     case protocol::ConnectionToHost::CONNECTING:
       SetSolidFill(kCreatedColor);
       scriptable_obj->SetConnectionStatus(
           ChromotingScriptableObject::STATUS_CONNECTING,
           ConvertConnectionError(error));
       break;
 
     case protocol::ConnectionToHost::CONNECTED:

[Wez, 2011/11/03 00:25:02]

Is there any point in keeping this case?

[Sergey Ulanov, 2011/11/03 01:21:12]

Yes, to avoid compiler warnings, otherwise I would have to add default: case..

-      scriptable_obj->SignalLoginChallenge();
       break;
 
     case protocol::ConnectionToHost::AUTHENTICATED:
       UnsetSolidFill();
       break;
 
     case protocol::ConnectionToHost::CLOSED:
       SetSolidFill(kDisconnectedColor);
       scriptable_obj->SetConnectionStatus(
           ChromotingScriptableObject::STATUS_CLOSED,
           ConvertConnectionError(error));
       break;
 
     case protocol::ConnectionToHost::FAILED:
       SetSolidFill(kFailedColor);
       scriptable_obj->SetConnectionStatus(
           ChromotingScriptableObject::STATUS_FAILED,
           ConvertConnectionError(error));
       break;
   }
 }
 
-void PepperView::UpdateLoginStatus(bool success, const std::string& info) {
-  DCHECK(context_->main_message_loop()->BelongsToCurrentThread());
-
-  // TODO(hclam): Re-consider the way we communicate with Javascript.
-  ChromotingScriptableObject* scriptable_obj = instance_->GetScriptableObject();
-  if (success) {
-    scriptable_obj->SetConnectionStatus(
-        ChromotingScriptableObject::STATUS_CONNECTED,
-        ChromotingScriptableObject::ERROR_NONE);
-  } else {
-    scriptable_obj->SignalLoginChallenge();
-  }
-}
-
 bool PepperView::SetPluginSize(const SkISize& plugin_size) {
   if (plugin_size_ == plugin_size)
     return false;
   plugin_size_ = plugin_size;
 
   pp::Size pp_size = pp::Size(plugin_size.width(), plugin_size.height());
 
   graphics2d_ = pp::Graphics2D(instance_, pp_size, true);
   if (!instance_->BindGraphics(graphics2d_)) {
     LOG(ERROR) << "Couldn't bind the device context.";
@@ skipping 76 matching lines @@
       (base::Time::Now() - paint_start).InMilliseconds());
 
   // If the last flush failed because there was already another one in progress
   // then we perform the flush now.
   if (flush_blocked_)
     FlushGraphics(base::Time::Now());
   return;
 }
 
 }  // namespace remoting