Merge pull request #3093 from rlr/pin-title-followup

fix(systemaddon): remove title from pinned site pref value
mozilla · Aug 4, 2017 · 9a4a0db · 9a4a0db
2 parents 3b2f4a3 + 480110b
commit 9a4a0db
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/system-addon/content-src/lib/link-menu-options.js b/system-addon/content-src/lib/link-menu-options.js
@@ -73,7 +73,7 @@ module.exports = {
     icon: "pin",
     action: ac.SendToMain({
       type: at.TOP_SITES_PIN,
-      data: {site: {url: site.url, title: site.hostname}, index}
+      data: {site: {url: site.url}, index}
     }),
     userEvent: "PIN"
   }),

diff --git a/system-addon/test/unit/content-src/components/LinkMenu.test.jsx b/system-addon/test/unit/content-src/components/LinkMenu.test.jsx
@@ -100,7 +100,7 @@ describe("<LinkMenu>", () => {
       menu_action_open_private_window: {url: FAKE_SITE.url, referrer: FAKE_SITE.referrer},
       menu_action_dismiss: FAKE_SITE.url,
       menu_action_delete: FAKE_SITE.url,
-      menu_action_pin: {site: {url: FAKE_SITE.url, title: FAKE_SITE.hostname}, index: FAKE_INDEX},
+      menu_action_pin: {site: {url: FAKE_SITE.url}, index: FAKE_INDEX},
       menu_action_unpin: {site: {url: FAKE_SITE.url}},
       menu_action_save_to_pocket: {site: {url: FAKE_SITE.url, title: FAKE_SITE.title}}
     };