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/logging.h"
 #include "base/message_loop_proxy.h"
 // TODO(wez): Remove this when crbug.com/86353 is complete.
 #include "ppapi/cpp/private/var_private.h"
@@ skipping 403 matching lines @@
   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());
+  VLOG(1) << "Ignoring setScaleToFit";

[Sergey Ulanov, 2011/12/20 00:05:35]

Do we need to keep this method just for backward compatibility? If so, please
add comments about it.

[Wez, 2011/12/20 07:14:14]

I've replaced this method with DoNothing().  I *think* we need to have an
implementation for setScaleToFit(), otherwise JavaScript will throw an exception
because the method is missing.

   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