Bug 1343850 - Fix PlacesProvider.jsm undefined variable (aLastKnownTitle vs. aTitle). r=mak, a=lizzard
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Thu, 02 Mar 2017 13:51:05 +0000
changeset 376632 a94edff8355a1c83ad2bfad80d48540a3cfdfcf6
parent 376631 d7d8bd983a300b3fa9a15be16b698c62e48f4b2b
child 376633 69efeba94b307f236a50a9e5fdd2bbd2e946657b
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmak, lizzard
bugs1343850
milestone53.0a2
Bug 1343850 - Fix PlacesProvider.jsm undefined variable (aLastKnownTitle vs. aTitle). r=mak, a=lizzard MozReview-Commit-ID: D86SQTcMLvM
browser/components/newtab/PlacesProvider.jsm
--- a/browser/components/newtab/PlacesProvider.jsm
+++ b/browser/components/newtab/PlacesProvider.jsm
@@ -110,17 +110,17 @@ Links.prototype = {
       // See toolkit/components/places/tests/unit/test_frecency_observers.js
       gLinks.emit("manyLinksChanged");
     },
 
     onVisit(aURI, aVisitId, aTime, aSessionId, aReferrerVisitId, aTransitionType,
             aGuid, aHidden, aVisitCount, aTyped, aLastKnownTitle) {
       // For new visits, if we're not batch processing, notify for a title update
       if (!this._batchProcessingDepth && aVisitCount == 1 && aLastKnownTitle) {
-        this.onTitleChanged(aURI, aTitle, aGuid);
+        this.onTitleChanged(aURI, aLastKnownTitle, aGuid);
       }
     },
 
     onTitleChanged: function historyObserver_onTitleChanged(aURI, aNewTitle) {
       if (NewTabUtils.linkChecker.checkLoadURI(aURI.spec)) {
         gLinks.emit("linkChanged", {
           url: aURI.spec,
           title: aNewTitle