Taking this

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/94fb1307c47c40f8c50d1cc03105e6d94d154e4c

commit 94fb1307c47c40f8c50d1cc03105e6d94d154e4c
Author: Sergio Villar Senin <svillar@igalia.com>
Date: Mon Jan 14 10:31:53 2019

Port BookMarkBubbleView unit test away of using SigninManager

First of all we don't need to use a FakeSigninManager as it works fine
with the SigninManager created by the factory from the profile. We
then go further and replace the SigninManager API by the
IdentityManager API.

In order not to crash we must modify
ChromeSigninClient::GetInstallDate to deal with the case of a
BrowserProcess without MetricsService as it happens with the
TestingBrowserProcess for example. This modification is interesting
per se as the BrowserProcess::metrics_service() clearly states that it
might return nullptr.

Bug:  920232 
Change-Id: I15377fa9838e1975c4655a775add84973c65c71a
Reviewed-on: https://chromium-review.googlesource.com/c/1403416
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Commit-Queue: Sergio Villar <svillar@igalia.com>
Cr-Commit-Position: refs/heads/master@{#622409}
[modify] https://crrev.com/94fb1307c47c40f8c50d1cc03105e6d94d154e4c/chrome/browser/signin/chrome_signin_client.cc
[modify] https://crrev.com/94fb1307c47c40f8c50d1cc03105e6d94d154e4c/chrome/browser/ui/views/bookmarks/bookmark_bubble_view_unittest.cc