diff options
Diffstat (limited to 'chromium/content/browser/web_contents')
3 files changed, 23 insertions, 12 deletions
diff --git a/chromium/content/browser/web_contents/web_contents_impl_browsertest.cc b/chromium/content/browser/web_contents/web_contents_impl_browsertest.cc index 860e005e623..9e4467db30f 100644 --- a/chromium/content/browser/web_contents/web_contents_impl_browsertest.cc +++ b/chromium/content/browser/web_contents/web_contents_impl_browsertest.cc @@ -4877,8 +4877,15 @@ class DidChangeVerticalScrollDirectionObserver : public WebContentsObserver { // Tests that DidChangeVerticalScrollDirection is called only when the vertical // scroll direction has changed and that it includes the correct details. +// TODO(crbug.com/1359225): This is flaky on the Mac10.14 bot. +#if BUILDFLAG(IS_MAC) +#define MAYBE_DidChangeVerticalScrollDirection \ + DISABLED_DidChangeVerticalScrollDirection +#else +#define MAYBE_DidChangeVerticalScrollDirection DidChangeVerticalScrollDirection +#endif IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest, - DidChangeVerticalScrollDirection) { + MAYBE_DidChangeVerticalScrollDirection) { net::EmbeddedTestServer* server = embedded_test_server(); EXPECT_TRUE(server->Start()); diff --git a/chromium/content/browser/web_contents/web_contents_view_aura.cc b/chromium/content/browser/web_contents/web_contents_view_aura.cc index 7a7f170f3cd..01e059f5686 100644 --- a/chromium/content/browser/web_contents/web_contents_view_aura.cc +++ b/chromium/content/browser/web_contents/web_contents_view_aura.cc @@ -365,6 +365,7 @@ aura::Window* GetHostWindow(aura::Window* window) { WebContentsViewAura::DropMetadata::DropMetadata( const ui::DropTargetEvent& event) { localized_location = event.location_f(); + root_location = event.root_location_f(); source_operations = event.source_operations(); flags = event.flags(); } @@ -1469,7 +1470,7 @@ void WebContentsViewAura::OnDragEntered(const ui::DropTargetEvent& event) { } void WebContentsViewAura::DragUpdatedCallback( - ui::DropTargetEvent event, + DropMetadata drop_metadata, std::unique_ptr<DropData> drop_data, base::WeakPtr<RenderWidgetHostViewBase> target, absl::optional<gfx::PointF> transformed_pt) { @@ -1490,24 +1491,23 @@ void WebContentsViewAura::DragUpdatedCallback( aura::Window* root_window = GetNativeView()->GetRootWindow(); aura::client::ScreenPositionClient* screen_position_client = aura::client::GetScreenPositionClient(root_window); - gfx::PointF screen_pt = event.root_location_f(); + gfx::PointF screen_pt = drop_metadata.root_location; if (screen_position_client) screen_position_client->ConvertPointToScreen(root_window, &screen_pt); if (target_rwh != current_rwh_for_drag_.get()) { if (current_rwh_for_drag_) { - gfx::PointF transformed_leave_point = event.location_f(); + gfx::PointF transformed_leave_point = drop_metadata.localized_location; static_cast<RenderWidgetHostViewBase*>( web_contents_->GetRenderWidgetHostView()) ->TransformPointToCoordSpaceForView( - event.location_f(), + drop_metadata.localized_location, static_cast<RenderWidgetHostViewBase*>( current_rwh_for_drag_->GetView()), &transformed_leave_point); current_rwh_for_drag_->DragTargetDragLeave(transformed_leave_point, screen_pt); } - DropMetadata drop_metadata(event); DragEnteredCallback(drop_metadata, std::move(drop_data), target, transformed_pt); } @@ -1518,10 +1518,11 @@ void WebContentsViewAura::DragUpdatedCallback( DCHECK(transformed_pt.has_value()); blink::DragOperationsMask op_mask = - ConvertToDragOperationsMask(event.source_operations()); + ConvertToDragOperationsMask(drop_metadata.source_operations); target_rwh->DragTargetDragOver( transformed_pt.value(), screen_pt, op_mask, - ui::EventFlagsToWebEventModifiers(event.flags()), base::DoNothing()); + ui::EventFlagsToWebEventModifiers(drop_metadata.flags), + base::DoNothing()); if (drag_dest_delegate_) drag_dest_delegate_->OnDragOver(); @@ -1531,7 +1532,6 @@ aura::client::DragUpdateInfo WebContentsViewAura::OnDragUpdated( const ui::DropTargetEvent& event) { if (web_contents_->ShouldIgnoreInputEvents()) return aura::client::DragUpdateInfo(); - aura::client::DragUpdateInfo drag_info; auto* focused_frame = web_contents_->GetFocusedFrame(); if (focused_frame && !web_contents_->GetBrowserContext()->IsOffTheRecord()) { @@ -1542,13 +1542,13 @@ aura::client::DragUpdateInfo WebContentsViewAura::OnDragUpdated( std::unique_ptr<DropData> drop_data = std::make_unique<DropData>(); // Calling this here as event.data might become invalid inside the callback. PrepareDropData(drop_data.get(), event.data()); - + DropMetadata drop_metadata(event); web_contents_->GetInputEventRouter() ->GetRenderWidgetHostAtPointAsynchronously( web_contents_->GetRenderViewHost()->GetWidget()->GetView(), event.location_f(), base::BindOnce(&WebContentsViewAura::DragUpdatedCallback, - weak_ptr_factory_.GetWeakPtr(), event, + weak_ptr_factory_.GetWeakPtr(), drop_metadata, std::move(drop_data))); drag_info.drag_operation = static_cast<int>(current_drag_op_); diff --git a/chromium/content/browser/web_contents/web_contents_view_aura.h b/chromium/content/browser/web_contents/web_contents_view_aura.h index 3425436e8c1..45d6a8691e9 100644 --- a/chromium/content/browser/web_contents/web_contents_view_aura.h +++ b/chromium/content/browser/web_contents/web_contents_view_aura.h @@ -84,6 +84,10 @@ class CONTENT_EXPORT WebContentsViewAura // Location local to WebContentsViewAura. gfx::PointF localized_location; + + // Root location of the drop target event. + gfx::PointF root_location; + // The supported DnD operation of the source. A bitmask of // ui::mojom::DragOperations. int source_operations; @@ -265,7 +269,7 @@ class CONTENT_EXPORT WebContentsViewAura std::unique_ptr<DropData> drop_data, base::WeakPtr<RenderWidgetHostViewBase> target, absl::optional<gfx::PointF> transformed_pt); - void DragUpdatedCallback(ui::DropTargetEvent event, + void DragUpdatedCallback(DropMetadata drop_metadata, std::unique_ptr<DropData> drop_data, base::WeakPtr<RenderWidgetHostViewBase> target, absl::optional<gfx::PointF> transformed_pt); |