Plumb media data from renderers up to MediaInternals in the browser process.

content/browser/renderer_host/media/media_log_host.cc

Index: content/browser/renderer_host/media/media_log_host.cc
diff --git a/content/browser/renderer_host/media/media_log_host.cc b/content/browser/renderer_host/media/media_log_host.cc
new file mode 100644
index 0000000000000000000000000000000000000000..95bc70bc5cd84cbb52f9d3d417721e14816ebe1b
--- /dev/null
+++ b/content/browser/renderer_host/media/media_log_host.cc
@@ -0,0 +1,37 @@
+// 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 "content/browser/renderer_host/media/media_log_host.h"
+
+#include "content/browser/renderer_host/media/media_observer.h"
+#include "content/browser/resource_context.h"
+#include "content/common/media/media_log_messages.h"
+
+MediaLogHost::MediaLogHost(const content::ResourceContext* resource_context)
+    : resource_context_(resource_context) {}
+
+MediaLogHost::~MediaLogHost() {}
+
+void MediaLogHost::OnChannelClosing() {
+  BrowserMessageFilter::OnChannelClosing();

[jam, 2011/07/29 16:08:55]

you don't need to override this

[Scott Franklin, 2011/07/29 22:30:25]

This appeared to be standard practice in every other *host file I looked at,
though I suppose they all had something to clean up.

+}
+
+void MediaLogHost::OnDestruct() const {
+  BrowserThread::DeleteOnIOThread::Destruct(this);

[jam, 2011/07/29 16:08:55]

ditto. there's no reason that I see that makes you care about which thread this
object is destructed on

[Scott Franklin, 2011/07/29 22:30:25]

Ditto.

+}
+
+bool MediaLogHost::OnMessageReceived(const IPC::Message& message,
+                                     bool* message_was_ok) {
+  bool handled = true;
+  IPC_BEGIN_MESSAGE_MAP_EX(MediaLogHost, message, *message_was_ok)
+    IPC_MESSAGE_HANDLER(MediaLogMsg_MediaEvent, OnMediaEvent)

[jam, 2011/07/29 16:08:55]

given that you have just one message, it seems easiest to just dispatch this in
RenderMessageFilter

[Scott Franklin, 2011/07/29 22:30:25]

Yeah, I was expecting many messages, but then one message with an enum turned
out to be much simpler.

It is much easier. Done.

+    IPC_MESSAGE_UNHANDLED(handled = false);
+  IPC_END_MESSAGE_MAP_EX()
+
+  return handled;
+}
+
+void MediaLogHost::OnMediaEvent(const media::MediaLog::Event& event) {
+  resource_context_->media_observer()->OnMediaEvent(event);
+}