Merge 21595 - Partial fix to 7028 Pinning in Win7.A complete fix will requir...

Merge 21595 - Partial fix to 7028  Pinning in Win7.A complete fix will require Gears to set the application id as a property on the shortcut  As of this cl, web applications hosted by Chrome will appear in their own groups on the task bar.  However, they can not be pinned from the main application window nor can they be pinned from the shortcut.  The former results in Chrome being pinned, and the latter results in a quick start button but does not group web applications under that button.  Instead in the latter case a web appliation will form a new group.

Bug=7028
Test=None.  (When there is a Win7 trybot there may be at test to check if the windows group in the taskbar properly)
Review URL: http://codereview.chromium.org/159336

TBR=brg@chromium.com


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=21783

[brg_google.com, 2009-07-28 17:30:23]

LGTM.  Please merge in http://codereview.chromium.org/160150 as well
as it has a COM usage fix.


On Mon, Jul 27, 2009 at 6:18 PM, <laforge@chromium.org> wrote:
>
> Reviewers: brg,
>
> Description:
> Merge 21595 - Partial fix to 7028 =A0Pinning in Win7.A complete fix will
> require Gears to set the application id as a property on the shortcut
> As of this cl, web applications hosted by Chrome will appear in their
> own groups on the task bar. =A0However, they can not be pinned from the
> main application window nor can they be pinned from the shortcut. =A0The
> former results in Chrome being pinned, and the latter results in a quick
> start button but does not group web applications under that button.
> Instead in the latter case a web appliation will form a new group.
>
> Bug=3D7028
> Test=3DNone. =A0(When there is a Win7 trybot there may be at test to chec=
k
> if the windows group in the taskbar properly)
> Review URL: http://codereview.chromium.org/159336
>
> TBR=3Dbrg@chromium.com
>
>
> Please review this at http://codereview.chromium.org/159468
>
> SVN Base: svn://chrome-svn/chrome/branches/195/src/
>
> Affected files:
> =A0MM =A0 =A0app/win_util.h
> =A0MM =A0 =A0app/win_util.cc
> =A0MM =A0 =A0chrome/browser/browser.cc
>
>
>