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

Unified Diff: blimp/engine/app/blimp_stability_metrics_provider_unittest.cc

Issue 2629743003: Remove all blimp engine code (Closed)
Patch Set: Use consistent comment style in //chrome Created 3 years, 11 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: blimp/engine/app/blimp_stability_metrics_provider_unittest.cc
diff --git a/blimp/engine/app/blimp_stability_metrics_provider_unittest.cc b/blimp/engine/app/blimp_stability_metrics_provider_unittest.cc
deleted file mode 100644
index f5a38806407054ff7e0353aabf424f0061578e7f..0000000000000000000000000000000000000000
--- a/blimp/engine/app/blimp_stability_metrics_provider_unittest.cc
+++ /dev/null
@@ -1,115 +0,0 @@
-// Copyright 2015 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 "blimp/engine/app/blimp_stability_metrics_provider.h"
-
-#include "base/macros.h"
-#include "components/metrics/proto/system_profile.pb.h"
-#include "components/prefs/pref_service.h"
-#include "components/prefs/scoped_user_pref_update.h"
-#include "components/prefs/testing_pref_service.h"
-#include "content/public/browser/browser_context.h"
-#include "content/public/browser/child_process_data.h"
-#include "content/public/browser/notification_details.h"
-#include "content/public/browser/notification_service.h"
-#include "content/public/browser/notification_source.h"
-#include "content/public/browser/notification_types.h"
-#include "content/public/browser/render_process_host.h"
-#include "content/public/browser/site_instance.h"
-#include "content/public/common/process_type.h"
-#include "content/public/test/mock_render_process_host.h"
-#include "content/public/test/test_browser_thread_bundle.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace {
-
-class BlimpStabilityMetricsProviderTest : public testing::Test {
- protected:
- BlimpStabilityMetricsProviderTest() : prefs_(new TestingPrefServiceSimple) {
- metrics::StabilityMetricsHelper::RegisterPrefs(prefs()->registry());
- }
-
- TestingPrefServiceSimple* prefs() { return prefs_.get(); }
-
- private:
- std::unique_ptr<TestingPrefServiceSimple> prefs_;
- content::TestBrowserThreadBundle thread_bundle_;
-
- DISALLOW_COPY_AND_ASSIGN(BlimpStabilityMetricsProviderTest);
-};
-
-} // namespace
-
-TEST_F(BlimpStabilityMetricsProviderTest, BrowserChildProcessObserver) {
- BlimpStabilityMetricsProvider provider(prefs());
-
- content::ChildProcessData child_process_data(content::PROCESS_TYPE_RENDERER);
- provider.BrowserChildProcessCrashed(child_process_data, 1);
- provider.BrowserChildProcessCrashed(child_process_data, 1);
-
- // Call ProvideStabilityMetrics to check that it will force pending tasks to
- // be executed immediately.
- metrics::SystemProfileProto system_profile;
-
- provider.ProvideStabilityMetrics(&system_profile);
-
- // Check current number of instances created.
- const metrics::SystemProfileProto_Stability& stability =
- system_profile.stability();
-
- EXPECT_EQ(2, stability.child_process_crash_count());
-}
-
-TEST_F(BlimpStabilityMetricsProviderTest, NotificationObserver) {
- BlimpStabilityMetricsProvider provider(prefs());
- content::NotificationSource irrelevant_source =
- content::Source<std::nullptr_t>(nullptr);
-
- // Load start should increase page load count.
- provider.Observe(
- content::NOTIFICATION_LOAD_START, irrelevant_source,
- content::NotificationService::NoDetails());
-
- // Crash and abnormal termination should increment renderer crash count.
- content::RenderProcessHost::RendererClosedDetails crash_details(
- base::TERMINATION_STATUS_PROCESS_CRASHED, 1);
- provider.Observe(
- content::NOTIFICATION_RENDERER_PROCESS_CLOSED, irrelevant_source,
- content::Details<content::RenderProcessHost::RendererClosedDetails>(
- &crash_details));
-
- content::RenderProcessHost::RendererClosedDetails term_details(
- base::TERMINATION_STATUS_ABNORMAL_TERMINATION, 1);
- provider.Observe(
- content::NOTIFICATION_RENDERER_PROCESS_CLOSED, irrelevant_source,
- content::Details<content::RenderProcessHost::RendererClosedDetails>(
- &term_details));
-
- // Kill does not increment renderer crash count.
- content::RenderProcessHost::RendererClosedDetails kill_details(
- base::TERMINATION_STATUS_PROCESS_WAS_KILLED, 1);
- provider.Observe(
- content::NOTIFICATION_RENDERER_PROCESS_CLOSED, irrelevant_source,
- content::Details<content::RenderProcessHost::RendererClosedDetails>(
- &kill_details));
-
- // Failed launch increments failed launch count.
- content::RenderProcessHost::RendererClosedDetails failed_launch_details(
- base::TERMINATION_STATUS_LAUNCH_FAILED, 1);
- provider.Observe(
- content::NOTIFICATION_RENDERER_PROCESS_CLOSED, irrelevant_source,
- content::Details<content::RenderProcessHost::RendererClosedDetails>(
- &failed_launch_details));
-
- metrics::SystemProfileProto system_profile;
-
- // Call ProvideStabilityMetrics to check that it will force pending tasks to
- // be executed immediately.
- provider.ProvideStabilityMetrics(&system_profile);
-
- EXPECT_EQ(1, system_profile.stability().page_load_count());
- EXPECT_EQ(2, system_profile.stability().renderer_crash_count());
- EXPECT_EQ(1, system_profile.stability().renderer_failed_launch_count());
- EXPECT_EQ(0, system_profile.stability().extension_renderer_crash_count());
-}
« no previous file with comments | « blimp/engine/app/blimp_stability_metrics_provider.cc ('k') | blimp/engine/app/blimp_system_url_request_context_getter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698