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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/ChromeBrowserConnectionTest.java

Issue 1172753002: Rename CustomTabs service related classes and interfaces (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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/customtabs/ChromeBrowserConnectionTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/ChromeBrowserConnectionTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/ChromeBrowserConnectionTest.java
deleted file mode 100644
index f1b6489aa5096aa6a0a8c8252a5a57617bcdd178..0000000000000000000000000000000000000000
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/customtabs/ChromeBrowserConnectionTest.java
+++ /dev/null
@@ -1,137 +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.customtabs;
-
-import android.app.Application;
-import android.content.Context;
-import android.os.Bundle;
-import android.os.IBinder;
-import android.os.Process;
-import android.test.InstrumentationTestCase;
-import android.test.suitebuilder.annotation.SmallTest;
-
-/** Tests for ChromeBrowserConnection. */
-public class ChromeBrowserConnectionTest extends InstrumentationTestCase {
- private ChromeBrowserConnection mBrowserConnection;
- private static final String URL = "http://www.google.com";
- private static final String INVALID_SCHEME_URL = "intent://www.google.com";
-
- @Override
- protected void setUp() throws Exception {
- super.setUp();
- Context context = getInstrumentation().getTargetContext().getApplicationContext();
- mBrowserConnection = ChromeBrowserConnection.getInstance((Application) context);
- }
-
- @Override
- protected void tearDown() throws Exception {
- super.tearDown();
- mBrowserConnection.cleanup(Process.myUid());
- }
-
- /**
- * Tests that we can register the callback. Registering null returns an
- * error code, and multiple registrations are not allowed.
- */
- @SmallTest
- public void testFinishSetup() {
- assertTrue("It should not be possible to set a null callback.",
- mBrowserConnection.finishSetup(null) != 0);
- IBrowserConnectionCallback cb = new IBrowserConnectionCallback.Stub() {
- @Override
- public void onUserNavigationStarted(long sessionId, String url, Bundle extras) {}
- @Override
- public void onUserNavigationFinished(long sessionId, String url, Bundle extras) {}
- @Override
- public IBinder asBinder() {
- return this;
- }
- };
- assertEquals(0, mBrowserConnection.finishSetup(cb));
- assertTrue("It should not be possible to set the callback twice.",
- mBrowserConnection.finishSetup(cb) != 0);
- }
-
- /**
- * Tests that {@link ChromeBrowserConnection#warmup(long)} succeeds and can
- * be issued multiple times.
- */
- @SmallTest
- public void testCanWarmup() {
- assertEquals(0, mBrowserConnection.warmup(0));
- // Can call it several times.
- assertEquals(0, mBrowserConnection.warmup(0));
- }
-
- /**
- * Tests that the session ID is positive, multiple sessions can be created,
- * and {@link ChromeBrowserConnection#newSession()} doesn't always return
- * the same session ID.
- */
- @SmallTest
- public void testNewSession() {
- long sessionId = mBrowserConnection.newSession();
- assertTrue("Session IDs should be strictly positive.", sessionId > 0);
- assertTrue("Session IDs should be unique.", mBrowserConnection.newSession() != sessionId);
- }
-
- /**
- * Calls warmup() and mayLaunchUrl(), checks for the expected result
- * (success or failure) and returns the result code.
- */
- private long assertWarmupAndMayLaunchUrl(long id, String url, boolean shouldSucceed) {
- mBrowserConnection.warmup(0);
- long sessionId = id == 0 ? mBrowserConnection.newSession() : id;
- mBrowserConnection.mayLaunchUrl(sessionId, url, null, null);
- long result = mBrowserConnection.mayLaunchUrl(sessionId, url, null, null);
- if (shouldSucceed) {
- assertEquals(sessionId, result);
- } else {
- assertTrue("The result should be negative to signal failure.", result < 0);
- }
- return result;
- }
-
- /**
- * Tests that
- * {@link ChromeBrowserConnection#mayLaunchUrl(long, String, Bundle, List<Bundle>)}
- * returns an error when called with an invalid session ID.
- */
- @SmallTest
- public void testNoMayLaunchUrlWithInvalidSessionId() {
- assertWarmupAndMayLaunchUrl(42, URL, false);
- assertWarmupAndMayLaunchUrl(-1, URL, false);
- }
-
- /**
- * Tests that
- * {@link ChromeBrowserConnection#mayLaunchUrl(long, String, Bundle, List<Bundle>)}
- * rejects invalid URL schemes.
- */
- @SmallTest
- public void testNoMayLaunchUrlWithInvalidScheme() {
- assertWarmupAndMayLaunchUrl(0, INVALID_SCHEME_URL, false);
- }
-
- /**
- * Tests that
- * {@link ChromeBrowserConnection#mayLaunchUrl(long, String, Bundle, List<Bundle>)}
- * succeeds.
- */
- @SmallTest
- public void testMayLaunchUrl() {
- assertWarmupAndMayLaunchUrl(0, URL, true);
- }
-
- /**
- * Tests that session IDs are forgotten properly.
- */
- @SmallTest
- public void testForgetsSessionId() {
- long sessionId = assertWarmupAndMayLaunchUrl(0, URL, true);
- mBrowserConnection.cleanup(Process.myUid());
- assertWarmupAndMayLaunchUrl(sessionId, URL, false);
- }
-}

Powered by Google App Engine
This is Rietveld 408576698