DBus MediaAnalyticsClient and media_perception pb.

extensions/browser/api/media_perception_private/media_perception_api_manager.cc

+// Copyright 2017 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 "extensions/browser/api/media_perception_private/media_perception_api_manager.h"
+
+#include "base/lazy_instance.h"
+#include "base/memory/ptr_util.h"
+#include "extensions/browser/event_router.h"
+
+namespace mpp = extensions::api::media_perception_private;
+
+namespace extensions {
+
+// static

[bmayer, 2017/04/13 17:41:59]

Static comments are typically discouraged in google3 so remove these unless its
common for the chromium codebase in which case stick to the convention.

[Luke Sorenson, 2017/04/13 17:58:21]

I saw this // static comment in other BrowserContext-based Factory methods so I
included it.

For example:
https://cs.chromium.org/chromium/src/extensions/browser/api/clipboard/clipboa...

+MediaPerceptionAPIManager* MediaPerceptionAPIManager::Get(
+    content::BrowserContext* context) {
+  return GetFactoryInstance()->Get(context);
+}
+
+static base::LazyInstance<
+    BrowserContextKeyedAPIFactory<MediaPerceptionAPIManager>>::DestructorAtExit
+    g_factory = LAZY_INSTANCE_INITIALIZER;
+
+// static
+BrowserContextKeyedAPIFactory<MediaPerceptionAPIManager>*
+MediaPerceptionAPIManager::GetFactoryInstance() {
+  return g_factory.Pointer();
+}
+
+MediaPerceptionAPIManager::MediaPerceptionAPIManager(
+    content::BrowserContext* context)
+    : browser_context_(context) {}
+
+MediaPerceptionAPIManager::~MediaPerceptionAPIManager() {}
+
+void MediaPerceptionAPIManager::TriggerOnMediaPerceptionEvent() {
+  EventRouter* router = EventRouter::Get(browser_context_);
+  if (router && router->HasEventListener(mpp::OnMediaPerception::kEventName)) {
+    mpp::MediaPerception media_perception;
+    media_perception.timestamp.reset(new int(777));
+    std::unique_ptr<Event> event(
+        new Event(events::MEDIA_PERCEPTION_PRIVATE_ON_MEDIA_PERCEPTION,
+                  mpp::OnMediaPerception::kEventName,
+                  std::move(mpp::OnMediaPerception::Create(media_perception))));
+    router->BroadcastEvent(std::move(event));
+  }
+}
+
+}  // namespace extensions