My favorites | Sign in
Logo
             
New issue | Search
for
| Advanced search | Search tips
Issue 22528: Create application shortcut - generic chromium icon used as opposed to the application specific icon
1 person starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  est...@chromium.org
Closed:  Sep 28
Cc:  phajdan...@chromium.org
OS-Linux
Size-Medium
Type-Bug
Pri-2
Mstone-4
Area-Internals


Sign in to add a comment
 
Reported by akarun87, Sep 21, 2009
Chrome Version       : 4.0.212.0~svn20090920r26664-0ubuntu1~ucd2
OS + version : Ubuntu Karmic Testing
CPU architecture (32-bit / 64-bit): 32 bit
window manager : gnome
URLs (if applicable) : gmail.com
Behavior in Firefox 3.x (if applicable):N/A
Behavior in Chrome for Windows (optional):The icon created is that of GMail 
and not google-chrome icon

What steps will reproduce the problem?
1.Visit gmail.com
2."Create application Shortcut"
3.Create a Desktop shortcut and App Menu shortcut


What is the expected result?
I must find an GMail icon on the desktop


What happens instead?
I find a Chromium icon (I need to mark it trusted - but that's just an 
extra step due to the OS)


Please provide any additional information below. Attach a screenshot
and backtrace if possible.

Comment 1 by est...@chromium.org, Sep 21, 2009
(No comment was entered for this change.)
Status: Available
Cc: phajdan...@chromium.org est...@chromium.org
Labels: -Area-Misc Area-BrowserBackend Pri-2 Mstone-4
Comment 2 by est...@chromium.org, Sep 28, 2009
(No comment was entered for this change.)
Status: Started
Owner: est...@chromium.org
Cc: -est...@chromium.org
Comment 3 by bugdroid1@chromium.org, Sep 28, 2009
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=27422 

------------------------------------------------------------------------
r27422 | estade@chromium.org | 2009-09-28 15:35:15 -0700 (Mon, 28 Sep 2009) | 5 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/create_application_shortcuts_dialog_gtk.cc?r1=27422&r2=27421
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/create_application_shortcuts_dialog_gtk.h?r1=27422&r2=27421
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/shell_integration.h?r1=27422&r2=27421
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/shell_integration_linux.cc?r1=27422&r2=27421
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/shell_integration_unittest.cc?r1=27422&r2=27421
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/tab_contents.cc?r1=27422&r2=27421
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/tab_contents.h?r1=27422&r2=27421

Use favicon for application shortcut icon.

BUG=22528

Review URL: http://codereview.chromium.org/249023
------------------------------------------------------------------------

Comment 4 by est...@chromium.org, Sep 28, 2009
(No comment was entered for this change.)
Status: Fixed
Comment 5 by mal.chromium, Dec 18, 2009
(No comment was entered for this change.)
Labels: -Area-BrowserBackend Area-Internals
Sign in to add a comment

Powered by Google Project Hosting