GitLab

This CL performs some cleanup to the VRServiceImpl class.

1) Removed 'IsAnotherHostPresenting()' declaration, since that function
was never actually defined anywhere.

2) It inlines 'SetInFocusedFrame', since there was only a single call
to it, and it makes the code somewhat easier to follow.

3) Moved the check on 'initialization_complete_' inside of
'RequestSession' to the top of the function in order to be more
consistent with other functions in this class.

4) Removed null checks for 'render_frame_host_', because it's not
allowed to be null.

Change-Id: Ice0433923b747a3fda3e5e0178af2260b8b14d07
Bug: 846392
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1764549
Commit-Queue: Will Cassella <cassew@google.com>
Reviewed-by: Alexander Cooper <alcooper@chromium.org>
Reviewed-by: Klaus Weidner <klausw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#691738}