Issue 253403002: Linux: The App Launcher now appears on the "Internet" menu, not "Other".

Issue 253403002: Linux: The App Launcher now appears on the "Internet" menu, not "Other". (Closed)

Created:
6 years, 8 months ago by Matt Giuca

Modified:
6 years, 8 months ago

Reviewers:
Elliot Glaysher

CC:
chromium-reviews, chrome-apps-syd-reviews_chromium.org

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Linux: The App Launcher now appears on the "Internet" menu, not "Other". If the App Launcher has already been installed, this change will have no effect until chrome is run with the flags: --reset-app-list-install-state --enable-app-list flags. BUG=366445 TEST=Run chrome --reset-app-list-install-state --enable-app-list. Chrome App Launcher shortut should be on "Internet" menu on desktop environments with an applications menu (eg. Gnome Classic or Cinnamon). Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266274

Patch Set 1 #

Patch Set 2 : Use a global array, instead of a global pointer to a string. #

Created: 6 years, 8 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+37 lines, -5 lines)			Patch
M	chrome/browser/shell_integration_linux.h	View		2 chunks	+2 lines, -0 lines	0 comments	Download
M	chrome/browser/shell_integration_linux.cc	View	1	7 chunks	+23 lines, -4 lines	0 comments	Download
M	chrome/browser/shell_integration_linux_unittest.cc	View		11 chunks	+12 lines, -1 line	0 comments	Download