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

Unified Diff: tools/testing/dart/browser_controller.dart

Issue 16168005: Remove hack for android resources. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/browser_controller.dart
diff --git a/tools/testing/dart/browser_controller.dart b/tools/testing/dart/browser_controller.dart
index 174f360800c55ffafc05da4668d43c3e3b080c34..9b5e966f03278829397620d9ed3f1469d92d69ae 100644
--- a/tools/testing/dart/browser_controller.dart
+++ b/tools/testing/dart/browser_controller.dart
@@ -342,21 +342,11 @@ class AndroidChrome extends Browser {
var turnScreenOnIntent =
new Intent(mainAction, turnScreenOnPackage, '.Main');
- // FIXME(kustermann): Remove this hack as soon as we've got a v8 mirror in
- // golo
var testing_resources_dir =
new Path('third_party/android_testing_resources');
- var testing_resources_dir_tmp =
- new Path('/tmp/android_testing_resources');
if (!new Directory.fromPath(testing_resources_dir).existsSync()) {
- DebugLogger.warning("$testing_resources_dir doesn't exist, "
- "trying to use $testing_resources_dir_tmp");
- testing_resources_dir = testing_resources_dir_tmp;
- if (!new Directory.fromPath(testing_resources_dir).existsSync()) {
- DebugLogger.error("$testing_resources_dir_tmp doesn't exist either. "
- " This is a fatal error. Exiting now.");
- exit(1);
- }
+ DebugLogger.error("$testing_resources_dir doesn't exist. Exiting now.");
+ exit(1);
}
var chromeAPK = testing_resources_dir.append('com.android.chrome-1.apk');
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698