DevTools: Allow ToolbarMenuButtons to be trigged via keyboard

third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 727 matching lines @@
     this._contextMenuHandler(contextMenu);
     contextMenu.show();
     this._lastTriggerTime = Date.now();
   }
 
   /**
    * @override
    * @param {!Event} event
    */
   _clicked(event) {
-    if (!this._triggerTimeout)

[pfeldman, 2017/03/31 00:02:05]

What if there wasn't? At least clearTimeout conditionally?

[einbinder, 2017/04/03 05:21:26]

Done.

-      return;
     clearTimeout(this._triggerTimeout);
     this._trigger(event);
   }
 };
 
 /**
  * @unrestricted
  */
 UI.ToolbarSettingToggle = class extends UI.ToolbarToggle {
   /**
@@ skipping 294 matching lines @@
   /**
    * @param {!Common.Setting} setting
    * @param {string=} tooltip
    * @param {string=} alternateTitle
    */
   constructor(setting, tooltip, alternateTitle) {
     super(alternateTitle || setting.title() || '', tooltip);
     UI.SettingsUI.bindCheckbox(this.inputElement, setting);
   }
 };