[Pkg-mozext-commits] [tabmixplus] 06/23: Unnecessary call to TMP_Places.getTabTitle when PlacesCommandHook.uniqueCurrentPages called from PlacesCommandHook.updateBookmarkAllTabsCommand, updateBookmarkAllTabsCommand only check uniqueCurrentPages.length

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository tabmixplus.

commit 58b3e1c811e76a05d0f47af3a208e8d034ff95c9
Author: onemen <tabmix.onemen at gmail.com>
Date:   Mon Feb 8 11:30:57 2016 +0200

    Unnecessary call to TMP_Places.getTabTitle when PlacesCommandHook.uniqueCurrentPages called from PlacesCommandHook.updateBookmarkAllTabsCommand, updateBookmarkAllTabsCommand only check uniqueCurrentPages.length
---
 chrome/content/places/places.js | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/chrome/content/places/places.js b/chrome/content/places/places.js
index 840c9fc..44b16bf 100644
--- a/chrome/content/places/places.js
+++ b/chrome/content/places/places.js
@@ -60,10 +60,15 @@ var TMP_Places = {
         ).toCode();
       }
 
+      let $LF = '\n        ';
       Tabmix.changeCode(PlacesCommandHook, "uniqueCurrentPages", {getter: true})._replace(
         'URIs.push(tab.linkedBrowser.currentURI);',
-        'let uri = tab.linkedBrowser.currentURI; \
-         URIs.push({uri: uri, title: TMP_Places.getTabTitle(tab, uri.spec)});'
+        'if (Tabmix.getCallerNameByIndex(2) == "PCH_updateBookmarkAllTabsCommand") {' + $LF +
+        '  $&' + $LF +
+        '} else {' + $LF +
+        '  let uri = tab.linkedBrowser.currentURI;' + $LF +
+        '  URIs.push({uri: uri, title: TMP_Places.getTabTitle(tab, uri.spec)});' + $LF +
+        '}'
       ).defineProperty();
     }
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/tabmixplus.git