diff options
| author | jdduke <jdduke@chromium.org> | 2015-01-09 17:55:21 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-01-10 01:57:13 +0000 |
| commit | 54720d1a7aac3e32164b5107b5e4037034aa98ed (patch) | |
| tree | ea987c9a4521da48925f20a3988cf2f1dc620458 /android_webview | |
| parent | 98f5d96fd700ada9cf0cfd629b5b5e308d151f50 (diff) | |
| download | chromium_src-54720d1a7aac3e32164b5107b5e4037034aa98ed.zip chromium_src-54720d1a7aac3e32164b5107b5e4037034aa98ed.tar.gz chromium_src-54720d1a7aac3e32164b5107b5e4037034aa98ed.tar.bz2 | |
[Android] Optimize JNI-related array copies
Prefer Get*ArrayRegion to Get*ArrayElements, avoiding potentially
redundant copies and JNI calls.
BUG=440566
Review URL: https://codereview.chromium.org/838783003
Cr-Commit-Position: refs/heads/master@{#310939}
Diffstat (limited to 'android_webview')
| -rw-r--r-- | android_webview/native/aw_assets.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/android_webview/native/aw_assets.cc b/android_webview/native/aw_assets.cc index 41f2270..ba734ae 100644 --- a/android_webview/native/aw_assets.cc +++ b/android_webview/native/aw_assets.cc @@ -23,7 +23,7 @@ bool OpenAsset(const std::string& filename, env, base::android::GetApplicationContext(), base::android::ConvertUTF8ToJavaString(env, filename).Release()); - std::vector<long> results; + std::vector<jlong> results; base::android::JavaLongArrayToLongVector(env, jarr.obj(), &results); DCHECK_EQ(3U, results.size()); *fd = static_cast<int>(results[0]); |