Position autofill popup widgets correctly under --site-per-process

components/autofill/core/browser/autofill_driver.h

Index: components/autofill/core/browser/autofill_driver.h
diff --git a/components/autofill/core/browser/autofill_driver.h b/components/autofill/core/browser/autofill_driver.h
index 82e0f1244a621b6d7d72a1b94e1a1e7ae66d90f8..3b2bd351406ecef014b577925752af0ece2dfd45 100644
--- a/components/autofill/core/browser/autofill_driver.h
+++ b/components/autofill/core/browser/autofill_driver.h
@@ -17,6 +17,10 @@ namespace net {
 class URLRequestContextGetter;
 }
 
+namespace gfx {
+class RectF;
+}
+
 namespace autofill {
 
 class FormStructure;
@@ -89,6 +93,9 @@ class AutofillDriver {
 
   // Informs the renderer that the popup has been hidden.
   virtual void PopupHidden() = 0;
+
+  // Transform bounding box screen coordinates to correct coordinate space.
+  virtual gfx::RectF TransformBoundingBox(const gfx::RectF& bounding_box) = 0;

[Ilya Sherman, 2016/01/20 23:12:15]

I think it might be nice to name this somehow so that it more clearly indicates
what sort of transformation is happening, i.e. what coordinate space is being
transformed from and to.  Perhaps naming it "TransformToScreenCoordinates" would
be clearer?  I'm not really sure what sorts of conventions there are for this
sort of thing, but I'm sure there are some.

[kenrb, 2016/01/22 18:33:58]

I think the new name is accurate.

 };
 
 }  // namespace autofill