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

Unified Diff: testing/android/native_test/java/src/org/chromium/native_test/NativeBrowserTestActivity.java

Issue 1138993009: Revert of [Android] Refactor the native test wrappers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
Index: testing/android/native_test/java/src/org/chromium/native_test/NativeBrowserTestActivity.java
diff --git a/testing/android/native_test/java/src/org/chromium/native_test/NativeBrowserTestActivity.java b/testing/android/native_test/java/src/org/chromium/native_test/NativeBrowserTestActivity.java
deleted file mode 100644
index ac2ddeef84a6dbca5b71775ae60e536965f2757c..0000000000000000000000000000000000000000
--- a/testing/android/native_test/java/src/org/chromium/native_test/NativeBrowserTestActivity.java
+++ /dev/null
@@ -1,46 +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.native_test;
-
-import android.os.Bundle;
-
-/**
- * An {@link android.app.Activity} for running native browser tests.
- */
-public abstract class NativeBrowserTestActivity extends NativeTestActivity {
-
- private static final String BROWSER_TESTS_FLAGS[] = {
- // content::kSingleProcessTestsFlag
- "--single_process",
-
- // switches::kUseFakeDeviceForMediaStream
- "--use-fake-device-for-media-stream",
-
- // switches::kUseFakeUIForMediaStream
- "--use-fake-ui-for-media-stream"
- };
-
- @Override
- public void onCreate(Bundle savedInstanceState) {
- super.onCreate(savedInstanceState);
- for (String flag : BROWSER_TESTS_FLAGS) {
- appendCommandLineFlags(flag);
- }
- }
-
- @Override
- public void onStart() {
- initializeBrowserProcess();
- super.onStart();
- }
-
- /** Initializes the browser process.
- *
- * This generally includes loading native libraries and switching to the native command line,
- * among other things.
- */
- protected abstract void initializeBrowserProcess();
-
-}

Powered by Google App Engine
This is Rietveld 408576698