Fix some UrlOverridingTests

I set out to figure out why some tests were still launching the play
store after
https://chromium-review.googlesource.com/c/chromium/src/+/3513091 made
the tests no longer require play store to be installed.

Given some tests test weird schemes I can't use the approach I used
there to make them not launch play, so instead I wrapped the
PackageManager to always say play isn't installed.

This didn't work initially because apparently the 4 tests I fixed here
weren't actually testing anything because they didn't correctly report
that they opened a new tab, and the default value for the last reported
UrlOverriding result was NO_OVERRIDE, which is what these tests were
expecting, so the tests finished before the navigation even started.

That last point probably means a bunch of tests here were at least
flakily not actually testing anything, so I changed the default to be
out of band and ensured that if a tab is created that the test
intended for one to be.

PackageManagerWrapper was also missing a function needed by the
support library, and so was throwing Exceptions when the hidden abstract
function was called. Rather than figuring out how to update the cipd
package, PackageManagerWrapper should really just be checked into
chromium.

Change-Id: If69df35d2f32154287ed476114cf111439d181e6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3597651
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Auto-Submit: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: Bo Liu <boliu@chromium.org>
Commit-Queue: Bo Liu <boliu@chromium.org>
Cr-Commit-Position: refs/heads/main@{#995142}
NOKEYCHECK=True
GitOrigin-RevId: c23d003d776226ff53cb72aa0319400a22d4fb1a
2 files changed