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

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/panel_menu.js

Issue 2943193002: Run clang-format on .js files in c/b/r/chromeos/chromevox (Closed)
Patch Set: Created 3 years, 6 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
Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/panel_menu.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel_menu.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel_menu.js
index b4b3f5c44f31a33df3365fa13bdf247daf92ca0c..53b1cbdee359ab34b51e9f2830bc3d046a0ad473 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel_menu.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel_menu.js
@@ -51,12 +51,12 @@ PanelMenu = function(menuMsg) {
this.items_ = [];
/**
- * The return value from window.setTimeout for a function to update the
- * scroll bars after an item has been added to a menu. Used so that we
- * don't re-layout too many times.
- * @type {?number}
- * @private
- */
+ * The return value from window.setTimeout for a function to update the
+ * scroll bars after an item has been added to a menu. Used so that we
+ * don't re-layout too many times.
+ * @type {?number}
+ * @private
+ */
this.updateScrollbarsTimeout_ = null;
/**
@@ -85,20 +85,24 @@ PanelMenu.prototype = {
this.menuElement.appendChild(menuItem.element);
// Sync the active index with focus.
- menuItem.element.addEventListener('focus', (function(index, event) {
- this.activeIndex_ = index;
- }).bind(this, this.items_.length - 1), false);
+ menuItem.element.addEventListener(
+ 'focus', (function(index, event) {
+ this.activeIndex_ = index;
+ }).bind(this, this.items_.length - 1),
+ false);
// Update the container height, adding a scroll bar if necessary - but
// to avoid excessive layout, schedule this once per batch of adding
// menu items rather than after each add.
if (!this.updateScrollbarsTimeout_) {
- this.updateScrollbarsTimeout_ = window.setTimeout((function() {
- var menuBounds = this.menuElement.getBoundingClientRect();
- var maxHeight = window.innerHeight - menuBounds.top;
- this.menuContainerElement.style.maxHeight = maxHeight + 'px';
- this.updateScrollbarsTimeout_ = null;
- }).bind(this), 0);
+ this.updateScrollbarsTimeout_ = window.setTimeout(
+ (function() {
+ var menuBounds = this.menuElement.getBoundingClientRect();
+ var maxHeight = window.innerHeight - menuBounds.top;
+ this.menuContainerElement.style.maxHeight = maxHeight + 'px';
+ this.updateScrollbarsTimeout_ = null;
+ }).bind(this),
+ 0);
}
return menuItem;
@@ -139,9 +143,11 @@ PanelMenu.prototype = {
this.menuBarItemElement.classList.remove('active');
this.activeIndex_ = -1;
- window.setTimeout((function() {
- this.menuContainerElement.style.visibility = 'hidden';
- }).bind(this), 0);
+ window.setTimeout(
+ (function() {
+ this.menuContainerElement.style.visibility = 'hidden';
+ }).bind(this),
+ 0);
},
/**
@@ -221,8 +227,7 @@ PanelMenu.prototype = {
return;
var query = String.fromCharCode(evt.charCode).toLowerCase();
- for (var i = this.activeIndex_ + 1;
- i != this.activeIndex_;
+ for (var i = this.activeIndex_ + 1; i != this.activeIndex_;
i = (i + 1) % this.items_.length) {
if (this.items_[i].text.toLowerCase().indexOf(query) == 0) {
this.activateItem(i);
@@ -283,12 +288,11 @@ PanelNodeMenu.prototype = {
return;
}
- this.walker_ = new AutomationTreeWalker(
- root,
- constants.Dir.FORWARD,
- {visit: function(node) {
- return !AutomationPredicate.shouldIgnoreNode(node);
- }});
+ this.walker_ = new AutomationTreeWalker(root, constants.Dir.FORWARD, {
+ visit: function(node) {
+ return !AutomationPredicate.shouldIgnoreNode(node);
+ }
+ });
this.nodeCount_ = 0;
this.selectNext_ = false;
this.findMoreNodes_();
@@ -315,12 +319,13 @@ PanelNodeMenu.prototype = {
output.withSpeech(range, range, Output.EventType.NAVIGATE);
var label = output.toString();
this.addMenuItem(label, '', '', (function() {
- var savedNode = node;
- return function() {
- chrome.extension.getBackgroundPage().ChromeVoxState
- .instance['navigateToRange'](cursors.Range.fromNode(savedNode));
- };
- }()));
+ var savedNode = node;
+ return function() {
+ chrome.extension.getBackgroundPage()
+ .ChromeVoxState.instance['navigateToRange'](
+ cursors.Range.fromNode(savedNode));
+ };
+ }()));
if (this.selectNext_) {
this.activateItem(this.items_.length - 1);

Powered by Google App Engine
This is Rietveld 408576698