Apps show default icon after icon file corruption. |
|||||||||
Issue descriptionInternal link http://b/35947207
,
Mar 16 2017
,
Mar 17 2017
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 20 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 20 2017
Clearing randomly added cc list by sheriffbot. sorry for the spam. thanks.
,
Mar 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/61676170d0826acfb754e8f434ef79fe909e271a commit 61676170d0826acfb754e8f434ef79fe909e271a Author: Long Cheng <lgcheng@google.com> Date: Mon Mar 20 23:30:28 2017 [Merge To M58]Fix Default icon issue when cached icon file is corrupted. Some crahes lead to app cached icon file corruption. Re send icon request to ARC if bad icon is observed. Limit icon request to once per app per scale per user session. Bug= 701979 Test=Unit test added. Test=Manual test. Manual corrupted cached icon. See icon restored. TBR=lhchavez@chromium.org, xiyuan@chromium.org Review-Url: https://codereview.chromium.org/2749973002 Cr-Commit-Position: refs/heads/master@{#457294} (cherry picked from commit f24408d3df3bf74f497111dd551d26a7509a07f4) Review-Url: https://codereview.chromium.org/2763753003 . Cr-Commit-Position: refs/branch-heads/3029@{#316} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} [modify] https://crrev.com/61676170d0826acfb754e8f434ef79fe909e271a/chrome/browser/ui/app_list/arc/arc_app_icon.cc [modify] https://crrev.com/61676170d0826acfb754e8f434ef79fe909e271a/chrome/browser/ui/app_list/arc/arc_app_icon.h [modify] https://crrev.com/61676170d0826acfb754e8f434ef79fe909e271a/chrome/browser/ui/app_list/arc/arc_app_list_prefs.cc [modify] https://crrev.com/61676170d0826acfb754e8f434ef79fe909e271a/chrome/browser/ui/app_list/arc/arc_app_list_prefs.h [modify] https://crrev.com/61676170d0826acfb754e8f434ef79fe909e271a/chrome/browser/ui/app_list/arc/arc_app_unittest.cc [modify] https://crrev.com/61676170d0826acfb754e8f434ef79fe909e271a/components/arc/test/fake_app_instance.cc [modify] https://crrev.com/61676170d0826acfb754e8f434ef79fe909e271a/components/arc/test/fake_app_instance.h
,
May 23 2017
,
Aug 1 2017
,
Jan 22 2018
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by bugdroid1@chromium.org
, Mar 16 2017