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

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

Issue 2480203002: ui: Cleanup class/struct forward declarations (Closed)
Patch Set: Sync CL to position 430550 Created 4 years, 1 month 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/list_container.js
diff --git a/chrome/browser/resources/md_history/list_container.js b/chrome/browser/resources/md_history/list_container.js
index f751a8345d1f38dc8b4db1e5d4a1185d31437a5e..a54bc2f39746463c7498807d7ac33775437f5180 100644
--- a/chrome/browser/resources/md_history/list_container.js
+++ b/chrome/browser/resources/md_history/list_container.js
@@ -260,8 +260,8 @@ Polymer({
var browserService = md_history.BrowserService.getInstance();
browserService.recordHistogram(
- 'HistoryPage.RemoveEntryPosition', index,
- UMA_MAX_BUCKET_VALUE);
+ 'HistoryPage.RemoveEntryPosition',
+ Math.min(index, UMA_MAX_BUCKET_VALUE), UMA_MAX_BUCKET_VALUE);
if (index <= UMA_MAX_SUBSET_BUCKET_VALUE) {
browserService.recordHistogram(
'HistoryPage.RemoveEntryPositionSubset', index,

Powered by Google App Engine
This is Rietveld 408576698