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

Unified Diff: ui/webui/resources/js/cr/ui/menu_button.js

Issue 2597013002: Run clang-format on ui/webui/resources (Closed)
Patch Set: remove cr_shared_menu.js Created 4 years 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: ui/webui/resources/js/cr/ui/menu_button.js
diff --git a/ui/webui/resources/js/cr/ui/menu_button.js b/ui/webui/resources/js/cr/ui/menu_button.js
index ff6c256045cfd448f91193e8254ffb2992c97d5b..a42aa3e28b3057fc7217acbc68f4eeb9d39b9a1b 100644
--- a/ui/webui/resources/js/cr/ui/menu_button.js
+++ b/ui/webui/resources/js/cr/ui/menu_button.js
@@ -67,9 +67,7 @@ cr.define('cr.ui', function() {
* The menu associated with the menu button.
* @type {cr.ui.Menu}
*/
- get menu() {
- return this.menu_;
- },
+ get menu() { return this.menu_; },
set menu(menu) {
if (typeof menu == 'string' && menu[0] == '#') {
menu = assert(this.ownerDocument.getElementById(menu.slice(1)));
@@ -143,8 +141,8 @@ cr.define('cr.ui', function() {
}
break;
case 'activate':
- var hideDelayed = e.target instanceof cr.ui.MenuItem &&
- e.target.checkable;
+ var hideDelayed =
+ e.target instanceof cr.ui.MenuItem && e.target.checkable;
this.hideMenu(hideDelayed ? HideType.DELAYED : HideType.INSTANT);
break;
case 'scroll':
@@ -183,11 +181,8 @@ cr.define('cr.ui', function() {
this.menu.updateCommands(this);
- var event = new UIEvent('menushow',{
- bubbles: true,
- cancelable: true,
- view: window
- });
+ var event = new UIEvent(
+ 'menushow', {bubbles: true, cancelable: true, view: window});
if (!this.dispatchEvent(event))
return;
@@ -232,11 +227,8 @@ cr.define('cr.ui', function() {
this.showingEvents_.removeAll();
this.focus();
- var event = new UIEvent('menuhide', {
- bubbles: true,
- cancelable: false,
- view: window
- });
+ var event = new UIEvent(
+ 'menuhide', {bubbles: true, cancelable: false, view: window});
this.dispatchEvent(event);
// On windows we might hide the menu in a right mouse button up and if
@@ -248,9 +240,7 @@ cr.define('cr.ui', function() {
/**
* Whether the menu is shown.
*/
- isMenuShown: function() {
- return this.hasAttribute('menu-shown');
- },
+ isMenuShown: function() { return this.hasAttribute('menu-shown'); },
/**
* Positions the menu below the menu button. At this point we do not use any
@@ -258,8 +248,8 @@ cr.define('cr.ui', function() {
* @private
*/
positionMenu_: function() {
- positionPopupAroundElement(this, this.menu, this.anchorType,
- this.invertLeftRight);
+ positionPopupAroundElement(
+ this, this.menu, this.anchorType, this.invertLeftRight);
},
/**

Powered by Google App Engine
This is Rietveld 408576698