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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeLowEndTest.java

Issue 2011433003: [Hera] Excise a bunch of document mode code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Deprecate histograms, clean up another unused one. Created 4 years, 7 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/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeLowEndTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeLowEndTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeLowEndTest.java
deleted file mode 100644
index 28b076e207df7b6c2893eac66c06f684268eef89..0000000000000000000000000000000000000000
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeLowEndTest.java
+++ /dev/null
@@ -1,124 +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.
-
-package org.chromium.chrome.browser.document;
-
-import static org.chromium.base.test.util.Restriction.RESTRICTION_TYPE_LOW_END_DEVICE;
-
-import android.app.ActivityManager;
-import android.app.ActivityManager.RunningServiceInfo;
-import android.content.Context;
-import android.os.Build;
-import android.test.suitebuilder.annotation.MediumTest;
-
-import org.chromium.base.BaseSwitches;
-import org.chromium.base.test.util.CommandLineFlags;
-import org.chromium.base.test.util.DisabledTest;
-import org.chromium.base.test.util.MinAndroidSdkLevel;
-import org.chromium.base.test.util.Restriction;
-import org.chromium.chrome.browser.ChromeApplication;
-import org.chromium.chrome.browser.tab.EmptyTabObserver;
-import org.chromium.chrome.browser.tab.Tab;
-import org.chromium.chrome.browser.tabmodel.EmptyTabModelSelectorObserver;
-import org.chromium.chrome.browser.tabmodel.document.DocumentTabModelSelector;
-import org.chromium.content.app.SandboxedProcessService;
-import org.chromium.content.browser.test.util.CallbackHelper;
-
-import java.util.List;
-import java.util.concurrent.atomic.AtomicReference;
-
-/**
- * Tests the how Document mode works on low end devices.
- */
-@MinAndroidSdkLevel(Build.VERSION_CODES.LOLLIPOP)
-// TODO(dskiba): remove the following switch once we have Svelte bots running L
-@CommandLineFlags.Add(BaseSwitches.ENABLE_LOW_END_DEVICE_MODE)
-@DisabledTest
-public class DocumentModeLowEndTest extends DocumentModeTestBase {
-
- @Restriction(RESTRICTION_TYPE_LOW_END_DEVICE)
- @MediumTest
- public void testNewTabLoadLowEnd() throws Exception {
- launchViaLaunchDocumentInstance(false, HREF_LINK, "href link page");
-
- final AtomicReference<Tab> backgroundTab = new AtomicReference<Tab>();
- final CallbackHelper tabCreatedCallback = new CallbackHelper();
- final CallbackHelper tabLoadStartedCallback = new CallbackHelper();
-
- final DocumentTabModelSelector selector =
- ChromeApplication.getDocumentTabModelSelector();
- selector.addObserver(new EmptyTabModelSelectorObserver() {
- @Override
- public void onNewTabCreated(final Tab newTab) {
- selector.removeObserver(this);
- backgroundTab.set(newTab);
- tabCreatedCallback.notifyCalled();
-
- assertFalse(newTab.getWebContents().isLoadingToDifferentDocument());
-
- newTab.addObserver(new EmptyTabObserver() {
- @Override
- public void onPageLoadStarted(Tab tab, String url) {
- newTab.removeObserver(this);
- tabLoadStartedCallback.notifyCalled();
- }
- });
- }
- });
-
- openLinkInBackgroundTab();
-
- // Tab should be created, but shouldn't start loading until we switch to it.
- assertEquals(1, tabCreatedCallback.getCallCount());
- assertEquals(0, tabLoadStartedCallback.getCallCount());
-
- switchToTab(backgroundTab.get());
-
- tabLoadStartedCallback.waitForCallback(0);
- }
-
- /**
- * Tests that "Open in new tab" command doesn't create renderer per tab
- * on low end devices.
- */
- @Restriction(RESTRICTION_TYPE_LOW_END_DEVICE)
- @MediumTest
- public void testNewTabRenderersLowEnd() throws Exception {
- launchViaLaunchDocumentInstance(false, HREF_LINK, "href link page");
-
- // Ignore any side effects that the first background tab might produce.
- openLinkInBackgroundTab();
-
- int rendererCountBefore = countRenderers();
-
- final int newTabCount = 5;
- for (int i = 0; i != newTabCount; ++i) {
- openLinkInBackgroundTab();
- }
-
- int rendererCountAfter = countRenderers();
-
- assertEquals(rendererCountBefore, rendererCountAfter);
- }
-
- /**
- * Returns the number of currently running renderer services.
- */
- private int countRenderers() {
- ActivityManager activityManager = (ActivityManager) mContext.getSystemService(
- Context.ACTIVITY_SERVICE);
-
- int rendererCount = 0;
- List<RunningServiceInfo> serviceInfos = activityManager.getRunningServices(
- Integer.MAX_VALUE);
- for (RunningServiceInfo serviceInfo : serviceInfos) {
- if (serviceInfo.service.getClassName().startsWith(
- SandboxedProcessService.class.getName())) {
- rendererCount++;
- }
- }
-
- return rendererCount;
- }
-}

Powered by Google App Engine
This is Rietveld 408576698