Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: mobile/android/base/resources/menu-large-v11/browser_app_menu.xml

Issue 5664054114779136: Issue 2350 - Remove Add-ons, Apps and Sync (Closed)
Patch Set: Removed sync from xml-v11, too Created April 21, 2015, 10:04 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mobile/android/base/resources/menu-large-v11/browser_app_menu.xml
diff --git a/mobile/android/base/resources/menu-large-v11/browser_app_menu.xml b/mobile/android/base/resources/menu-large-v11/browser_app_menu.xml
index 2e6a8997b47bde22832a61c9fa5fa272bbd11c35..affa10be8144732ae2654b6afd546c24b38efa7e 100644
--- a/mobile/android/base/resources/menu-large-v11/browser_app_menu.xml
+++ b/mobile/android/base/resources/menu-large-v11/browser_app_menu.xml
@@ -82,13 +82,9 @@
android:icon="@drawable/ic_menu_downloads"
android:title="@string/downloads"/>
- <item android:id="@+id/addons"
- android:icon="@drawable/ic_menu_addons"
- android:title="@string/addons"/>
+ <!-- removed addons as of issue #2350 -->
Felix Dahlke 2015/04/22 13:29:59 Can't we just make these invisible instead of full
- <item android:id="@+id/apps"
- android:icon="@drawable/ic_menu_apps"
- android:title="@string/apps"/>
+ <!-- removed apps as of issue #2350 -->
<item android:id="@+id/new_guest_session"
android:icon="@drawable/ic_menu_guest"

Powered by Google App Engine
This is Rietveld