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

Unified Diff: third_party/WebKit/Source/devtools/front_end/ui/ContextMenu.js

Issue 2519213002: [DevTools] Remove preventFollow and special checks for links throughout frontend. (Closed)
Patch Set: Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/ui/ContextMenu.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/ContextMenu.js b/third_party/WebKit/Source/devtools/front_end/ui/ContextMenu.js
index 09a391db73d6ddd57f584ad7a78ce36521f8db89..22ad486932dcb0123364624bb731182cbbee2ac5 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/ContextMenu.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/ContextMenu.js
@@ -298,6 +298,10 @@ UI.ContextMenu = class extends UI.ContextSubMenuItem {
this._id = 0;
/** @type {!Map<string, !UI.ContextSubMenuItem>} */
this._namedSubMenus = new Map();
+
+ var target = event.deepElementFromPoint();
+ if (target)
+ this.appendApplicableItems(/** @type {!Object} */ (target));
lushnikov 2016/11/22 01:11:49 won't this result in double-applicable-items? You
dgozman 2016/11/22 01:33:19 I did actually remove them from installHandler cal
}
static initialize() {
@@ -321,7 +325,6 @@ UI.ContextMenu = class extends UI.ContextSubMenuItem {
*/
function handler(event) {
var contextMenu = new UI.ContextMenu(event);
- contextMenu.appendApplicableItems(/** @type {!Object} */ (event.deepElementFromPoint()));
contextMenu.show();
}
}

Powered by Google App Engine
This is Rietveld 408576698