GitLab

The issue here is that the nullptr returned from the function
creates an error path which the caller must handle, or else the
code crashes.

In the updater, a null version and an empty version mean the same
thing, therefore, this CL replaces the nullptr with "".

There is an unrelated change to reduce the verbosity  of logging
for a repeating callback.


Bug: 1128195
Change-Id: I94a8853306edf7bb6eaddfe5ac0807ccaf9e0a2b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2411551
Auto-Submit: Sorin Jianu <sorin@chromium.org>
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Cr-Commit-Position: refs/heads/master@{#806990}