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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/OffTheRecordTabModelTest.java

Issue 2288353002: Replace references to OffTheRecord from OffTheRecordTabModel to Incognito (Closed)
Patch Set: actually rename to right thing Created 4 years, 4 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/tabmodel/OffTheRecordTabModelTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/OffTheRecordTabModelTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/OffTheRecordTabModelTest.java
deleted file mode 100644
index 3fcdc0fcedda6881665b24f9a2ad3cfac1fac7b5..0000000000000000000000000000000000000000
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/OffTheRecordTabModelTest.java
+++ /dev/null
@@ -1,60 +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.tabmodel;
-
-import android.test.suitebuilder.annotation.SmallTest;
-
-import org.chromium.base.ThreadUtils;
-import org.chromium.base.test.util.Feature;
-import org.chromium.chrome.browser.tab.Tab;
-import org.chromium.chrome.browser.tabmodel.TabModel.TabLaunchType;
-import org.chromium.chrome.test.ChromeTabbedActivityTestBase;
-import org.chromium.content_public.browser.LoadUrlParams;
-
-/**
- * Tests for OffTheRecordTabModel.
- */
-public class OffTheRecordTabModelTest extends ChromeTabbedActivityTestBase {
- private TabModel mTabModel;
-
- @Override
- public void startMainActivity() throws InterruptedException {
- startMainActivityOnBlankPage();
- mTabModel = getActivity().getTabModelSelector().getModel(true);
- }
-
- private class CloseAllDuringAddTabTabModelObserver extends EmptyTabModelObserver {
- @Override
- public void willAddTab(Tab tab, TabLaunchType type) {
- mTabModel.closeAllTabs();
- }
- }
-
- private void createTabOnUiThread() {
- ThreadUtils.runOnUiThreadBlocking(new Runnable() {
- @Override
- public void run() {
- getActivity().getTabCreator(true).createNewTab(new LoadUrlParams("about:blank"),
- TabLaunchType.FROM_CHROME_UI, null);
- }
- });
- }
-
- /**
- * Verify that a close all operation that occurs while a tab is being added does not crash the
- * browser and results in 1 valid tab. This test simulates the case where the user selects
- * "Close all incognito tabs" then quickly clicks the "+" button to add a new incognito tab.
- * See crbug.com/496651.
- */
- @SmallTest
- @Feature({"OffTheRecord"})
- public void testCloseAllDuringAddTabDoesNotCrash() {
- createTabOnUiThread();
- assertEquals(1, mTabModel.getCount());
- mTabModel.addObserver(new CloseAllDuringAddTabTabModelObserver());
- createTabOnUiThread();
- assertEquals(1, mTabModel.getCount());
- }
-}

Powered by Google App Engine
This is Rietveld 408576698