Bug 1550289 - changing pocket url r=thecount
authorGavin Lazar Suntop <gavin@gsuntop.com>
Mon, 13 May 2019 21:43:54 +0000
changeset 532507 30c5aa2279c6d4f3952b0db3da22fed9ff21f949
parent 532506 c23eb772cf2d59a8d7954340a010768a757e71f9
child 532508 ff2278c703c15245c7d8639f4fc10d13ba44b51c
push id11268
push usercsabou@mozilla.com
push dateTue, 14 May 2019 15:24:22 +0000
treeherdermozilla-beta@5fb7fcd568d6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersthecount
bugs1550289
milestone68.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1550289 - changing pocket url r=thecount Differential Revision: https://phabricator.services.mozilla.com/D30557
browser/components/pocket/content/Pocket.jsm
browser/components/pocket/test/browser_pocket_library_menu_action.js
--- a/browser/components/pocket/content/Pocket.jsm
+++ b/browser/components/pocket/content/Pocket.jsm
@@ -8,17 +8,17 @@ var EXPORTED_SYMBOLS = ["Pocket"];
 
 const {Services} = ChromeUtils.import("resource://gre/modules/Services.jsm");
 
 ChromeUtils.defineModuleGetter(this, "BrowserUtils",
   "resource://gre/modules/BrowserUtils.jsm");
 
 var Pocket = {
   get site() { return Services.prefs.getCharPref("extensions.pocket.site"); },
-  get listURL() { return "https://" + Pocket.site + "/?src=ff_ext"; },
+  get listURL() { return "https://" + Pocket.site + "/firefox_learnmore?src=ff_library"; },
 
   openList(event) {
     let win = event.view;
     let where = win.whereToOpenLink(event);
     // Never override the current tab unless it's blank:
     if (where == "current" && !win.gBrowser.selectedTab.isEmpty) {
       where = "tab";
     }
--- a/browser/components/pocket/test/browser_pocket_library_menu_action.js
+++ b/browser/components/pocket/test/browser_pocket_library_menu_action.js
@@ -21,19 +21,19 @@ add_task(async function() {
   await libraryPromise;
 
   let pocketLibraryButton = document.getElementById("appMenu-library-pocket-button");
   ok(pocketLibraryButton, "library menu should have pocket button");
   is(pocketLibraryButton.disabled, false, "element appMenu-library-pocket-button is not disabled");
 
   info("clicking on pocket library button");
   let pocketPagePromise = BrowserTestUtils.waitForNewTab(gBrowser,
-    "https://example.com/browser/browser/components/pocket/test/pocket_actions_test.html/?src=ff_ext");
+    "https://example.com/browser/browser/components/pocket/test/pocket_actions_test.html/firefox_learnmore?src=ff_library");
   pocketLibraryButton.click();
   await pocketPagePromise;
 
   is(gBrowser.currentURI.spec,
-    "https://example.com/browser/browser/components/pocket/test/pocket_actions_test.html/?src=ff_ext",
+    "https://example.com/browser/browser/components/pocket/test/pocket_actions_test.html/firefox_learnmore?src=ff_library",
     "pocket button in library menu button opens correct page");
 
   BrowserTestUtils.removeTab(tab);
   BrowserTestUtils.removeTab(gBrowser.selectedTab);
 });