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

components/autofill/core/browser/test_autofill_driver.cc

Index: components/autofill/core/browser/test_autofill_driver.cc
diff --git a/components/autofill/core/browser/test_autofill_driver.cc b/components/autofill/core/browser/test_autofill_driver.cc
index 38a83ed7d1b9b2240d36e2a8d1990cfe4d922eb9..c0442bee082f67723ef585b2c3367aba0872e99c 100644
--- a/components/autofill/core/browser/test_autofill_driver.cc
+++ b/components/autofill/core/browser/test_autofill_driver.cc
@@ -6,6 +6,7 @@
 
 #include "base/test/sequenced_worker_pool_owner.h"
 #include "base/threading/sequenced_worker_pool.h"
+#include "ui/gfx/geometry/rect_f.h"
 
 namespace autofill {
 
@@ -71,4 +72,9 @@ void TestAutofillDriver::RendererShouldPreviewFieldWithValue(
 void TestAutofillDriver::PopupHidden() {
 }
 
+gfx::RectF TestAutofillDriver::TransformBoundingBox(

[Charlie Reis, 2016/01/21 00:33:03]

Is there any way to test the new API, either here or in content/?

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

The API is tested implicitly by SitePerProcessBrowserTest.CreateContextMenuTest,
which uses the TransformPointToRootCoordSpace in the same way as we use it here.
Context menus are easier to test this with because the browser can provide the
coordinates of the mouse click that generates the context menu (and then verify
that they have been converted back to the original coords), whereas for popups
the coordinates come from the page content and I have had trouble finding a way
to know what the correct values are.

[Charlie Reis, 2016/01/22 23:04:54]

Acknowledged.

+    const gfx::RectF& bounding_box) {
+  return bounding_box;
+}
+
 }  // namespace autofill