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

Unified Diff: components/cronet/android/test/src/org/chromium/net/TestFilesInstaller.java

Issue 2318493003: Clean up PathUtils unused parameters. (Closed)
Patch Set: Fix more. Created 4 years, 3 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 | « components/cronet/android/test/src/org/chromium/net/CronetTestFramework.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/android/test/src/org/chromium/net/TestFilesInstaller.java
diff --git a/components/cronet/android/test/src/org/chromium/net/TestFilesInstaller.java b/components/cronet/android/test/src/org/chromium/net/TestFilesInstaller.java
index 5fb5521b2870f211607930989e58ec5adb4a16a3..371f48a262c0fc1e7d6f969baaf9224a6e6d4fc5 100644
--- a/components/cronet/android/test/src/org/chromium/net/TestFilesInstaller.java
+++ b/components/cronet/android/test/src/org/chromium/net/TestFilesInstaller.java
@@ -42,7 +42,7 @@ public final class TestFilesInstaller {
* Returns the installed path of the test files.
*/
public static String getInstalledPath(Context context) {
- return PathUtils.getDataDirectory(context) + "/" + TEST_FILE_ASSET_PATH;
+ return PathUtils.getDataDirectory() + "/" + TEST_FILE_ASSET_PATH;
}
/**
@@ -64,7 +64,7 @@ public final class TestFilesInstaller {
AssetManager assetManager = context.getAssets();
String files[] = assetManager.list(path);
Log.i(TAG, "Loading " + path + " ...");
- String root = PathUtils.getDataDirectory(context);
+ String root = PathUtils.getDataDirectory();
if (files.length == 0) {
// The path is a file, so copy the file now.
copyTestFile(context, path, root + "/" + path);
« no previous file with comments | « components/cronet/android/test/src/org/chromium/net/CronetTestFramework.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698