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

Unified Diff: chrome/browser/resources/history/history.js

Issue 439313004: history: remove unused variable. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sergiu@ review 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/history/history.js
diff --git a/chrome/browser/resources/history/history.js b/chrome/browser/resources/history/history.js
index f71c4992d985dfac0d05218b1ab03232550e3d8a..12e2d84a83db2de3aa369007af1edb1f686d6157 100644
--- a/chrome/browser/resources/history/history.js
+++ b/chrome/browser/resources/history/history.js
@@ -844,15 +844,6 @@ HistoryModel.prototype.canFillPage_ = function(page) {
};
/**
- * Enables or disables grouping by domain.
- * @param {boolean} groupByDomain New groupByDomain_ value.
- */
-HistoryModel.prototype.setGroupByDomain = function(groupByDomain) {
- this.groupByDomain_ = groupByDomain;
- this.offset_ = 0;
-};
-
-/**
* Gets whether we are grouped by domain.
* @return {boolean} Whether the results are grouped by domain.
*/
@@ -1766,7 +1757,6 @@ PageState.prototype.getHashData = function() {
var result = {
q: '',
page: 0,
- grouped: false,
range: 0,
offset: 0
};
@@ -1777,9 +1767,8 @@ PageState.prototype.getHashData = function() {
var hashSplit = window.location.hash.substr(1).split('&');
for (var i = 0; i < hashSplit.length; i++) {
var pair = hashSplit[i].split('=');
- if (pair.length > 1) {
+ if (pair.length > 1)
result[pair[0]] = decodeURIComponent(pair[1].replace(/\+/g, ' '));
- }
}
return result;
@@ -1847,7 +1836,6 @@ function load() {
// Create default view.
var hashData = pageState.getHashData();
- var grouped = (hashData.grouped == 'true') || historyModel.getGroupByDomain();
var page = parseInt(hashData.page, 10) || historyView.getPage();
var range = /** @type {HistoryModel.Range} */(parseInt(hashData.range, 10)) ||
historyView.getRangeInDays();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698