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

Side by Side Diff: chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/DocumentModeAssassinTest.java

Issue 2013883002: ✈ Remove unnecessary flag and DocumentModeManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.tabmodel; 5 package org.chromium.chrome.browser.tabmodel;
6 6
7 import android.content.Context; 7 import android.content.Context;
8 import android.content.SharedPreferences; 8 import android.content.SharedPreferences;
9 import android.test.MoreAsserts; 9 import android.test.MoreAsserts;
10 import android.test.suitebuilder.annotation.MediumTest; 10 import android.test.suitebuilder.annotation.MediumTest;
11 import android.test.suitebuilder.annotation.SmallTest; 11 import android.test.suitebuilder.annotation.SmallTest;
12 12
13 import org.chromium.base.ContextUtils; 13 import org.chromium.base.ContextUtils;
14 import org.chromium.base.Log; 14 import org.chromium.base.Log;
15 import org.chromium.base.StreamUtil; 15 import org.chromium.base.StreamUtil;
16 import org.chromium.base.ThreadUtils; 16 import org.chromium.base.ThreadUtils;
17 import org.chromium.base.test.util.AdvancedMockContext; 17 import org.chromium.base.test.util.AdvancedMockContext;
18 import org.chromium.chrome.browser.TabState; 18 import org.chromium.chrome.browser.TabState;
19 import org.chromium.chrome.browser.preferences.DocumentModeManager;
20 import org.chromium.chrome.browser.tab.Tab; 19 import org.chromium.chrome.browser.tab.Tab;
21 import org.chromium.chrome.browser.tabmodel.DocumentModeAssassin.DocumentModeAss assinForTesting; 20 import org.chromium.chrome.browser.tabmodel.DocumentModeAssassin.DocumentModeAss assinForTesting;
22 import org.chromium.chrome.browser.tabmodel.DocumentModeAssassin.DocumentModeAss assinObserver; 21 import org.chromium.chrome.browser.tabmodel.DocumentModeAssassin.DocumentModeAss assinObserver;
23 import org.chromium.chrome.browser.tabmodel.TabPersistentStoreTest.MockTabPersis tentStoreObserver; 22 import org.chromium.chrome.browser.tabmodel.TabPersistentStoreTest.MockTabPersis tentStoreObserver;
24 import org.chromium.chrome.browser.tabmodel.TabPersistentStoreTest.TestTabModelS elector; 23 import org.chromium.chrome.browser.tabmodel.TabPersistentStoreTest.TestTabModelS elector;
25 import org.chromium.chrome.browser.tabmodel.TestTabModelDirectory.TabStateInfo; 24 import org.chromium.chrome.browser.tabmodel.TestTabModelDirectory.TabStateInfo;
26 import org.chromium.chrome.browser.tabmodel.document.ActivityDelegate; 25 import org.chromium.chrome.browser.tabmodel.document.ActivityDelegate;
27 import org.chromium.chrome.browser.tabmodel.document.DocumentTabModel; 26 import org.chromium.chrome.browser.tabmodel.document.DocumentTabModel;
28 import org.chromium.chrome.browser.tabmodel.document.DocumentTabModelImpl; 27 import org.chromium.chrome.browser.tabmodel.document.DocumentTabModelImpl;
29 import org.chromium.chrome.browser.tabmodel.document.MockDocumentTabModel; 28 import org.chromium.chrome.browser.tabmodel.document.MockDocumentTabModel;
(...skipping 336 matching lines...) Expand 10 before | Expand all | Expand 10 after
366 mFinishAllDocumentActivitiesCallback.waitForCallback(0); 365 mFinishAllDocumentActivitiesCallback.waitForCallback(0);
367 changeDoneCallback.waitForCallback(0); 366 changeDoneCallback.waitForCallback(0);
368 confirmUserIsInTabbedMode(); 367 confirmUserIsInTabbedMode();
369 } 368 }
370 369
371 private void confirmUserIsInTabbedMode() { 370 private void confirmUserIsInTabbedMode() {
372 // Check that the preference was written out correctly. 371 // Check that the preference was written out correctly.
373 ThreadUtils.runOnUiThreadBlocking(new Runnable() { 372 ThreadUtils.runOnUiThreadBlocking(new Runnable() {
374 @Override 373 @Override
375 public void run() { 374 public void run() {
376 assertEquals(DocumentModeManager.OPTED_OUT_OF_DOCUMENT_MODE, 375 assertTrue(DocumentModeAssassin.isOptedOutOfDocumentMode());
377 DocumentModeManager.getInstance(mContext).getOptOutState ForTesting());
378 } 376 }
379 }); 377 });
380 } 378 }
381 379
382 /** 380 /**
383 * Tests that the {@link DocumentTabModel}'s data is properly saved out for a 381 * Tests that the {@link DocumentTabModel}'s data is properly saved out for a
384 * {@link TabModelImpl}. 382 * {@link TabModelImpl}.
385 */ 383 */
386 @MediumTest 384 @MediumTest
387 public void testWriteTabModelMetadata() throws Exception { 385 public void testWriteTabModelMetadata() throws Exception {
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
656 mDocumentModeDirectory.writeTabStateFile(TAB_STATE_INFO[i]); 654 mDocumentModeDirectory.writeTabStateFile(TAB_STATE_INFO[i]);
657 } 655 }
658 656
659 // Write out some random files that should have been ignored by migratio n. 657 // Write out some random files that should have been ignored by migratio n.
660 writeUselessFileToDirectory(mDocumentModeDirectory.getDataDirectory(), " ignored.file"); 658 writeUselessFileToDirectory(mDocumentModeDirectory.getDataDirectory(), " ignored.file");
661 writeUselessFileToDirectory(mDocumentModeDirectory.getDataDirectory(), 659 writeUselessFileToDirectory(mDocumentModeDirectory.getDataDirectory(),
662 TabState.SAVED_TAB_STATE_FILE_PREFIX + "_unparseable"); 660 TabState.SAVED_TAB_STATE_FILE_PREFIX + "_unparseable");
663 661
664 } 662 }
665 } 663 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698