diff options
author | rvargas@chromium.org <rvargas@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-21 00:41:15 +0000 |
---|---|---|
committer | rvargas@chromium.org <rvargas@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-21 00:41:15 +0000 |
commit | a0830591020c23ad911aa287390a4ae942c5dc41 (patch) | |
tree | 650128541ee17d686b45cb0b163db4e38522f015 /base/android | |
parent | dd1dad31497696ef81234339e6e3cf6f5af26eea (diff) | |
download | chromium_src-a0830591020c23ad911aa287390a4ae942c5dc41.zip chromium_src-a0830591020c23ad911aa287390a4ae942c5dc41.tar.gz chromium_src-a0830591020c23ad911aa287390a4ae942c5dc41.tar.bz2 |
net: Update FileStream to use base::File instead of PlatformFile.
As collateral damage, OpenContentUriForRead is now returning File instead of
a plain file descriptor.
BUG=322664
TEST=net_unittests
R=willchan@chromium.org
Review URL: https://codereview.chromium.org/189393002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@258472 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/android')
-rw-r--r-- | base/android/content_uri_utils.cc | 7 | ||||
-rw-r--r-- | base/android/content_uri_utils.h | 3 |
2 files changed, 5 insertions, 5 deletions
diff --git a/base/android/content_uri_utils.cc b/base/android/content_uri_utils.cc index 64d6ad2..0e0c0ea 100644 --- a/base/android/content_uri_utils.cc +++ b/base/android/content_uri_utils.cc @@ -6,7 +6,6 @@ #include "base/android/jni_android.h" #include "base/android/jni_string.h" -#include "base/platform_file.h" #include "jni/ContentUriUtils_jni.h" using base::android::ConvertUTF8ToJavaString; @@ -25,15 +24,15 @@ bool ContentUriExists(const FilePath& content_uri) { env, base::android::GetApplicationContext(), j_uri.obj()); } -int OpenContentUriForRead(const FilePath& content_uri) { +File OpenContentUriForRead(const FilePath& content_uri) { JNIEnv* env = base::android::AttachCurrentThread(); ScopedJavaLocalRef<jstring> j_uri = ConvertUTF8ToJavaString(env, content_uri.value()); jint fd = Java_ContentUriUtils_openContentUriForRead( env, base::android::GetApplicationContext(), j_uri.obj()); if (fd < 0) - return base::kInvalidPlatformFileValue; - return fd; + return File(); + return File(fd); } } // namespace base diff --git a/base/android/content_uri_utils.h b/base/android/content_uri_utils.h index ec820ef..827ec92 100644 --- a/base/android/content_uri_utils.h +++ b/base/android/content_uri_utils.h @@ -9,6 +9,7 @@ #include "base/base_export.h" #include "base/basictypes.h" +#include "base/files/file.h" #include "base/files/file_path.h" namespace base { @@ -17,7 +18,7 @@ bool RegisterContentUriUtils(JNIEnv* env); // Opens a content uri for read and returns the file descriptor to the caller. // Returns -1 if the uri is invalid. -BASE_EXPORT int OpenContentUriForRead(const FilePath& content_uri); +BASE_EXPORT File OpenContentUriForRead(const FilePath& content_uri); // Check whether a content uri exists. BASE_EXPORT bool ContentUriExists(const FilePath& content_uri); |