Report video and network stats averaged over 1s, and create corresponding UMA metrics.

remoting/client/plugin/chromoting_instance.h

 // Copyright (c) 2012 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.
 
 #ifndef REMOTING_CLIENT_PLUGIN_CHROMOTING_INSTANCE_H_
 #define REMOTING_CLIENT_PLUGIN_CHROMOTING_INSTANCE_H_
 
 #include <string>
 
 #include "base/gtest_prod_util.h"
@@ skipping 201 matching lines @@
   // remove this method and use PostChromotingMessage() instead.
   void PostLegacyJsonMessage(const std::string& method,
                              scoped_ptr<base::DictionaryValue> data);
 
   // Posts trapped keys to the web-app to handle.
   void SendTrappedKey(uint32 usb_keycode, bool pressed);
 
   // Callback for DelegatingSignalStrategy.
   void SendOutgoingIq(const std::string& iq);
 
-  void SendPerfStats();
+  void SendDisplayPerfStats();
+
+  void SendUMAPerfStats();

[Wez, 2015/07/08 23:42:29]

nit: SendUmaPerfStats() as per style-guide naming conventions.

Also suggest a single-line comment to introduce these two methods e.g. "Helpers
to send stats to the web-app for display and UMA logging."

[anandc, 2015/07/09 19:17:10]

Reverted to a single function now.

 
   void ProcessLogToUI(const std::string& message);
 
   // Returns true if the hosting content has the chrome-extension:// scheme.
   bool IsCallerAppOrExtension();
 
   // Returns true if there is a ConnectionToHost and it is connected.
   bool IsConnected();
 
   // Used as the |FetchSecretCallback| for Me2Me connections.
@@ skipping 48 matching lines @@
 
   // Weak reference to this instance, used for global logging and task posting.
   base::WeakPtrFactory<ChromotingInstance> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingInstance);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_PLUGIN_CHROMOTING_INSTANCE_H_