Bug 201052 – Remove logging that contains a URL

• Component: WebKit Misc.
• Status: RESOLVED FIXED

[Keith Rollin]

checkURLReceivedFromWebProcess in WebProcessProxy.cpp contains an old logging line that logs a URL. We don't log URLs any more for privacy reasons, so remove this.

A search for WTFLogAlways.*url turns up other matches, but those are either false positives or cases where the URLs are logged only on demand by the developer as part of debugging. checkURLReceivedFromWebProcess is the only place where a URL is logged as a matter of course.

[Radar WebKit Bug Importer]

<rdar://problem/54613204>

[Keith Rollin]

Created attachment 377042 [details]
Patch

[Chris Dumez]

Comment on attachment 377042 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=377042&action=review

> Source/WebKit/UIProcess/WebProcessProxy.cpp:-561
> -    WTFLogAlways("Received an unexpected URL from the web process: '%s'\n", url.string().utf8().data());

Shouldn't we keep the logging though? How to we know now that's why the load is failing?

[Keith Rollin]

Created attachment 377043 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 377043 [details]
Patch

Clearing flags on attachment: 377043

Committed r249029: <https://trac.webkit.org/changeset/249029>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.