Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(214)

Unified Diff: chrome/browser/metrics/desktop_engagement/desktop_engagement_observer.cc

Issue 2333113002: Rename DesktopEngagement* to DesktopSessionDuration*. (Closed)
Patch Set: Change name Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/metrics/desktop_engagement/desktop_engagement_observer.cc
diff --git a/chrome/browser/metrics/desktop_engagement/desktop_engagement_observer.cc b/chrome/browser/metrics/desktop_engagement/desktop_engagement_observer.cc
deleted file mode 100644
index bf4dda506a1497d4808b5de3f1fd1a80eebb4b6b..0000000000000000000000000000000000000000
--- a/chrome/browser/metrics/desktop_engagement/desktop_engagement_observer.cc
+++ /dev/null
@@ -1,64 +0,0 @@
-// Copyright 2016 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 "chrome/browser/metrics/desktop_engagement/desktop_engagement_observer.h"
-
-#include "chrome/browser/metrics/desktop_engagement/desktop_engagement_service.h"
-#include "content/public/browser/render_view_host.h"
-
-DEFINE_WEB_CONTENTS_USER_DATA_KEY(metrics::DesktopEngagementObserver);
-
-namespace metrics {
-
-DesktopEngagementObserver::DesktopEngagementObserver(
- content::WebContents* web_contents,
- DesktopEngagementService* service)
- : content::WebContentsObserver(web_contents), service_(service) {
- RegisterInputEventObserver(web_contents->GetRenderViewHost());
-}
-
-DesktopEngagementObserver::~DesktopEngagementObserver() {}
-
-// static
-DesktopEngagementObserver* DesktopEngagementObserver::CreateForWebContents(
- content::WebContents* web_contents) {
- DCHECK(web_contents);
-
- if (!DesktopEngagementService::IsInitialized())
- return nullptr;
-
- DesktopEngagementObserver* observer = FromWebContents(web_contents);
- if (!observer) {
- observer = new DesktopEngagementObserver(web_contents,
- DesktopEngagementService::Get());
- web_contents->SetUserData(UserDataKey(), observer);
- }
- return observer;
-}
-
-void DesktopEngagementObserver::RegisterInputEventObserver(
- content::RenderViewHost* host) {
- if (host != nullptr)
- host->GetWidget()->AddInputEventObserver(this);
-}
-
-void DesktopEngagementObserver::UnregisterInputEventObserver(
- content::RenderViewHost* host) {
- if (host != nullptr)
- host->GetWidget()->RemoveInputEventObserver(this);
-}
-
-void DesktopEngagementObserver::OnInputEvent(
- const blink::WebInputEvent& event) {
- service_->OnUserEvent();
-}
-
-void DesktopEngagementObserver::RenderViewHostChanged(
- content::RenderViewHost* old_host,
- content::RenderViewHost* new_host) {
- UnregisterInputEventObserver(old_host);
- RegisterInputEventObserver(new_host);
-}
-
-} // namespace metrics

Powered by Google App Engine
This is Rietveld 408576698