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

Side by Side Diff: ios/chrome/app/resources/history/history.js

Issue 2280853002: Slightly nicer icon.js APIs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /////////////////////////////////////////////////////////////////////////////// 5 ///////////////////////////////////////////////////////////////////////////////
6 // Globals: 6 // Globals:
7 /** @const */ var RESULTS_PER_PAGE = 150; 7 /** @const */ var RESULTS_PER_PAGE = 150;
8 8
9 // Amount of time between pageviews that we consider a 'break' in browsing, 9 // Amount of time between pageviews that we consider a 'break' in browsing,
10 // measured in milliseconds. 10 // measured in milliseconds.
(...skipping 415 matching lines...) Expand 10 before | Expand all | Expand 10 after
426 this.domain_); 426 this.domain_);
427 return node; 427 return node;
428 }; 428 };
429 429
430 /** 430 /**
431 * Set the favicon for an element. 431 * Set the favicon for an element.
432 * @param {Element} el The DOM element to which to add the icon. 432 * @param {Element} el The DOM element to which to add the icon.
433 * @private 433 * @private
434 */ 434 */
435 Visit.prototype.addFaviconToElement_ = function(el) { 435 Visit.prototype.addFaviconToElement_ = function(el) {
436 var url = cr.icon.getFaviconImageSet(this.url_, 32, 'touch-icon'); 436 var url = cr.icon.getFavicon(this.url_, 32, 'touch-icon');
437 el.style.backgroundImage = url; 437 el.style.backgroundImage = url;
438 chrome.requestFavicon(el, url); 438 chrome.requestFavicon(el, url);
439 }; 439 };
440 440
441 /** 441 /**
442 * Launch a search for more history entries from the same domain. 442 * Launch a search for more history entries from the same domain.
443 * @private 443 * @private
444 */ 444 */
445 Visit.prototype.showMoreFromSite_ = function() { 445 Visit.prototype.showMoreFromSite_ = function() {
446 recordUmaAction('HistoryPage_EntryMenuShowMoreFromSite'); 446 recordUmaAction('HistoryPage_EntryMenuShowMoreFromSite');
(...skipping 1905 matching lines...) Expand 10 before | Expand all | Expand 10 after
2352 historyView.reload(); 2352 historyView.reload();
2353 } 2353 }
2354 2354
2355 // Add handlers to HTML elements. 2355 // Add handlers to HTML elements.
2356 document.addEventListener('DOMContentLoaded', load); 2356 document.addEventListener('DOMContentLoaded', load);
2357 2357
2358 // This event lets us enable and disable menu items before the menu is shown. 2358 // This event lets us enable and disable menu items before the menu is shown.
2359 document.addEventListener('canExecute', function(e) { 2359 document.addEventListener('canExecute', function(e) {
2360 e.canExecute = true; 2360 e.canExecute = true;
2361 }); 2361 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698