Add observers for DIP scale change.

content/browser/android/content_view_core_impl.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/android/content_view_core_impl.h"
 
 #include <stddef.h>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
@@ skipping 866 matching lines @@
 WebContents* ContentViewCoreImpl::GetWebContents() const {
   return web_contents_;
 }
 
 void ContentViewCoreImpl::SetFocus(JNIEnv* env,
                                    const JavaParamRef<jobject>& obj,
                                    jboolean focused) {
   SetFocusInternal(focused);
 }
 
+void ContentViewCoreImpl::SetDIPScale(JNIEnv* env,
+                                      const JavaParamRef<jobject>& obj,
+                                      jfloat dipScale) {
+  dpi_scale_ = dipScale;

[boliu, 2016/09/14 04:35:32]

I remember making this comment before. What's responsible for updating all the
places where this is used?

[Tima Vaisburd, 2016/09/15 00:05:03]

I probably never understood completely, so let me ask a more detailed question:
are you talking about the member variables that store the result which includes
DIP scale and thus depend on it indirectly?

For instance, OnSelectionEvent() scales the selection rect up with DIP scale and
passes it to Java side where it is stored in mSelectionRect. mSelectionRect
should be recalculated unless the selection goes away during DIP scale changing
process.

This job is not done.

Better yet, as much as possible should be in DIP pixels.

[boliu, 2016/09/15 05:27:41]

Yep
Yeah, moving all that conversion to java would be one super big help. I guess
you can either leave that as a TODO in this CL, or maybe do that refactor first.

+}
+
 void ContentViewCoreImpl::SetFocusInternal(bool focused) {
   if (!GetRenderWidgetHostViewAndroid())
     return;
 
   if (focused)
     GetRenderWidgetHostViewAndroid()->Focus();
   else
     GetRenderWidgetHostViewAndroid()->Blur();
 }
 
@@ skipping 705 matching lines @@
     return ScopedJavaLocalRef<jobject>();
 
   return view->GetJavaObject();
 }
 
 bool RegisterContentViewCore(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace content