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

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

Issue 618133002: All changes to ui/webui/ from bookmarks CL that block other CLs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@true_master
Patch Set: add var Created 6 years, 3 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
« no previous file with comments | « ui/webui/resources/js/cr/ui/context_menu_handler.js ('k') | ui/webui/resources/js/cr/ui/splitter.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/webui/resources/js/cr/ui/list.js
diff --git a/ui/webui/resources/js/cr/ui/list.js b/ui/webui/resources/js/cr/ui/list.js
index 720f0704ee2d43203d0cc1f73242aaaca512882b..32d9b00065726c7fb4b7b86eb7fc48e9eba79326 100644
--- a/ui/webui/resources/js/cr/ui/list.js
+++ b/ui/webui/resources/js/cr/ui/list.js
@@ -155,7 +155,7 @@ cr.define('cr.ui', function() {
/**
* Cached item for measuring the default item size by measureItem().
- * @type {ListItem}
+ * @type {cr.ui.ListItem}
*/
cachedMeasuredItem_: null,
@@ -523,8 +523,7 @@ cr.define('cr.ui', function() {
* @private
*/
handleElementBlur_: function(e) {
- if (!this.contains(e.relatedTarget))
- this.hasElementFocus = false;
+ this.hasElementFocus = false;
},
/**
@@ -769,7 +768,7 @@ cr.define('cr.ui', function() {
* Takes a value from the data model and finds the associated list item.
* @param {*} value The value in the data model that we want to get the list
* item for.
- * @return {ListItem} The first found list item or null if not found.
+ * @return {cr.ui.ListItem} The first found list item or null if not found.
*/
getListItem: function(value) {
var dataModel = this.dataModel;
@@ -783,7 +782,7 @@ cr.define('cr.ui', function() {
/**
* Find the list item element at the given index.
* @param {number} index The index of the list item to get.
- * @return {ListItem} The found list item or null if not found.
+ * @return {cr.ui.ListItem} The found list item or null if not found.
*/
getListItemByIndex: function(index) {
return this.cachedItems_[index] || null;
@@ -791,7 +790,7 @@ cr.define('cr.ui', function() {
/**
* Find the index of the given list item element.
- * @param {ListItem} item The list item to get the index of.
+ * @param {HTMLLIElement} item The list item to get the index of.
* @return {number} The index of the list item, or -1 if not found.
*/
getIndexOfListItem: function(item) {
@@ -805,7 +804,7 @@ cr.define('cr.ui', function() {
/**
* Creates a new list item.
* @param {*} value The value to use for the item.
- * @return {!ListItem} The newly created list item.
+ * @return {!cr.ui.ListItem} The newly created list item.
*/
createItem: function(value) {
var item = new this.itemConstructor_(value);
@@ -1159,7 +1158,7 @@ cr.define('cr.ui', function() {
* updated and restored. Assumed the listItem relates to the same data item
* as the lead item in the begin of the batch update.
*
- * @param {ListItem} leadItem Already existing lead item.
+ * @param {cr.ui.ListItem} leadItem Already existing lead item.
*/
restoreLeadItem: function(leadItem) {
delete this.cachedItems_[leadItem.listIndex];
@@ -1202,7 +1201,7 @@ cr.define('cr.ui', function() {
* the visible range).
*
* Item returned from this method won't be removed until it remains a lead
- * item or til the data model changes (unlike other items that could be
+ * item or till the data model changes (unlike other items that could be
* removed when they go out of the visible range).
*
* @return {cr.ui.ListItem} The lead item for the list.
« no previous file with comments | « ui/webui/resources/js/cr/ui/context_menu_handler.js ('k') | ui/webui/resources/js/cr/ui/splitter.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698