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

Review URL: https://codereview.chromium.org/1294333002

Cr-Commit-Position: refs/heads/master@{#345903}


CrOS-Libchrome-Original-Commit: b105fe36022d7664683839e0c961b770b2092554
2 files changed
tree: d2a085666e34cc86f635b7d7f891282b2bf142ba
  1. base/
  2. build/
  3. components/
  4. dbus/
  5. device/
  6. ipc/
  7. mojo/
  8. testing/
  9. third_party/
  10. ui/