Fix crash in ExternalPnoScanRequestManager

This CL does several things:
1. Fixes the nullpointer exeption that could happen during logging when
   e.getMessage() is null.
2. Adds more logging to improve debugging
3. Removes the triggering of "onRemoved" when the same caller
   re-registers the callback since it could actually confuse the
   caller to think the new registration failed if they reuse the same
   callback object.

Bug: 234654617
Test: atest com.android.server.wifi
Test: manually verified with adb shell commands
Change-Id: Ifebac6fceb63ea4f94ab9cd064cd76aa79638ac9
3 files changed