GitLab

Existing menu code such as RenderViewContextMenu assumes the icons are
available synchronously.

We cache the favicon for each web app. If no icon is cached, the menu
item is shown with no visible icon.

References to Bookmark Apps in RenderViewContextMenu are replaced
with references to web apps.

Bug: 1052707
Change-Id: Ib6dd23c448e01dfb36575008532fcce76b339ddd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2216834
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Commit-Queue: Maggie Cai <mxcai@chromium.org>
Auto-Submit: Eric Willigers <ericwilligers@chromium.org>
Reviewed-by: Maggie Cai <mxcai@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: Alexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#772556}