diff options
author | jbudorick <jbudorick@chromium.org> | 2015-02-11 13:54:59 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-02-11 21:55:59 +0000 |
commit | ca44323780b6b34889c13dfe78aa105825d8f3c0 (patch) | |
tree | 367d682dd9f8073e713f7774a39441352ba9ce5a /base/test | |
parent | 5f96ac2fd3ff36501431ac9c284dba9d6ffce303 (diff) | |
download | chromium_src-ca44323780b6b34889c13dfe78aa105825d8f3c0.zip chromium_src-ca44323780b6b34889c13dfe78aa105825d8f3c0.tar.gz chromium_src-ca44323780b6b34889c13dfe78aa105825d8f3c0.tar.bz2 |
[Android] Reland: Convert ContentShellTest.apk to isolate.
This is a reland of https://codereview.chromium.org/878163004/.
https://codereview.chromium.org/902053006/ passes isolate files to the test
runner for bots that have not been converted to recipes.
BUG=400499
TBR=maruel@chromium.org
Review URL: https://codereview.chromium.org/913513002
Cr-Commit-Position: refs/heads/master@{#315843}
Diffstat (limited to 'base/test')
-rw-r--r-- | base/test/android/javatests/src/org/chromium/base/test/util/UrlUtils.java | 28 |
1 files changed, 24 insertions, 4 deletions
diff --git a/base/test/android/javatests/src/org/chromium/base/test/util/UrlUtils.java b/base/test/android/javatests/src/org/chromium/base/test/util/UrlUtils.java index 09e1fd6..797585f 100644 --- a/base/test/android/javatests/src/org/chromium/base/test/util/UrlUtils.java +++ b/base/test/android/javatests/src/org/chromium/base/test/util/UrlUtils.java @@ -16,20 +16,41 @@ public class UrlUtils { /** * Construct the full path of a test data file. - * @param path Pathname relative to external/chrome/testing/data + * @param path Pathname relative to external/chrome/test/data */ public static String getTestFilePath(String path) { + // TODO(jbudorick): Remove DATA_DIR once everything has been isolated. crbug/400499 return PathUtils.getExternalStorageDirectory() + DATA_DIR + path; } + // TODO(jbudorick): Remove this function once everything has been isolated and switched back + // to getTestFilePath. crbug/400499 + /** + * Construct the full path of a test data file. + * @param path Pathname relative to external/ + */ + public static String getIsolatedTestFilePath(String path) { + return PathUtils.getExternalStorageDirectory() + "/" + path; + } + /** * Construct a suitable URL for loading a test data file. - * @param path Pathname relative to external/chrome/testing/data + * @param path Pathname relative to external/chrome/test/data */ public static String getTestFileUrl(String path) { return "file://" + getTestFilePath(path); } + // TODO(jbudorick): Remove this function once everything has been isolated and switched back + // to getTestFileUrl. crbug/400499 + /** + * Construct a suitable URL for loading a test data file. + * @param path Pathname relative to external/ + */ + public static String getIsolatedTestFileUrl(String path) { + return "file://" + getIsolatedTestFilePath(path); + } + /** * Construct a data:text/html URI for loading from an inline HTML. * @param html An unencoded HTML @@ -41,8 +62,7 @@ public class UrlUtils { // ' '->'+' needs to be undone and replaced with ' '->'%20' // to match the Data URI requirements. String encoded = - "data:text/html;utf-8," + - java.net.URLEncoder.encode(html, "UTF-8"); + "data:text/html;utf-8," + java.net.URLEncoder.encode(html, "UTF-8"); encoded = encoded.replace("+", "%20"); return encoded; } catch (java.io.UnsupportedEncodingException e) { |