An unfortunate number of instances of this warning are actually
depending on this behavior, but in those cases it's probably better to
have an actual method making the intent explicit. The exception is
WrappableSearcherManager, which is intended to have minimal diff from
the implementation we copied from Lucene.
In the cases in PatchScriptBuilder, it was using "x == y || x.equals(y)"
as a sort of null-safe comparison, which we now have Objects#equals for.
Change-Id: I8781765813459c7fd03a45d4ef0fb355b8c5f486
b9640499d4