Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(113)

Side by Side Diff: content/browser/browser_plugin/browser_plugin_guest.cc

Issue 2655463015: Correctly set dragLeave and dragEnd coords for OOPIF drag and drop (Closed)
Patch Set: Added checks for null RWH on drag end Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/browser_plugin/browser_plugin_guest.h" 5 #include "content/browser/browser_plugin/browser_plugin_guest.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 10
(...skipping 847 matching lines...) Expand 10 before | Expand all | Expand 10 after
858 // coming from the guest. 858 // coming from the guest.
859 if (!embedder->DragEnteredGuest(this)) 859 if (!embedder->DragEnteredGuest(this))
860 ignore_dragged_url_ = false; 860 ignore_dragged_url_ = false;
861 break; 861 break;
862 case blink::WebDragStatusOver: 862 case blink::WebDragStatusOver:
863 widget->DragTargetDragOver(location, location, mask, 863 widget->DragTargetDragOver(location, location, mask,
864 drop_data.key_modifiers); 864 drop_data.key_modifiers);
865 break; 865 break;
866 case blink::WebDragStatusLeave: 866 case blink::WebDragStatusLeave:
867 embedder->DragLeftGuest(this); 867 embedder->DragLeftGuest(this);
868 widget->DragTargetDragLeave(); 868 widget->DragTargetDragLeave(gfx::Point(), gfx::Point());
869 ignore_dragged_url_ = true; 869 ignore_dragged_url_ = true;
870 break; 870 break;
871 case blink::WebDragStatusDrop: 871 case blink::WebDragStatusDrop:
872 widget->DragTargetDrop(filtered_data, location, location, 872 widget->DragTargetDrop(filtered_data, location, location,
873 drop_data.key_modifiers); 873 drop_data.key_modifiers);
874 874
875 if (!ignore_dragged_url_ && filtered_data.url.is_valid()) 875 if (!ignore_dragged_url_ && filtered_data.url.is_valid())
876 delegate_->DidDropLink(filtered_data.url); 876 delegate_->DidDropLink(filtered_data.url);
877 ignore_dragged_url_ = true; 877 ignore_dragged_url_ = true;
878 break; 878 break;
(...skipping 186 matching lines...) Expand 10 before | Expand all | Expand 10 after
1065 range, character_bounds); 1065 range, character_bounds);
1066 } 1066 }
1067 #endif 1067 #endif
1068 1068
1069 void BrowserPluginGuest::SetContextMenuPosition(const gfx::Point& position) { 1069 void BrowserPluginGuest::SetContextMenuPosition(const gfx::Point& position) {
1070 if (delegate_) 1070 if (delegate_)
1071 delegate_->SetContextMenuPosition(position); 1071 delegate_->SetContextMenuPosition(position);
1072 } 1072 }
1073 1073
1074 } // namespace content 1074 } // namespace content
OLDNEW
« no previous file with comments | « components/test_runner/event_sender.cc ('k') | content/browser/renderer_host/render_widget_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698