Connecting webrtc-internals WebUI frontend with the backend

chrome/browser/media/chrome_webrtc_internals.h

 // Copyright (c) 2013 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 CONTENT_BROWSER_MEDIA_WEBRTC_INTERNALS_H_
-#define CONTENT_BROWSER_MEDIA_WEBRTC_INTERNALS_H_
+#ifndef CHROME_BROWSER_MEDIA_CHROME_WEBRTC_INTERNALS_H_
+#define CHROME_BROWSER_MEDIA_CHROME_WEBRTC_INTERNALS_H_
 
 #include "base/memory/singleton.h"
 #include "base/observer_list.h"
 #include "base/process.h"
 #include "base/values.h"
-#include "content/common/content_export.h"
-
-struct PeerConnectionInfo;
-
-namespace content {
+#include "content/public/browser/webrtc_internals.h"
 
 class WebRTCInternalsUIObserver;
 
 // This is a singleton class running in the browser process.
 // It collects peer connection infomation from the renderers,
 // forwards the data to WebRTCInternalsUIObserver and
 // sends data collecting commands to the renderers.
-class CONTENT_EXPORT WebRTCInternals {
+class ChromeWebRTCInternals : public content::WebRTCInternals {
  public:
-  static WebRTCInternals* GetInstance();
+  static ChromeWebRTCInternals* GetInstance();
 
   // Methods called when peer connection status changes.

[jam, 2013/01/18 22:23:45]

nit: don't document overridden methods. just say

// content::WebRTCInternals implementation:

[jiayl, 2013/01/18 22:39:51]

Done.

-  void AddPeerConnection(base::ProcessId pid, const PeerConnectionInfo& info);
-  void RemovePeerConnection(base::ProcessId pid, int lid);
+  virtual void AddPeerConnection(base::ProcessId pid,
+                                 int lid,
+                                 const std::string& url,
+                                 const std::string& servers,
+                                 const std::string& constraints) OVERRIDE;
+  virtual void RemovePeerConnection(base::ProcessId pid, int lid) OVERRIDE;
 
   // Methods for adding or removing WebRTCInternalsUIObserver.
   void AddObserver(WebRTCInternalsUIObserver *observer);
   void RemoveObserver(WebRTCInternalsUIObserver *observer);
 
  private:
-  friend struct DefaultSingletonTraits<WebRTCInternals>;
+  friend struct DefaultSingletonTraits<ChromeWebRTCInternals>;
 
-  WebRTCInternals();
-  virtual ~WebRTCInternals();
+  ChromeWebRTCInternals();
+  virtual ~ChromeWebRTCInternals();
 
   // Send updates to observers on UI thread.
   void SendUpdate(const std::string& command, base::Value* value);
 
-  // Only the IO thread should access these fields.
+  // Only the UI thread should access these fields.

[jam, 2013/01/18 22:23:45]

nit: this class is now always used on the UI thread, so the threading comments
are unnecessary

[jiayl, 2013/01/18 22:39:51]

Done.

   ObserverList<WebRTCInternalsUIObserver> observers_;
   base::ListValue peer_connection_data_;
 };
 
-}  // namespace content
-
-#endif  // CONTENT_BROWSER_MEDIA_WEBRTC_INTERNALS_H_
+#endif  // CHROME_BROWSER_MEDIA_CHROME_WEBRTC_INTERNALS_H_