GitLab

This CL refactors ChromeOSMetricsProvider::UpdateUserTypeUMA() to use
the guard clause pattern and removes an unnecessary return statement
in a void function. This CL also adds browser tests for the
UMA.PrimaryUserType histogram.

Bug: 1140768
Change-Id: I19f4f3d3e97b11bc3806bd1c67eca514888fa377
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2499171Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Commit-Queue: Toby Huang <tobyhuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#821740}