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

Unified Diff: chromecast/metrics/cast_stability_metrics_provider.h

Issue 638803002: Reorganizes Chromecast code to better reflect functional dependencies. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: extra DEPS: chromecast/crash/android --> chromecast/common Created 6 years, 2 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: chromecast/metrics/cast_stability_metrics_provider.h
diff --git a/chromecast/metrics/cast_stability_metrics_provider.h b/chromecast/metrics/cast_stability_metrics_provider.h
deleted file mode 100644
index bba94b62f438b9c1eb69ad0f6df31f2c9bdb98ad..0000000000000000000000000000000000000000
--- a/chromecast/metrics/cast_stability_metrics_provider.h
+++ /dev/null
@@ -1,73 +0,0 @@
-// Copyright 2014 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 CHROMECAST_METRICS_CAST_STABILITY_METRICS_PROVIDER_H_
-#define CHROMECAST_METRICS_CAST_STABILITY_METRICS_PROVIDER_H_
-
-#include "base/basictypes.h"
-#include "base/metrics/user_metrics.h"
-#include "base/process/kill.h"
-#include "components/metrics/metrics_provider.h"
-#include "content/public/browser/browser_child_process_observer.h"
-#include "content/public/browser/notification_observer.h"
-#include "content/public/browser/notification_registrar.h"
-
-class MetricsService;
-class PrefRegistrySimple;
-
-namespace content {
-class RenderProcessHost;
-class WebContents;
-}
-
-namespace chromecast {
-namespace metrics {
-
-// CastStabilityMetricsProvider gathers and logs stability-related metrics.
-// Loosely based on ChromeStabilityMetricsProvider from chrome/browser/metrics.
-class CastStabilityMetricsProvider
- : public ::metrics::MetricsProvider,
- public content::BrowserChildProcessObserver,
- public content::NotificationObserver {
- public:
- // Registers local state prefs used by this class.
- static void RegisterPrefs(PrefRegistrySimple* registry);
-
- CastStabilityMetricsProvider();
- virtual ~CastStabilityMetricsProvider();
-
- // metrics::MetricsDataProvider implementation:
- virtual void OnRecordingEnabled() override;
- virtual void OnRecordingDisabled() override;
- virtual void ProvideStabilityMetrics(
- ::metrics::SystemProfileProto* system_profile_proto) override;
-
- private:
- // content::NotificationObserver implementation:
- virtual void Observe(int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) override;
-
- // content::BrowserChildProcessObserver implementation:
- virtual void BrowserChildProcessCrashed(
- const content::ChildProcessData& data) override;
-
- // Records a renderer process crash.
- void LogRendererCrash(content::RenderProcessHost* host,
- base::TerminationStatus status,
- int exit_code);
-
- // Records a renderer process hang.
- void LogRendererHang();
-
- // Registrar for receiving stability-related notifications.
- content::NotificationRegistrar registrar_;
-
- DISALLOW_COPY_AND_ASSIGN(CastStabilityMetricsProvider);
-};
-
-} // namespace metrics
-} // namespace chromecast
-
-#endif // CHROMECAST_METRICS_CAST_STABILITY_METRICS_PROVIDER_H_
« no previous file with comments | « chromecast/metrics/cast_metrics_service_client_unittest.cc ('k') | chromecast/metrics/cast_stability_metrics_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698