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

Side by Side Diff: Source/devtools/front_end/AdvancedSearchView.js

Issue 218613013: DevTools: Decouple shortcuts from actions, introduce shortcut contexts (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased patch Created 6 years, 8 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 | Annotate | Revision Log
« no previous file with comments | « Source/devtools/front_end/ActionRegistry.js ('k') | Source/devtools/front_end/Context.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 /** 5 /**
6 * @constructor 6 * @constructor
7 * @extends {WebInspector.VBox} 7 * @extends {WebInspector.VBox}
8 */ 8 */
9 WebInspector.AdvancedSearchView = function() 9 WebInspector.AdvancedSearchView = function()
10 { 10 {
(...skipping 344 matching lines...) Expand 10 before | Expand all | Expand 10 after
355 /** 355 /**
356 * @constructor 356 * @constructor
357 * @implements {WebInspector.ActionDelegate} 357 * @implements {WebInspector.ActionDelegate}
358 */ 358 */
359 WebInspector.AdvancedSearchView.ToggleDrawerViewActionDelegate = function() 359 WebInspector.AdvancedSearchView.ToggleDrawerViewActionDelegate = function()
360 { 360 {
361 } 361 }
362 362
363 WebInspector.AdvancedSearchView.ToggleDrawerViewActionDelegate.prototype = { 363 WebInspector.AdvancedSearchView.ToggleDrawerViewActionDelegate.prototype = {
364 /** 364 /**
365 * @param {!Event} event
366 * @return {boolean} 365 * @return {boolean}
367 */ 366 */
368 handleAction: function(event) 367 handleAction: function()
369 { 368 {
370 if (WebInspector.Dialog.currentInstance())
371 return false;
372 var searchView = this._searchView(); 369 var searchView = this._searchView();
373 if (!searchView) 370 if (!searchView)
374 return false; 371 return false;
375 if (!searchView.isShowing() || searchView._search !== document.activeEle ment) { 372 if (!searchView.isShowing() || searchView._search !== document.activeEle ment) {
376 WebInspector.inspectorView.showPanel("sources"); 373 WebInspector.inspectorView.showPanel("sources");
377 searchView.toggle(); 374 searchView.toggle();
378 } else { 375 } else {
379 WebInspector.inspectorView.closeDrawer(); 376 WebInspector.inspectorView.closeDrawer();
380 } 377 }
381 return true; 378 return true;
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
437 /** 434 /**
438 * @param {!WebInspector.ProjectSearchConfig} searchConfig 435 * @param {!WebInspector.ProjectSearchConfig} searchConfig
439 * @return {!WebInspector.SearchResultsPane} 436 * @return {!WebInspector.SearchResultsPane}
440 */ 437 */
441 createSearchResultsPane: function(searchConfig) { } 438 createSearchResultsPane: function(searchConfig) { }
442 } 439 }
443 440
444 importScript("SearchConfig.js"); 441 importScript("SearchConfig.js");
445 importScript("FileBasedSearchResultsPane.js"); 442 importScript("FileBasedSearchResultsPane.js");
446 importScript("SourcesSearchScope.js"); 443 importScript("SourcesSearchScope.js");
OLDNEW
« no previous file with comments | « Source/devtools/front_end/ActionRegistry.js ('k') | Source/devtools/front_end/Context.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698