gecko-dev/mobile
Grigory Kruglov 631a64a4b9 Bug 1381359 - Hide AS Pin Site item from classic Top Sites context menu r=liuche
Our home panel context menu is shared by classic top sites, history and bookmark panels.
For acitivty stream, we've added a new context menu, and a new type of pins. However,
old context menu is still used for history and bookmarks. New style of pins allows for
items to be pinned from those panels to activity stream. This requires having both "classic"
pin buttons for old Top Sites panel, as well as the new AS Pin button.

However, initial implementation forgot to disable the new AS Pin button for the classic
Top Sites panel context menu. This caused two "pin site" menu items to appear, and the
AS one would crash the browser if it's interacted with.

We'll be unifying context menus at some point, and removing the old Top Sites code,
and so this simple patch should be sufficient.

MozReview-Commit-ID: 3cxVTv6QP8L

--HG--
extra : rebase_source : 967ad0512f7362286218cb038ce237c9e5c59aa8
2017-08-11 16:56:07 -04:00
..
android Bug 1381359 - Hide AS Pin Site item from classic Top Sites context menu r=liuche 2017-08-11 16:56:07 -04:00
locales Bug 1375277 - New safebrowsing threat type "POTENTIALLY_HARMFUL_APPLICATION" introduced by v4. r=francois 2017-07-27 15:37:57 +08:00