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

Unified Diff: chrome/browser/resources/md_history/history_item.js

Issue 2684693004: MD History: Remove list-container and list-behavior (Closed)
Patch Set: Rename listeners Created 3 years, 10 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/md_history/history_item.js
diff --git a/chrome/browser/resources/md_history/history_item.js b/chrome/browser/resources/md_history/history_item.js
index 8f53904e3b0abbe4baf956745309294188f6214e..0415d721acbeac447d589ca52d0f4f977fd1ce5d 100644
--- a/chrome/browser/resources/md_history/history_item.js
+++ b/chrome/browser/resources/md_history/history_item.js
@@ -122,9 +122,6 @@ cr.define('md_history', function() {
numberOfItems: Number,
- // The path of this history item inside its parent.
- path: String,
-
// Search term used to obtain this history-item.
searchTerm: String,
},
@@ -199,7 +196,7 @@ cr.define('md_history', function() {
this.$.checkbox.focus();
this.fire('history-checkbox-select', {
- element: this,
+ index: this.index,
shiftKey: e.shiftKey,
});
},
@@ -262,7 +259,6 @@ cr.define('md_history', function() {
target: Polymer.dom(e).localTarget,
index: this.index,
item: this.item,
- path: this.path,
});
// Stops the 'tap' event from closing the menu when it opens.
@@ -329,28 +325,6 @@ cr.define('md_history', function() {
});
/**
- * Check whether the time difference between the given history item and the
- * next one is large enough for a spacer to be required.
- * @param {Array<HistoryEntry>} visits
- * @param {number} currentIndex
- * @param {string} searchedTerm
- * @return {boolean} Whether or not time gap separator is required.
- */
- HistoryItem.needsTimeGap = function(visits, currentIndex, searchedTerm) {
- if (currentIndex >= visits.length - 1 || visits.length == 0)
- return false;
-
- var currentItem = visits[currentIndex];
- var nextItem = visits[currentIndex + 1];
-
- if (searchedTerm)
- return currentItem.dateShort != nextItem.dateShort;
-
- return currentItem.time - nextItem.time > BROWSING_GAP_TIME &&
- currentItem.dateRelativeDay == nextItem.dateRelativeDay;
- };
-
- /**
* @param {number} numberOfResults
* @param {string} searchTerm
* @return {string} The title for a page of search results.
« no previous file with comments | « chrome/browser/resources/md_history/compiled_resources2.gyp ('k') | chrome/browser/resources/md_history/history_list.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698