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

Unified Diff: Source/WebCore/inspector/front-end/inspector.js

Issue 9968120: Merge 112992 - Web Inspector: "Pause on start" doesn't change checked state in workers panel (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/1084/
Patch Set: Created 8 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/WebCore/inspector/front-end/UIUtils.js ('k') | Source/WebCore/inspector/front-end/treeoutline.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/WebCore/inspector/front-end/inspector.js
===================================================================
--- Source/WebCore/inspector/front-end/inspector.js (revision 113151)
+++ Source/WebCore/inspector/front-end/inspector.js (working copy)
@@ -588,7 +588,7 @@
return;
// Prevent the link from navigating, since we don't do any navigation by following links normally.
- event.consume();
+ event.consume(true);
function followLink()
{
@@ -685,14 +685,14 @@
if (event.keyIdentifier === "F1" ||
(event.keyIdentifier === helpKey && event.shiftKey && (!WebInspector.isBeingEdited(event.target) || event.metaKey))) {
WebInspector.shortcutsScreen.show();
- event.consume();
+ event.consume(true);
return;
}
if (WebInspector.currentFocusElement() && WebInspector.currentFocusElement().handleKeyEvent) {
WebInspector.currentFocusElement().handleKeyEvent(event);
if (event.handled) {
- event.consume();
+ event.consume(true);
return;
}
}
@@ -700,7 +700,7 @@
if (WebInspector.inspectorView.currentPanel()) {
WebInspector.inspectorView.currentPanel().handleShortcut(event);
if (event.handled) {
- event.consume();
+ event.consume(true);
return;
}
}
@@ -708,7 +708,7 @@
WebInspector.searchController.handleShortcut(event);
WebInspector.advancedSearchController.handleShortcut(event);
if (event.handled) {
- event.consume();
+ event.consume(true);
return;
}
@@ -717,13 +717,13 @@
case "U+0052": // R key
if ((event.metaKey && isMac) || (event.ctrlKey && !isMac)) {
PageAgent.reload(event.shiftKey);
- event.consume();
+ event.consume(true);
}
break;
case "F5":
if (!isMac) {
PageAgent.reload(event.ctrlKey || event.shiftKey);
- event.consume();
+ event.consume(true);
}
break;
}
@@ -737,20 +737,20 @@
case 187: // +
if (isValidZoomShortcut) {
WebInspector._zoomIn();
- event.consume();
+ event.consume(true);
}
break;
case 109: // -
case 189: // -
if (isValidZoomShortcut) {
WebInspector._zoomOut();
- event.consume();
+ event.consume(true);
}
break;
case 48: // 0
if (isValidZoomShortcut) {
WebInspector._resetZoom();
- event.consume();
+ event.consume(true);
}
break;
}
« no previous file with comments | « Source/WebCore/inspector/front-end/UIUtils.js ('k') | Source/WebCore/inspector/front-end/treeoutline.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698