GitLab

The need to access the WebContents when selecting the UKM source id for
ContactsManagerImpl appears to be from before the introduction of
RenderFrameHostImpl::GetPageUkmSourceId. The WebContents is now only
used to check whether there is a native window. This logic seems
unrelated to UKM source id selection, since there is already code that
checks for the existence of the native window to show a dialog in
ContactsProviderAndroid.

Noticed while auditing uses of WebContents::FromRenderFrameHost.

Bug: None
Change-Id: I68f652ad3c3cb2c40571bde59c7ea3e50618f7b5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422484Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
Commit-Queue: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809824}