[NetLog Viewer] Resolve TODOs from M59 and M87 in dns_view.js
[NetLog Viewer] Resolve TODOs from M59 and M87 in dns_view.js

* Drop the valueOrDefault function, which was used only for host cache
  fields that might not exist prior to M59.
* Drop the support for pre-M87 `error` (now it's just `net_error`).
* (Unrelated) Clean up Array detection with `Array.isArray()`.

Bug: chromium:1312236
Change-Id: I300656ce136be665532cd6ec998a839647a31aea
Reviewed-on: https://chromium-review.googlesource.com/c/catapult/+/3563511
Reviewed-by: Matt Menke <db2b536c538f3dbd8e890e06e08bcb9ec34adf6d@chromium.org>
Commit-Queue: Dan McArdle <8fb240f11ec935676fcfd8aed04f42f9ee63efa8@chromium.org>

catapult-project/catapult,catapult-project/catapult,catapult-project/catapult,catapult-project/catapult,catapult-project/catapult,catapult-project/catapult,catapult-project/catapult