Skip updating jumplist icons if creation of folder JumplistIcons fails

Skip updating jumplist icons if creation of folder JumplistIcons fails

According to UMA, 0.14% of jumplist IO operations fail to create folder 
JumplistIcons after its successful deletion for unknown reasons. This 
could impact roughly 0.14% Chrome users. In this case, updating jumplist
icons is meaningless and should be skipped. 

BUG=40407, 179576
Review-Url: https://codereview.chromium.org/2739683003
Cr-Commit-Position: refs/heads/master@{#455489}
Committed: https://chromium.googlesource.com/chromium/src/+/051b0af2835a602463b2960894f296f3dcc22680

[chengx, 2017-03-07 18:39:47]

The CQ bit was checked by chengx@chromium.org to run a CQ dry run

[chengx, 2017-03-07 18:40:59]

Description was changed from

==========
Exit early if CreateDirectory() fails.

BUG=40407, 179576
==========

to

==========
Skip updating jumplist icons if creation of folder JumplistIcons fails

BUG=40407, 179576
==========

[commit-bot: I haz the power, 2017-03-07 18:41:07]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[chengx, 2017-03-07 18:48:08]

Description was changed from

==========
Skip updating jumplist icons if creation of folder JumplistIcons fails

BUG=40407, 179576
==========

to

==========
Skip updating jumplist icons if creation of folder JumplistIcons fails

According to UMA, 0.14% of jumplist IO operations fail to create folder 
JumplistIcons after its successful deletion for unknown reasons. This 
could impact roughly 0.14% Chrome users. In this case, updating jumplist
icons is meaningless and should be skipped. 

BUG=40407, 179576
==========

[chengx, 2017-03-07 18:49:54]

chengx@chromium.org changed reviewers:
+ grt@google.com

[chengx, 2017-03-07 18:49:54]

PTAL~

[commit-bot: I haz the power, 2017-03-07 19:31:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-07 19:31:20]

Dry run: This issue passed the CQ dry run.

[chengx, 2017-03-07 21:27:29]

chengx@chromium.org changed reviewers:
+ grt@chromium.org
- grt@google.com

[chengx, 2017-03-07 21:27:30]

Hi grt@, it seems I used your corp email rather than chromium email by mistake.
Sorry about that. It is now fixed.

[grt (UTC plus 2), 2017-03-08 09:05:04]

lgtm

[chengx, 2017-03-08 17:54:24]

The CQ bit was checked by chengx@chromium.org

[commit-bot: I haz the power, 2017-03-08 17:54:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-08 18:00:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488995664904940, "parent_rev":
"1cc087cf821116b546d7f02fffa189dd5b9f1a41", "commit_rev":
"051b0af2835a602463b2960894f296f3dcc22680"}

[commit-bot: I haz the power, 2017-03-08 18:01:17]

Description was changed from

==========
Skip updating jumplist icons if creation of folder JumplistIcons fails

According to UMA, 0.14% of jumplist IO operations fail to create folder 
JumplistIcons after its successful deletion for unknown reasons. This 
could impact roughly 0.14% Chrome users. In this case, updating jumplist
icons is meaningless and should be skipped. 

BUG=40407, 179576
==========

to

==========
Skip updating jumplist icons if creation of folder JumplistIcons fails

According to UMA, 0.14% of jumplist IO operations fail to create folder 
JumplistIcons after its successful deletion for unknown reasons. This 
could impact roughly 0.14% Chrome users. In this case, updating jumplist
icons is meaningless and should be skipped. 

BUG=40407, 179576

Review-Url: https://codereview.chromium.org/2739683003
Cr-Commit-Position: refs/heads/master@{#455489}
Committed:
https://chromium.googlesource.com/chromium/src/+/051b0af2835a602463b2960894f2...
==========

[commit-bot: I haz the power, 2017-03-08 18:01:18]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/051b0af2835a602463b2960894f2...