Update viewport_to_dip_scale on bound compositor host

content/renderer/pepper/pepper_plugin_instance_impl.cc

Index: content/renderer/pepper/pepper_plugin_instance_impl.cc
diff --git a/content/renderer/pepper/pepper_plugin_instance_impl.cc b/content/renderer/pepper/pepper_plugin_instance_impl.cc
index 5607f6cc418db84c4705234bfe0c6f47c17f7dbf..c72ea8d8987d32c2f7266807be8ad36ffe5849b4 100644
--- a/content/renderer/pepper/pepper_plugin_instance_impl.cc
+++ b/content/renderer/pepper/pepper_plugin_instance_impl.cc
@@ -2351,12 +2351,15 @@ PP_Bool PepperPluginInstanceImpl::BindGraphics(PP_Instance instance,
   if (compositor) {
     if (compositor->BindToInstance(this)) {

[oshima, 2016/08/23 22:25:49]

+bbudge@ (owner)

I feel it's better to do this in BindToInstance, but I'd follow the owner's
recommendation.

[bbudge, 2016/08/23 23:32:39]

I might not understand the comment, but this seems like the only way, since the
compositor might not have a viewport_to_dip_scale_ value set. Also, this class
makes another call to set_viewport_to_dip_scale when handling view changes:

https://cs.chromium.org/chromium/src/content/renderer/pepper/pepper_plugin_in...

       bound_compositor_ = compositor;
+      bound_compositor_->set_viewport_to_dip_scale(viewport_to_dip_scale_);
       UpdateLayer(true);
       return PP_TRUE;
     }
   } else if (graphics_2d) {
     if (graphics_2d->BindToInstance(this)) {
       bound_graphics_2d_platform_ = graphics_2d;
+      bound_graphics_2d_platform_->set_viewport_to_dip_scale(
+          viewport_to_dip_scale_);
       UpdateLayer(true);
       return PP_TRUE;
     }