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

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

Issue 2123863004: ScreenCapture for Android phase1, part II (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments and rebase 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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.app.Activity; 7 import android.app.Activity;
8 import android.test.InstrumentationTestCase; 8 import android.test.InstrumentationTestCase;
9 import android.test.UiThreadTest; 9 import android.test.UiThreadTest;
10 import android.test.suitebuilder.annotation.SmallTest; 10 import android.test.suitebuilder.annotation.SmallTest;
(...skipping 237 matching lines...) Expand 10 before | Expand all | Expand 10 after
248 Tab tab1 = selector0.addMockTab(); 248 Tab tab1 = selector0.addMockTab();
249 Tab tab2 = selector1.addMockIncognitoTab(); 249 Tab tab2 = selector1.addMockIncognitoTab();
250 250
251 assertFalse(manager.tabExistsInAnySelector(tab1.getId() - 1)); 251 assertFalse(manager.tabExistsInAnySelector(tab1.getId() - 1));
252 assertTrue(manager.tabExistsInAnySelector(tab1.getId())); 252 assertTrue(manager.tabExistsInAnySelector(tab1.getId()));
253 assertTrue(manager.tabExistsInAnySelector(tab2.getId())); 253 assertTrue(manager.tabExistsInAnySelector(tab2.getId()));
254 assertFalse(manager.tabExistsInAnySelector(tab2.getId() + 1)); 254 assertFalse(manager.tabExistsInAnySelector(tab2.getId() + 1));
255 255
256 AsyncTabParamsManager.getAsyncTabParams().clear(); 256 AsyncTabParamsManager.getAsyncTabParams().clear();
257 final int asyncTabId = 123; 257 final int asyncTabId = 123;
258 final TabReparentingParams dummyParams = new TabReparentingParams(null, null, null, false); 258 final TabReparentingParams dummyParams = new TabReparentingParams(null, null, null, false);
Ted C 2016/08/18 03:55:01 I think this was failing because the tab in the pa
braveyao 2016/08/18 18:16:51 Done. I must read the codes wrong and thought Inte
259 assertFalse(manager.tabExistsInAnySelector(asyncTabId)); 259 assertFalse(manager.tabExistsInAnySelector(asyncTabId));
260 AsyncTabParamsManager.add(asyncTabId, dummyParams); 260 AsyncTabParamsManager.add(asyncTabId, dummyParams);
261 try { 261 try {
262 assertTrue(manager.tabExistsInAnySelector(asyncTabId)); 262 assertTrue(manager.tabExistsInAnySelector(asyncTabId));
263 } finally { 263 } finally {
264 AsyncTabParamsManager.getAsyncTabParams().clear(); 264 AsyncTabParamsManager.getAsyncTabParams().clear();
265 } 265 }
266 } 266 }
267
268 /**
269 * Tests that getTabById() functions properly.
270 */
271 @SmallTest
272 @Feature({"Multiwindow"})
273 @UiThreadTest
274 public void testGetTabById() {
275 final TabWindowManager manager = TabWindowManager.getInstance();
276
277 ChromeActivity activity0 = buildActivity();
278 ChromeActivity activity1 = buildActivity();
279 MockTabModelSelector selector0 = requestSelector(activity0, 0);
280 MockTabModelSelector selector1 = requestSelector(activity1, 1);
281 Tab tab1 = selector0.addMockTab();
282 Tab tab2 = selector1.addMockIncognitoTab();
283
284 assertNull(manager.getTabById(tab1.getId() - 1));
285 assertNotNull(manager.getTabById(tab1.getId()));
286 assertNotNull(manager.getTabById(tab2.getId()));
287 assertNull(manager.getTabById(tab2.getId() + 1));
288 }
267 } 289 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698