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

Unified Diff: base/test/android/javatests/src/org/chromium/base/test/util/UrlUtils.java

Issue 2854823007: Move screenshot capture to Java-side. (Closed)
Patch Set: Move screenshot capture to Java-side. Created 3 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: base/test/android/javatests/src/org/chromium/base/test/util/UrlUtils.java
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 7433d8e0481076edab908f2e3ed3af4f9f3d9508..664eaa961acf45492efabce7a05326384b606032 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
@@ -41,6 +41,21 @@ public class UrlUtils {
}
/**
+ * Returns the root of the test data output directory.
+ */
+ public static String getTestFileOutputRoot() {
+ return PathUtils.getExternalStorageDirectory() + "/chromium_tests_output_root";
+ }
+
+ /**
+ * Construct the full path of a test data output file.
+ * @param path Pathname relative to external/
+ */
+ public static String getTestFileOutputPath(String path) {
+ return getTestFileOutputRoot() + "/" + path;
jbudorick 2017/05/08 17:08:39 Should this check that path doesn't start with "/"
mikecase (-- gone --) 2017/05/08 18:35:31 This is all gone now. So...fixed
+ }
+
+ /**
* Construct a suitable URL for loading a test data file.
* @param path Pathname relative to external/chrome/test/data
*/

Powered by Google App Engine
This is Rietveld 408576698