mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Bug 1403437 - Use MenuUtils#safeSetVisible to hide '.*_add_to_launcher' safely. r=nechen 

MozReview-Commit-ID: iraSQCSvPw

--HG--
extra : rebase_source : a7769fed5217a05da2df86745b83b859712f3aa6
This commit is contained in:
Jing-wei Wu 2017-09-28 11:27:46 +08:00
Родитель f9569348f8
Коммит 1b01c60c49
2 изменённых файлов: 4 добавлений и 7 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

7
mobile/android/base/java/org/mozilla/gecko/home/HomeFragment.java
Просмотреть файл

@ -31,6 +31,7 @@ import org.mozilla.gecko.reader.ReadingListHelper;
import org.mozilla.gecko.restrictions.Restrictable;
import org.mozilla.gecko.restrictions.Restrictions;
import org.mozilla.gecko.util.Clipboard;
import org.mozilla.gecko.util.MenuUtils;
import org.mozilla.gecko.util.StringUtils;
import org.mozilla.gecko.util.ThreadUtils;
import org.mozilla.gecko.util.UIAsyncTask;
@ -208,11 +209,7 @@ public abstract class HomeFragment extends Fragment {
menu.findItem(R.id.home_open_private_tab).setVisible(false);
menu.findItem(R.id.home_copyurl).setVisible(false);
menu.findItem(R.id.home_share).setVisible(false);
final MenuItem addToLauncherItem = menu.findItem(R.id.home_add_to_launcher);
if (addToLauncherItem != null) {
addToLauncherItem.setVisible(false);
}
MenuUtils.safeSetVisible(menu, R.id.home_add_to_launcher, false);
menu.findItem(R.id.home_set_as_homepage).setVisible(false);
menu.findItem(R.id.home_as_pin).setVisible(false);

4
mobile/android/base/java/org/mozilla/gecko/toolbar/BrowserToolbar.java
Просмотреть файл

@ -262,7 +262,7 @@ public abstract class BrowserToolbar extends ThemedRelativeLayout
String url = tab.getURL();
if (url == null) {
menu.findItem(R.id.copyurl).setVisible(false);
menu.findItem(R.id.add_to_launcher).setVisible(false);
MenuUtils.safeSetVisible(menu, R.id.add_to_launcher, false);
menu.findItem(R.id.set_as_homepage).setVisible(false);
}
@ -273,7 +273,7 @@ public abstract class BrowserToolbar extends ThemedRelativeLayout
} else {
// if there is no tab, remove anything tab dependent
menu.findItem(R.id.copyurl).setVisible(false);
menu.findItem(R.id.add_to_launcher).setVisible(false);
MenuUtils.safeSetVisible(menu, R.id.add_to_launcher, false);
menu.findItem(R.id.set_as_homepage).setVisible(false);
MenuUtils.safeSetVisible(menu, R.id.subscribe, false);
MenuUtils.safeSetVisible(menu, R.id.add_search_engine, false);