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

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/ui/UIUtils.js

Issue 2393763002: [DevTools] Cleanup DOMExtension.js. (Closed)
Patch Set: review comments Created 4 years, 2 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
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/ui/Tooltip.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2011 Google Inc. All rights reserved. 2 * Copyright (C) 2011 Google Inc. All rights reserved.
3 * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved. 3 * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
4 * Copyright (C) 2007 Matt Lilek (pewtermoose@gmail.com). 4 * Copyright (C) 2007 Matt Lilek (pewtermoose@gmail.com).
5 * Copyright (C) 2009 Joseph Pecoraro 5 * Copyright (C) 2009 Joseph Pecoraro
6 * 6 *
7 * Redistribution and use in source and binary forms, with or without 7 * Redistribution and use in source and binary forms, with or without
8 * modification, are permitted provided that the following conditions 8 * modification, are permitted provided that the following conditions
9 * are met: 9 * are met:
10 * 10 *
(...skipping 780 matching lines...) Expand 10 before | Expand all | Expand 10 after
791 { 791 {
792 if (event.target.document.nodeType === Node.DOCUMENT_NODE) 792 if (event.target.document.nodeType === Node.DOCUMENT_NODE)
793 document.body.classList.add("inactive"); 793 document.body.classList.add("inactive");
794 } 794 }
795 795
796 /** 796 /**
797 * @param {!Event} event 797 * @param {!Event} event
798 */ 798 */
799 WebInspector._focusChanged = function(event) 799 WebInspector._focusChanged = function(event)
800 { 800 {
801 var node = event.deepActiveElement(); 801 var document = event.target && event.target.ownerDocument;
802 WebInspector.Widget.focusWidgetForNode(node); 802 var element = document ? document.deepActiveElement() : null;
803 WebInspector.Widget.focusWidgetForNode(element);
803 } 804 }
804 805
805 /** 806 /**
806 * @param {!Element} element 807 * @param {!Element} element
807 * @constructor 808 * @constructor
808 */ 809 */
809 WebInspector.ElementFocusRestorer = function(element) 810 WebInspector.ElementFocusRestorer = function(element)
810 { 811 {
811 this._element = element; 812 this._element = element;
812 this._previous = element.ownerDocument.deepActiveElement(); 813 this._previous = element.ownerDocument.deepActiveElement();
(...skipping 1143 matching lines...) Expand 10 before | Expand all | Expand 10 after
1956 * @param {string} title 1957 * @param {string} title
1957 * @return {!Element} 1958 * @return {!Element}
1958 */ 1959 */
1959 WebInspector.linkifyDocumentationURLAsNode = function(article, title) 1960 WebInspector.linkifyDocumentationURLAsNode = function(article, title)
1960 { 1961 {
1961 return WebInspector.linkifyURLAsNode("https://developers.google.com/web/tool s/chrome-devtools/" + article, title, undefined, true); 1962 return WebInspector.linkifyURLAsNode("https://developers.google.com/web/tool s/chrome-devtools/" + article, title, undefined, true);
1962 } 1963 }
1963 1964
1964 /** @type {!WebInspector.ThemeSupport} */ 1965 /** @type {!WebInspector.ThemeSupport} */
1965 WebInspector.themeSupport; 1966 WebInspector.themeSupport;
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/ui/Tooltip.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698