Correctly set dragLeave and dragEnd coords for OOPIF drag and drop

chrome/browser/ui/views/drag_and_drop_interactive_uitest.cc

 // Copyright 2016 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 <algorithm>
 #include <initializer_list>
 #include <memory>
 #include <string>
 #include <vector>
 
@@ skipping 916 matching lines @@
   {
     DOMDragEventWaiter dragleave_event_waiter("dragleave", left_frame());
     DOMDragEventWaiter dragenter_event_waiter("dragenter", right_frame());
     state->left_frame_events_counter->Reset();
     state->right_frame_events_counter->Reset();
     ASSERT_TRUE(SimulateMouseMoveToRightFrame());
 
     {  // Verify dragleave DOM event.
       std::string dragleave_event;
 
-      // TODO(paulmeyer): https://crbug.com/669695: Need to unify coordinates
-      // passed to dragend when OOPIFs are present or not.
-      state->expected_dom_event_data.set_expected_client_position(
-          "<no expectation>");
-      state->expected_dom_event_data.set_expected_page_position(
-          "<no expectation>");
+      state->expected_dom_event_data.set_expected_client_position("(355, 150)");
+      state->expected_dom_event_data.set_expected_page_position("(355, 150)");
 
       EXPECT_TRUE(
           dragleave_event_waiter.WaitForNextMatchingEvent(&dragleave_event));
       EXPECT_THAT(dragleave_event, state->expected_dom_event_data.Matches());
     }
 
     {  // Verify dragenter DOM event.
       std::string dragenter_event;
 
       // Update expected event coordinates after SimulateMouseMoveToRightFrame
@@ skipping 65 matching lines @@
     EXPECT_THAT(drop_event, state->expected_dom_event_data.Matches());
   }
 
   // Verify dragend DOM event.
   {
     // TODO(lukasza): Figure out why the drop event sees different values of
     // DataTransfer.dropEffect and DataTransfer.types properties.
     state->expected_dom_event_data.set_expected_drop_effect("copy");
     state->expected_dom_event_data.set_expected_mime_types("");
 
-    // TODO(paulmeyer): https://crbug.com/669695: Need to unify coordinates
-    // passed to dragend when OOPIFs are present or not.
+    // TODO: https://crbug.com/686136: dragEnd coordinates for non-OOPIF
+    // scenarios are currently broken.
     state->expected_dom_event_data.set_expected_client_position(
         "<no expectation>");
     state->expected_dom_event_data.set_expected_page_position(
         "<no expectation>");
 
     std::string dragend_event;
     EXPECT_TRUE(
         state->dragend_event_waiter->WaitForNextMatchingEvent(&dragend_event));
     state->dragend_event_waiter.reset();
     EXPECT_THAT(dragend_event, state->expected_dom_event_data.Matches());
@@ skipping 244 matching lines @@
 // of a drag operation, and cross-site drags should be allowed across a
 // navigation.
 
 INSTANTIATE_TEST_CASE_P(
     SameSiteSubframe, DragAndDropBrowserTest, ::testing::Values(false));
 
 INSTANTIATE_TEST_CASE_P(
     CrossSiteSubframe, DragAndDropBrowserTest, ::testing::Values(true));
 
 }  // namespace chrome