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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2088443003: Shortcut ctrl+shift+T added on android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unused includes in recently_closed_tab_bridge.cc Created 4 years, 5 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
« chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelImpl.java ('K') | « chrome/android/javatests/src/org/chromium/chrome/browser/multiwindow/MultiWindowUtilsTest.java ('k') | chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/document/MockDocumentTabModel.java » ('j') | chrome/browser/android/recently_closed_tabs_bridge.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/UndoTabModelTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/UndoTabModelTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/UndoTabModelTest.java
index 450008e09e7430010450826c8935ed3bfbc5039d..d1b0fa353aa8f2a8a43375dc086267b408f2db47 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/UndoTabModelTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/UndoTabModelTest.java
@@ -4,17 +4,27 @@
package org.chromium.chrome.browser.tabmodel;
+import android.annotation.TargetApi;
+import android.os.Build;
import android.test.suitebuilder.annotation.MediumTest;
import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.FlakyTest;
+import org.chromium.base.test.util.MinAndroidSdkLevel;
import org.chromium.base.test.util.Restriction;
+import org.chromium.base.test.util.UrlUtils;
+import org.chromium.chrome.browser.ChromeTabbedActivity;
+import org.chromium.chrome.browser.ChromeTabbedActivity2;
+import org.chromium.chrome.browser.multiwindow.MultiWindowUtilsTest;
+import org.chromium.chrome.browser.tab.EmptyTabObserver;
import org.chromium.chrome.browser.tab.Tab;
import org.chromium.chrome.browser.tabmodel.TabModel.TabLaunchType;
import org.chromium.chrome.browser.tabmodel.TabModel.TabSelectionType;
import org.chromium.chrome.test.ChromeTabbedActivityTestBase;
import org.chromium.chrome.test.util.ChromeRestriction;
import org.chromium.content.browser.test.util.CallbackHelper;
+import org.chromium.content.browser.test.util.Criteria;
+import org.chromium.content.browser.test.util.CriteriaHelper;
import org.chromium.content_public.browser.LoadUrlParams;
import java.util.concurrent.TimeoutException;
@@ -24,6 +34,8 @@ import java.util.concurrent.TimeoutException;
*/
public class UndoTabModelTest extends ChromeTabbedActivityTestBase {
private static final Tab[] EMPTY = new Tab[] { };
+ private static final String TEST_URL_0 = UrlUtils.encodeHtmlDataUri("<html>test_url_0.</html>");
+ private static final String TEST_URL_1 = UrlUtils.encodeHtmlDataUri("<html>test_url_1</html>");
@Override
public void startMainActivity() throws InterruptedException {
@@ -75,6 +87,28 @@ public class UndoTabModelTest extends ChromeTabbedActivityTestBase {
});
}
+ private void createFullyLoadedTabOnUiThread(final ChromeTabbedActivity activity,
+ final String url) {
+ final CallbackHelper tabCallbackHelper = new CallbackHelper();
+ final TestTabObserver observer = new TestTabObserver(tabCallbackHelper);
+
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ @Override
+ public void run() {
+ activity.getTabCreator(false).createNewTab(new LoadUrlParams(url),
+ TabLaunchType.FROM_CHROME_UI, null).addObserver(observer);
+ }
+ });
+
+ // Must wait for the page to be fully loaded.
+ try {
+ tabCallbackHelper.waitForCallback(0);
+ } catch (TimeoutException e) {
+ fail("Failed to load the tab.");
+ } catch (InterruptedException e) {
+ }
+ }
+
private void selectTabOnUiThread(final TabModel model, final Tab tab) {
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
@@ -321,6 +355,28 @@ public class UndoTabModelTest extends ChromeTabbedActivityTestBase {
}
}
+ private void openMostRecentlyClosedTabOnUiThread(final TabModelSelector selector) {
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ @Override
+ public void run() {
+ ((TabModelSelectorImpl) selector).getCurrentModel().openMostRecentlyClosedTab();
+ }
+ });
+ }
+
+ // Helper class that notifies when a page load is finished.
+ private static class TestTabObserver extends EmptyTabObserver {
Theresa 2016/07/07 19:00:13 nit: Maybe this should be called TabLoadedObserver
xingliu 2016/07/08 21:26:48 Done.
+ private CallbackHelper mCallbackHelper;
+ public TestTabObserver(CallbackHelper callbackHelper) {
+ super();
+ mCallbackHelper = callbackHelper;
+ }
+ @Override
+ public void onPageLoadFinished(Tab tab) {
+ mCallbackHelper.notifyCalled();
+ }
+ }
+
/**
* Test undo with a single tab with the following actions/expected states:
* Action Model List Close List Comprehensive List
@@ -1410,4 +1466,127 @@ public class UndoTabModelTest extends ChromeTabbedActivityTestBase {
assertTrue(tab0.isClosing());
assertFalse(tab0.isInitialized());
}
+
+ /**
+ * Test opening recently closed tabs using the rewound list in Java
Theresa 2016/07/07 19:00:13 nit: add a period after "Java"
xingliu 2016/07/08 21:26:48 Done.
+ * @throws InterruptedException
+ */
+ @MediumTest
+ public void testOpenRecentlyClosedTab() throws InterruptedException {
+ TabModelSelector selector = getActivity().getTabModelSelector();
+ TabModel model = selector.getModel(false);
+ ChromeTabCreator tabCreator = getActivity().getTabCreator(false);
+
+ createTabOnUiThread(tabCreator);
+
+ Tab tab0 = model.getTabAt(0);
+ Tab tab1 = model.getTabAt(1);
+ Tab[] allTabs = new Tab[]{tab0, tab1};
+
+ closeTabOnUiThread(model, tab1, true);
+ checkState(model, new Tab[]{tab0}, tab0, new Tab[]{tab1}, allTabs, tab0);
+
+ // Ensure tab recovery, ordering, and reuse of {@link Tab} objects in java.
Theresa 2016/07/07 19:00:13 nit: capitalize Java
xingliu 2016/07/08 21:26:48 Done.
+ openMostRecentlyClosedTabOnUiThread(selector);
+ checkState(model, allTabs, tab0, EMPTY, allTabs, tab0);
+ }
+
+ /**
+ * Test opening recently closed tab using native tab restore service.
+ * @throws InterruptedException
+ */
+ @MediumTest
+ public void testOpenRecentlyClosedTabNative() throws InterruptedException {
+ final TabModelSelector selector = getActivity().getTabModelSelector();
+ final TabModel model = selector.getModel(false);
+
+ // Create new tab and attach observer to listen to loaded event.
+ // We need to listen for page load finished events, then it has navigation
Theresa 2016/07/07 19:00:13 nit: maybe rephrase this as something like "Native
xingliu 2016/07/08 21:26:48 Done.
+ // history, then native code can successfully recover the page.
+ createFullyLoadedTabOnUiThread(getActivity(), TEST_URL_0);
+
+ // Close the tab, and commit pending closure.
+ assertEquals(model.getCount(), 2);
+ closeTabOnUiThread(model, model.getTabAt(1), false);
+ assertEquals(1, model.getCount());
+
+ // Recover the page.
+ openMostRecentlyClosedTabOnUiThread(selector);
+
+ assertEquals(2, model.getCount());
+ Tab newTab = model.getTabAt(1);
+ assertEquals(TEST_URL_0, newTab.getUrl());
+ Tab[] tabs = new Tab[]{model.getTabAt(0), newTab};
+ checkState(model, tabs, newTab, EMPTY, tabs, newTab);
+ }
+
+ /**
+ * Test opening recently closed tab when we have multiple window.
Theresa 2016/07/07 19:00:13 nit: s/window/windows
xingliu 2016/07/08 21:26:48 Done.
+ * @throws InterruptedException
+ */
+ @MediumTest
+ @MinAndroidSdkLevel(24)
+ @TargetApi(Build.VERSION_CODES.LOLLIPOP)
+ public void testOpenRecentlyClosedTabMultiWindow() throws InterruptedException {
Theresa 2016/07/07 19:00:13 Thank you for adding this test. Can we do one more
xingliu 2016/07/08 21:26:48 Sure, also did manual testing, fallback logic work
+ final ChromeTabbedActivity2 secondActivity =
+ MultiWindowUtilsTest.createSecondChromeTabbedActivity(getActivity());
+
+ // Wait for the second window to be fully initialized.
+ CriteriaHelper.pollUiThread(new Criteria() {
+ @Override
+ public boolean isSatisfied() {
+ return secondActivity.getTabModelSelector().isTabStateInitialized();
+ }
+ });
+
+ // First window context.
+ final TabModelSelector firstSelector = getActivity().getTabModelSelector();
+ final TabModel firstModel = firstSelector.getModel(false);
+
+ // Second window context.
+ final TabModelSelector secondSelector = secondActivity.getTabModelSelector();
+ final TabModel secondModel = secondSelector.getModel(false);
+
+ // Assert initial number of tabs in two windows.
+ assertEquals("Unexpecetd number of tabs in first window.", 1, firstModel.getCount());
Theresa 2016/07/07 19:00:13 s/Unexpecetd/Unexpected
xingliu 2016/07/08 21:26:48 Done.My bad..
+ assertEquals("Unexpecetd number of tabs in second window.", 1, secondModel.getCount());
+
+ // Create tabs.
+ createFullyLoadedTabOnUiThread(getActivity(), TEST_URL_0);
+ createFullyLoadedTabOnUiThread(secondActivity, TEST_URL_1);
+
+ assertEquals("Unexpecetd number of tabs in first window.", 2, firstModel.getCount());
+ assertEquals("Unexpecetd number of tabs in second window.", 2, secondModel.getCount());
+
+ // Close one tab in the first window.
+ closeTabOnUiThread(firstModel, firstModel.getTabAt(1), false);
+ assertEquals("Unexpecetd number of tabs in first window.", 1, firstModel.getCount());
+ assertEquals("Unexpecetd number of tabs in second window.", 2, secondModel.getCount());
+
+ // Close one tab in the second window.
+ closeTabOnUiThread(secondModel, secondModel.getTabAt(1), false);
+ assertEquals("Unexpecetd number of tabs in first window.", 1, firstModel.getCount());
+ assertEquals("Unexpecetd number of tabs in second window.", 1, secondModel.getCount());
+
+ // Restore one tab.
+ openMostRecentlyClosedTabOnUiThread(firstSelector);
+ assertEquals("Unexpecetd number of tabs in first window.", 1, firstModel.getCount());
+ assertEquals("Unexpecetd number of tabs in second window.", 2, secondModel.getCount());
+
+ // Restore one more tab.
+ openMostRecentlyClosedTabOnUiThread(firstSelector);
+
+ // Check final states of both windows.
+ Tab[] firstWindowTabs = new Tab[]{firstModel.getTabAt(0), firstModel.getTabAt(1)};
+ Tab[] secondWindowTabs = new Tab[]{secondModel.getTabAt(0), secondModel.getTabAt(1)};
+ checkState(firstModel, firstWindowTabs, firstModel.getTabAt(1), EMPTY, firstWindowTabs,
+ firstModel.getTabAt(1));
+ checkState(secondModel, secondWindowTabs, secondModel.getTabAt(1), EMPTY, secondWindowTabs,
+ secondModel.getTabAt(1));
+ assertEquals(TEST_URL_0, firstWindowTabs[1].getUrl());
+ assertEquals(TEST_URL_1, secondWindowTabs[1].getUrl());
+
+ // Close the second window.
+ secondActivity.finishAndRemoveTask();
+ }
}
« chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelImpl.java ('K') | « chrome/android/javatests/src/org/chromium/chrome/browser/multiwindow/MultiWindowUtilsTest.java ('k') | chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/document/MockDocumentTabModel.java » ('j') | chrome/browser/android/recently_closed_tabs_bridge.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698