GitLab

Before this CL: favicon is always loaded when creating specifics out of
a bookmark node.

After this CL: a new paramter is added to control this behavior and
ignores the favicon field if the favicon is not loaded already.

This is added such that in debug page chrome://sync-internals,
the load of a favicon is avoid if not loaded already.

Bug: 516866
Change-Id: I14093cea131d44e83cca1b5ab07d64d6b46061c1
Reviewed-on: https://chromium-review.googlesource.com/c/1254066
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596550}