Moving WebRTC logging related files from content to chrome.

chrome/renderer/media/webrtc_logging_handler_impl.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_RENDERER_MEDIA_WEBRTC_LOGGING_HANDLER_IMPL_H_
-#define CONTENT_RENDERER_MEDIA_WEBRTC_LOGGING_HANDLER_IMPL_H_
+#ifndef CHROME_RENDERER_MEDIA_WEBRTC_LOGGING_HANDLER_IMPL_H_
+#define CHROME_RENDERER_MEDIA_WEBRTC_LOGGING_HANDLER_IMPL_H_
 
 #include <string>
 
 #include "base/shared_memory.h"
-#include "base/threading/non_thread_safe.h"
-#include "content/common/content_export.h"
+#include "content/public/renderer/webrtc_log_message_delegate.h"
 #include "ipc/ipc_channel_proxy.h"
-#include "third_party/libjingle/overrides/logging/log_message_delegate.h"
 
 namespace base {
 class MessageLoopProxy;
 }
 
-namespace content {
+namespace chrome {
 
 class PartialCircularBuffer;
 class WebRtcLoggingMessageFilter;
 
 // WebRtcLoggingHandlerImpl handles WebRTC logging. There is one object per
 // render process, owned by WebRtcLoggingMessageFilter. It communicates with
 // WebRtcLoggingHandlerHost and receives logging messages from libjingle and
 // writes them to a shared memory buffer.
-class CONTENT_EXPORT WebRtcLoggingHandlerImpl
-    : public NON_EXPORTED_BASE(talk_base::LogMessageDelegate),
-      public NON_EXPORTED_BASE(base::NonThreadSafe) {
+class WebRtcLoggingHandlerImpl

[jam, 2013/05/30 15:01:17]

nit: chrome implementations of content interface Foo are usually named
ChromeFoo, so I would name this ChromeWebRtcLoggingHandler

[Henrik Grunell, 2013/05/31 19:26:38]

So, wouldn't this class be called ChromeWebRtcLogMessageDelegate then to follow
the naming convention strictly? I'll land this CL and change the name separately
afterwards based on your response.

[jam, 2013/05/31 19:38:10]

oops, sorry yes that would be ChromeWebRtcLogMessageDelegate

+    : public content::WebRtcLogMessageDelegate,
+      public base::NonThreadSafe {
  public:
   WebRtcLoggingHandlerImpl(
-      const scoped_refptr<base::MessageLoopProxy>& io_message_loop);
+      const scoped_refptr<base::MessageLoopProxy>& io_message_loop,
+      WebRtcLoggingMessageFilter* message_filter);
 
   virtual ~WebRtcLoggingHandlerImpl();
 
-  // talk_base::LogMessageDelegate implementation.
+  // content::WebRtcLogMessageDelegate implementation.
+  virtual void InitLogging(const std::string& app_session_id,
+                           const std::string& app_url) OVERRIDE;
   virtual void LogMessage(const std::string& message) OVERRIDE;
 
   void OnFilterRemoved();
 
   void OnLogOpened(base::SharedMemoryHandle handle, uint32 length);
   void OnOpenLogFailed();
 
  private:
   scoped_refptr<base::MessageLoopProxy> io_message_loop_;
   scoped_ptr<base::SharedMemory> shared_memory_;
-  scoped_ptr<content::PartialCircularBuffer> circular_buffer_;
+  scoped_ptr<chrome::PartialCircularBuffer> circular_buffer_;
+
+  WebRtcLoggingMessageFilter* message_filter_;
+  bool log_initialized_;
 
   DISALLOW_COPY_AND_ASSIGN(WebRtcLoggingHandlerImpl);
 };
 
-}  // namespace content
+}  // namespace chrome
 
-#endif  // CONTENT_RENDERER_MEDIA_WEBRTC_LOGGING_HANDLER_IMPL_H_
+#endif  // CHROME_RENDERER_MEDIA_WEBRTC_LOGGING_HANDLER_IMPL_H_