Stop creating shortcut for web apps when it is installing.

Stop creating shortcut for web apps when it is installing.

BUG=255413
TEST=1.Launch Chrome; 2.Add an app from web store eg: (https://chrome.google.com/webstore/detail/shoot-many-robots-arena-k/gejophacjbklkgfilkoaeniiooobobkf?utm_source=chrome-ntp-icon). ; 3.When app downloading is in process drag the icon of the app out of the app launcher, the chrome should not crash.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=209944

[zhchbin, 2013-06-29 03:05:20]

Please review.

[zhchbin, 2013-07-03 03:32:02]

On 2013/06/29 03:05:20, zhchbin wrote:
> Please review.

Ping again.

[koz (OOO until 15th September), 2013-07-03 04:20:08]

lgtm once the comment is removed

https://chromiumcodereview.appspot.com/18153015/diff/1/chrome/browser/ui/app_...
File chrome/browser/ui/app_list/app_list_view_delegate.cc (right):

https://chromiumcodereview.appspot.com/18153015/diff/1/chrome/browser/ui/app_...
chrome/browser/ui/app_list/app_list_view_delegate.cc:106: // The installation of
extension isn't finished yet.
This comment is unnecessary, could you remove it? (it's too specific - the
extension might not be there for any number of reasons).

[zhchbin, 2013-07-03 05:59:59]

https://chromiumcodereview.appspot.com/18153015/diff/1/chrome/browser/ui/app_...
File chrome/browser/ui/app_list/app_list_view_delegate.cc (right):

https://chromiumcodereview.appspot.com/18153015/diff/1/chrome/browser/ui/app_...
chrome/browser/ui/app_list/app_list_view_delegate.cc:106: // The installation of
extension isn't finished yet.
On 2013/07/03 04:20:09, koz wrote:
> This comment is unnecessary, could you remove it? (it's too specific - the
> extension might not be there for any number of reasons).

Done.

[commit-bot: I haz the power, 2013-07-03 06:00:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zhchbin@gmail.com/18153015/6001

[commit-bot: I haz the power, 2013-07-03 08:13:29]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2013-07-03 08:18:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zhchbin@gmail.com/18153015/6001

[commit-bot: I haz the power, 2013-07-03 10:13:19]

Change committed as 209944