History: Fix RTL layout in grouped history and some other minor fixes.

chrome/browser/resources/history/history.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 <include src="../uber/uber_utils.js">
 
 ///////////////////////////////////////////////////////////////////////////////
 // Globals:
 /** @const */ var RESULTS_PER_PAGE = 150;
 
@@ skipping 157 matching lines @@
 // Visit, private: ------------------------------------------------------------
 
 /**
  * Extracts and returns the domain (and subdomains) from a URL.
  * @param {string} url The url.
  * @return {string} The domain. An empty string is returned if no domain can
  *     be found.
  * @private
  */
 Visit.prototype.getDomainFromURL_ = function(url) {
-  var domain = url.replace(/^.+:\/\//, '').match(/[^/]+/);
+  // TODO(sergiu): Extract the domain from the C++ side and send it here.
+  var domain = url.replace(/^.+?:\/\//, '').match(/[^/]+/);
   return domain ? domain[0] : '';
 };
 
 /**
  * Add child text nodes to a node such that occurrences of the specified text is
  * highlighted.
  * @param {Node} node The node under which new text nodes will be made as
  *     children.
  * @param {string} content Text to be added beneath |node| as one or more
  *     text nodes.
@@ skipping 210 matching lines @@
  * @param {Array} results A list of results.
  */
 HistoryModel.prototype.addResults = function(info, results) {
   $('loading-spinner').hidden = true;
   this.inFlight_ = false;
   this.isQueryFinished_ = info.finished;
   this.queryCursor_ = info.cursor;
   this.queryStartTime = info.queryStartTime;
   this.queryEndTime = info.queryEndTime;
 
-  // If the results are not for the current search term there's nothing more
-  // to do.
+  // If the results are not for the current search term then there is nothing
+  // more to do.
   if (info.term != this.searchText_)
     return;
 
   // If necessary, sort the results from newest to oldest.
   if (!results.sorted)
     results.sort(function(a, b) { return b.time - a.time; });
 
   var lastVisit = this.visits_.slice(-1)[0];
   var lastDay = lastVisit ? lastVisit.dateRelativeDay : null;
 
@@ skipping 409 matching lines @@
  * Record that the given visit has been rendered.
  * @param {Visit} visit The visit that was rendered.
  * @private
  */
 HistoryView.prototype.setVisitRendered_ = function(visit) {
   visit.isRendered = true;
   this.currentVisits_.push(visit);
 };
 
 /**
- * This function generates and adds the grouped visits DOM for a certain
- * domain. This includes the clickable arrow and domain name and the visit
- * entries for that domain.
+ * Generates and adds the grouped visits DOM for a certain domain. This
+ * includes the clickable arrow and domain name and the visit entries for
+ * that domain.
  * @param {Element} results DOM object to which to add the elements.
  * @param {string} domain Current domain name.
  * @param {Array} domainVisits Array of visits for this domain.
  * @private
  */
 HistoryView.prototype.getGroupedVisitsDOM_ = function(
     results, domain, domainVisits) {
   // Add a new domain entry.
   var siteResults = results.appendChild(
       createElementWithClassName('li', 'site-entry'));
   // Make a wrapper that will contain the arrow, the favicon and the domain.
   var siteDomainWrapper = siteResults.appendChild(
       createElementWithClassName('div', 'site-domain-wrapper'));
   var siteArrow = siteDomainWrapper.appendChild(
       createElementWithClassName('div', 'site-domain-arrow collapse'));
   siteArrow.textContent = '►';
   var siteDomain = siteDomainWrapper.appendChild(
       createElementWithClassName('div', 'site-domain'));
   var numberOfVisits = createElementWithClassName('span', 'number-visits');
   numberOfVisits.textContent = loadTimeData.getStringF('numbervisits',
                                                        domainVisits.length);
-  siteDomain.textContent = domain;
+  var domainElement = document.createElement('span');
+  domainElement.textContent = domain;
+  siteDomain.appendChild(domainElement);
   siteDomain.appendChild(numberOfVisits);
   siteResults.appendChild(siteDomainWrapper);
   var resultsList = siteResults.appendChild(
       createElementWithClassName('ol', 'site-results'));
 
   domainVisits[0].addFaviconToElement_(siteDomain);
 
   siteDomainWrapper.addEventListener('click', toggleHandler);
   // Collapse until it gets toggled.
   resultsList.style.height = 0;
@@ skipping 57 matching lines @@
       visitsByDomain[domain] = [];
       domains.push(domain);
     }
     visitsByDomain[domain].push(visit);
   }
   var sortByVisits = function(a, b) {
     return visitsByDomain[b].length - visitsByDomain[a].length;
   };
   domains.sort(sortByVisits);
 
-  for (var i = 0, domain; domain = domains[i]; i++) {
+  for (var i = 0, domain; domain = domains[i]; i++)
     this.getGroupedVisitsDOM_(results, domain, visitsByDomain[domain]);
-  }
 };
 
 /**
  * Adds the results for a month.
  * @param {Array} visits Visits returned by the query.
  * @param {Element} parentElement Element to which to add the results to.
  * @private
  */
 HistoryView.prototype.addMonthResults_ = function(visits, parentElement) {
   if (visits.length == 0)
@@ skipping 579 matching lines @@
     historyView.reload();
 }
 
 // Add handlers to HTML elements.
 document.addEventListener('DOMContentLoaded', load);
 
 // This event lets us enable and disable menu items before the menu is shown.
 document.addEventListener('canExecute', function(e) {
   e.canExecute = true;
 });