GitLab

Simplify tests in FaviconRequestHandlerTest

Whenever possible, we replace use of EXPECT_CALL macro with use of
ON_CALL in the constructor of the fixture to make the expectations
of each test clearer.

Change-Id: I86a509898e3e2d9893166e46114d94d79274e207
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1660865Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Victor Vianna <victorvianna@google.com>
Cr-Commit-Position: refs/heads/master@{#681245}