tree 5da6f05aad340f0ddd0b66c8e0ed9d2037463fca
parent d0bd51908bf0a4e0c12f706ff243b53f67c1e265
author Jeevan Shikaram <jshikaram@chromium.org> 1558498567 +0000
committer Commit Bot <commit-bot@chromium.org> 1558498567 +0000

[App Service] Get file path in a background thread.

ExtensionResource::GetFilePath will DCHECK if it's not called on a background
thread. Currently, the App Service calls this method on the UI thread, which
causes App Management to crash when DCHECKs are enabled.

This CL refactors apps::LoadIconFromExtension to ensure that this method is
only called on a background thread.

Bug: 916380
Change-Id: I3ba8757e884c6336b4b7b61fc46363c8273fcc86
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1619586
Reviewed-by: Nigel Tao <nigeltao@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org>
Cr-Commit-Position: refs/heads/master@{#662061}