Refactoring of the client-side input pipeline and scaling dimension management.

remoting/client/plugin/chromoting_scriptable_object.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/chromoting_scriptable_object.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/message_loop_proxy.h"
 // TODO(wez): Remove this when crbug.com/86353 is complete.
@@ skipping 85 matching lines @@
   AddAttribute(kVideoBandwidthAttribute, Var());
   AddAttribute(kVideoFrameRateAttribute, Var());
   AddAttribute(kVideoCaptureLatencyAttribute, Var());
   AddAttribute(kVideoEncodeLatencyAttribute, Var());
   AddAttribute(kVideoDecodeLatencyAttribute, Var());
   AddAttribute(kVideoRenderLatencyAttribute, Var());
   AddAttribute(kRoundTripLatencyAttribute, Var());
 
   AddMethod("connect", &ChromotingScriptableObject::DoConnect);
   AddMethod("disconnect", &ChromotingScriptableObject::DoDisconnect);
-  AddMethod("setScaleToFit", &ChromotingScriptableObject::DoSetScaleToFit);
   AddMethod("onIq", &ChromotingScriptableObject::DoOnIq);
   AddMethod("releaseAllKeys", &ChromotingScriptableObject::DoReleaseAllKeys);
+
+  // Older versions of the web app expect a setScaleToFit method.
+  AddMethod("setScaleToFit", &ChromotingScriptableObject::DoNothing);
 }
 
 bool ChromotingScriptableObject::HasProperty(const Var& name, Var* exception) {
   // TODO(ajwong): Check if all these name.is_string() sentinels are required.
   if (!name.is_string()) {
     *exception = Var("HasProperty expects a string for the name.");
     return false;
   }
 
   PropertyNameMap::const_iterator iter = property_names_.find(name.AsString());
@@ skipping 284 matching lines @@
   return Var();
 }
 
 Var ChromotingScriptableObject::DoDisconnect(const std::vector<Var>& args,
                                              Var* exception) {
   VLOG(1) << "Disconnecting from host.";
   instance_->Disconnect();
   return Var();
 }
 
-Var ChromotingScriptableObject::DoSetScaleToFit(const std::vector<Var>& args,
-                                                Var* exception) {
-  if (args.size() != 1) {
-    *exception = Var("Usage: setScaleToFit(scale_to_fit)");
-    return Var();
-  }
-
-  if (!args[0].is_bool()) {
-    *exception = Var("scale_to_fit must be a boolean.");
-    return Var();
-  }
-
-  VLOG(1) << "Setting scale-to-fit.";
-  instance_->SetScaleToFit(args[0].AsBool());
+Var ChromotingScriptableObject::DoNothing(const std::vector<Var>& args,
+                                          Var* exception) {
   return Var();
 }
 
 Var ChromotingScriptableObject::DoOnIq(const std::vector<Var>& args,
                                        Var* exception) {
   if (args.size() != 1) {
     *exception = Var("Usage: onIq(response_xml)");
     return Var();
   }
 
@@ skipping 11 matching lines @@
     const std::vector<pp::Var>& args, pp::Var* exception) {
   if (args.size() != 0) {
     *exception = Var("Usage: DoReleaseAllKeys()");
     return Var();
   }
   instance_->ReleaseAllKeys();
   return Var();
 }
 
 }  // namespace remoting