Hold a reference to any kind of context in WindowAndroid.

ui/android/window_android.h

Index: ui/android/window_android.h
diff --git a/ui/android/window_android.h b/ui/android/window_android.h
index 58f1e280bd6a9639ac9368cce647ed9a1ef2c86b..e250eec7e08897de59173766ab12beba6a0705f1 100644
--- a/ui/android/window_android.h
+++ b/ui/android/window_android.h
@@ -69,6 +69,9 @@ class UI_ANDROID_EXPORT WindowAndroid {
   // Return whether the specified Android permission can be requested by Chrome.
   bool CanRequestPermission(const std::string& permission);
 
+  // This is public only for use from ResourceManager

[Ted C, 2015/10/29 22:39:19]

We specifically wanted to avoid exposing the context to anyone as it is
extremely easy to get the lifecycle wrong.

Despite having this comment, having this method here allows easy abuse and I
doubt it will stop anyone.

I think to keep it consistent with the ActivityWindowAndroid implementation, it
should return a JavaObjectWeakGlobalRef with very clear documentation that it
should never be held on to longer than the lifetime of the context (whatever
that may be).  For example, I know nothing of a Presentation context and do not
know if the resource manager lifecycle will match it exactly.

If it outlives it, then we're going to hold onto that and leak everything
attached to it.

[gsennton, 2015/11/06 17:36:13]

Done.

+  base::android::ScopedJavaLocalRef<jobject> GetJavaDisplayContext();
+
  private:
   ~WindowAndroid();