Fix wrong usages of ScopedJavaLocalRef::Release().

Fix wrong usages of ScopedJavaLocalRef::Release().

Release() is used for transfer of ownership and indicates that the
caller will take responsibility for deleting the local reference when
it's done with it. This is not the right thing to use when calling Java
functions, which do *not* take ownership of the local references you
pass to them.

Also clean up passing bare jobjects around in content shell; it's saner
to always use ScopedJavaLocalRef.

BUG=522014
Committed: https://crrev.com/b105fe36022d7664683839e0c961b770b2092554
Cr-Commit-Position: refs/heads/master@{#345903}

[Torne, 2015-08-18 12:18:51]

torne@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[Torne, 2015-08-18 12:18:51]

With this and my other CL to handle return types combined, all calls to
Release() in the chromium codebase are eliminated with the exception of
content/browser/android/java/gin_java_script_to_java_types_coercion.cc which is
doing Weird Complicated Things. I haven't actually verified those but they seem
plausible, will check with mikhail.

[Torne, 2015-08-18 12:29:11]

torne@chromium.org changed reviewers:
+ dcheng@chromium.org, dmazzoni@chromium.org, fgorski@chromium.org,
mef@chromium.org, tedchoc@chromium.org

[Torne, 2015-08-18 12:29:11]

fgorski:
  components/gcm_driver/gcm_driver_android.cc
dcheng:
  ui/base/clipboard/clipboard_android.cc
tedchoc:
  chrome/browser/android/history_report/history_report_jni_bridge.cc
  chrome/browser/android/recently_closed_tabs_bridge.cc
  components/web_contents_delegate_android/web_contents_delegate_android.cc
  content/shell/android/shell_manager.cc
  content/shell/android/shell_manager.h
  content/shell/browser/shell_android.cc
dmazzoni:
  chrome/browser/speech/tts_android.cc
rmcilroy:
  base/android/java_handler_thread.cc
  base/android/scoped_java_ref.h
mef:
  components/cronet/android/test/test_upload_data_stream_handler.cc

[Mike West, 2015-08-18 13:29:55]

Drive-by LGTM for content/shell.

[rmcilroy, 2015-08-18 13:46:24]

base/android LGTM.

[fgorski, 2015-08-18 15:59:42]

components/gcm_driver lgtm

[Ted C, 2015-08-21 17:11:30]

On 2015/08/18 15:59:42, fgorski wrote:
> components/gcm_driver lgtm

lgtm

[dmazzoni, 2015-08-24 19:06:17]

chrome/browser/speech LGTM

[mef, 2015-08-24 19:32:28]

components/cronet lgtm

[Torne, 2015-08-25 10:01:42]

dcheng: ping

[Torne, 2015-08-27 12:04:33]

torne@chromium.org changed reviewers:
+ sky@chromium.org

[Torne, 2015-08-27 12:04:34]

sky, can I get a review for ui/ ?

[sky, 2015-08-27 16:50:22]

LGTM

[Torne, 2015-08-27 16:51:16]

The CQ bit was checked by torne@chromium.org

[commit-bot: I haz the power, 2015-08-27 16:52:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1294333002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1294333002/20001

[commit-bot: I haz the power, 2015-08-27 16:57:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-27 16:58:34]

Patchset 2 (id:??) landed as
https://crrev.com/b105fe36022d7664683839e0c961b770b2092554
Cr-Commit-Position: refs/heads/master@{#345903}