Add app icons to Task Manager. (b/28817774)
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/46a8fa93b2dd79ecbd9435917641b3103e764892 commit 46a8fa93b2dd79ecbd9435917641b3103e764892 Author: yusukes <yusukes@chromium.org> Date: Tue May 31 22:12:58 2016 Add a comment to ActivityName::activity_name. This reflects the change in ag/1087804. BUG= 616159 Review-Url: https://codereview.chromium.org/2026913002 Cr-Commit-Position: refs/heads/master@{#396947} [modify] https://crrev.com/46a8fa93b2dd79ecbd9435917641b3103e764892/components/arc/intent_helper/activity_icon_loader.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49 commit cd2c2981937eb5c34e8cbf04ad2b772bf366cd49 Author: yusukes <yusukes@chromium.org> Date: Thu Jun 02 06:24:03 2016 Add ARC app icons to the Task Manager UI * Add |packages| member variable to arc::ArcProcess. * Modify ArcProcessTask::ArcProcessTask() so it fetches and sets an icon based on a package name in |packages|. BUG= 616159 Review-Url: https://codereview.chromium.org/2023643002 Cr-Commit-Position: refs/heads/master@{#397310} [add] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/browser/chromeos/arc/arc_process.cc [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/browser/chromeos/arc/arc_process.h [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/browser/chromeos/arc/arc_process_service.cc [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/browser/memory/tab_manager_delegate_chromeos.cc [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/browser/memory/tab_manager_delegate_chromeos_unittest.cc [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/browser/task_management/providers/arc/arc_process_task.cc [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/browser/task_management/providers/arc/arc_process_task.h [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/browser/task_management/providers/arc/arc_process_task_provider.cc [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/chrome/chrome_browser_chromeos.gypi [modify] https://crrev.com/cd2c2981937eb5c34e8cbf04ad2b772bf366cd49/components/arc/common/process.mojom
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b229a826d51cf8d5663184bcade36f3df9ee6014 commit b229a826d51cf8d5663184bcade36f3df9ee6014 Author: yusukes <yusukes@chromium.org> Date: Fri Jun 03 02:27:48 2016 Retry icon fetching after intent_helper is ready. * Modify ActivityIconLoader::GetActivityIcon() so it returns an error code. * Modify ArcProcessTask so it calls GetActivityIcon() again when the function returns a temporary error. BUG= 616159 TEST=All ARC tasks in Task Manager have their icons Review-Url: https://codereview.chromium.org/2034543003 Cr-Commit-Position: refs/heads/master@{#397581} [modify] https://crrev.com/b229a826d51cf8d5663184bcade36f3df9ee6014/chrome/browser/task_management/providers/arc/arc_process_task.cc [modify] https://crrev.com/b229a826d51cf8d5663184bcade36f3df9ee6014/chrome/browser/task_management/providers/arc/arc_process_task.h [modify] https://crrev.com/b229a826d51cf8d5663184bcade36f3df9ee6014/components/arc/intent_helper/activity_icon_loader.cc [modify] https://crrev.com/b229a826d51cf8d5663184bcade36f3df9ee6014/components/arc/intent_helper/activity_icon_loader.h [modify] https://crrev.com/b229a826d51cf8d5663184bcade36f3df9ee6014/components/arc/intent_helper/activity_icon_loader_unittest.cc [modify] https://crrev.com/b229a826d51cf8d5663184bcade36f3df9ee6014/components/arc/intent_helper/link_handler_model_impl.cc
Comment 1 by bugdroid1@chromium.org
, May 31 2016