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

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

Issue 12560002: Change var blah_blah -> blahBlah. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 9 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 | « chrome/browser/resources/gesture_config.js ('k') | chrome/browser/resources/net_internals/source_entry.js » ('j') | 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 49e89276a6b25b66434600157903b19922a6f0bb..f2a155424ad4c513f8db6e6a2c1a43fb9a8c72c6 100644
--- a/chrome/browser/resources/history/history.js
+++ b/chrome/browser/resources/history/history.js
@@ -615,7 +615,7 @@ HistoryModel.prototype.queryHistory_ = function() {
* @private
*/
HistoryModel.prototype.haveDataForPage_ = function(page) {
- return (page * RESULTS_PER_PAGE < this.getSize());
+ return page * RESULTS_PER_PAGE < this.getSize();
};
/**
@@ -1263,21 +1263,21 @@ function PageState(model, view) {
// TODO(glen): Replace this with a bound method so we don't need
// public model and view.
- this.checker_ = setInterval((function(state_obj) {
- var hashData = state_obj.getHashData();
- var isGroupedByDomain = (hashData.grouped == 'true');
+ this.checker_ = window.setInterval(function(stateObj) {
+ var hashData = stateObj.getHashData();
+ var isGroupedByDomain = hashData.grouped == 'true';
var page = parseInt(hashData.page, 10);
var range = parseInt(hashData.range, 10);
var offset = parseInt(hashData.offset, 10);
- if (hashData.q != state_obj.model.getSearchText() ||
- page != state_obj.view.getPage() ||
- isGroupedByDomain != state_obj.view.model_.getGroupByDomain() ||
- range != state_obj.model.rangeInDays ||
- offset != state_obj.model.offset) {
- state_obj.view.setPageState(hashData.q, page, isGroupedByDomain,
- range, offset);
+ if (hashData.q != stateObj.model.getSearchText() ||
+ page != stateObj.view.getPage() ||
+ isGroupedByDomain != stateObj.view.model_.getGroupByDomain() ||
+ range != stateObj.model.rangeInDays ||
+ offset != stateObj.model.offset) {
+ stateObj.view.setPageState(hashData.q, page, isGroupedByDomain,
+ range, offset);
}
- }), 50, this);
+ }, 50, this);
}
/**
@@ -1600,9 +1600,9 @@ function updateParentCheckbox(checkbox) {
if (!entry)
return;
- var group_checkbox = entry.querySelector('.site-domain-wrapper input');
- if (group_checkbox)
- group_checkbox.checked = false;
+ var groupCheckbox = entry.querySelector('.site-domain-wrapper input');
+ if (groupCheckbox)
+ groupCheckbox.checked = false;
}
function entryBoxMousedown(event) {
« no previous file with comments | « chrome/browser/resources/gesture_config.js ('k') | chrome/browser/resources/net_internals/source_entry.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698