Remove asParsedURL on the String prototype
Use a static function on the ParsedURL class instead.
A lot of our tooling is happier linking through to the class method
rather than the prototype thing, and setting things on prototypes is
generally discouraged in JS for a bunch of reasons.
Change-Id: I95cf70ed58e7cca6d4c2bf7633b69199f513d068
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/1958524
Reviewed-by: Tim van der Lippe <tvanderlippe@chromium.org>
Commit-Queue: Peter Marshall <petermarshall@chromium.org>
diff --git a/front_end/audits/AuditsStatusView.js b/front_end/audits/AuditsStatusView.js
index f3b2454..d5090b6 100644
--- a/front_end/audits/AuditsStatusView.js
+++ b/front_end/audits/AuditsStatusView.js
@@ -83,7 +83,7 @@
this._reset();
this.updateStatus(ls`Loading\u2026`);
- const parsedURL = this._inspectedURL.asParsedURL();
+ const parsedURL = Common.ParsedURL.fromString(this._inspectedURL);
const pageHost = parsedURL && parsedURL.host;
const statusHeader = pageHost ? ls`Auditing ${pageHost}` : ls`Auditing your web page`;
this._renderStatusHeader(statusHeader);