Bug 209763 – Web Inspector: provide a way for inspector to turn on/off ITP debug mode and AdClickAttribution debug mode

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Devin Rousso]

Now that we send ITP debug logs and AdClickAttribution debug logs to Web Inspector, we should have a way for Web Inspector to turn those debug modes on/off in addition to (or instead of) the Debug menu toggle.

[Devin Rousso]

Created attachment 396147 [details]
[Patch] WIP

needs tests and ChangeLog entries, but the logic seems to be working and I believe is therefore reviewable :)

[EWS Watchlist]

This patch modifies the inspector protocol. Please ensure that any frontend changes appropriately use feature checks for new protocol features.

[Devin Rousso]

Created attachment 396168 [details]
[Patch] WIP

don't broadcast a console message if the state doesn't change

[Blaze Burg]

Comment on attachment 396168 [details]
[Patch] WIP

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

Looks good to me overall. Some things to finish per our discussion:
- Inspector tests for new settings
- Handling multiple inspected pages in NetworkProcess
- Recover from crashed inspected pages that turned on ITP debug mode

> Source/WebCore/inspector/InspectorClient.h:67
> +    enum class Preference {

Can we call these DeveloperPreferences just to not mix them up with Safari preferences or WebCore settings?

[Devin Rousso]

Created attachment 397920 [details]
Patch

[Devin Rousso]

Created attachment 397934 [details]
Patch

[John Wilander]

Does this automatically turn off ITP Debug Mode when the browser is quit/killed?

[Devin Rousso]

(In reply to John Wilander from comment #7)
> Does this automatically turn off ITP Debug Mode when the browser is quit/killed?
It should turn off ITP Debug Mode and Ad Click Attribution Debug Mode when Web Inspector closes.

Also, these paths shouldn't involve anything persistent (e.g. `NSUserDefaults`), so another "yes" as well :)

[Blaze Burg]

Comment on attachment 397934 [details]
Patch

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

r=me, please investigate Windows EWS failure:

  UnifiedSource-f74e0903-3.cpp
C:\cygwin\home\buildbot\worker\Windows-EWS\build\Source\WebCore\inspector/agents/InspectorPageAgent.cpp(464,1): error C2051: case expression not constant (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-6.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]

> Source/WebInspectorUI/ChangeLog:12
> +            Enable: [ ] ITP Debug Mode

If we have documentation for these things, then a link to that documentation from the settings page would be Awesome.

If not, shouldn't we document it?

[Devin Rousso]

Comment on attachment 397934 [details]
Patch

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

(In reply to Brian Burg from comment #9)
> r=me, please investigate Windows EWS failure:
> 
> UnifiedSource-f74e0903-3.cpp
> C:\cygwin\home\buildbot\worker\Windows-EWS\build\Source\WebCore\inspector/agents/InspectorPageAgent.cpp(464,1): error C2051: case expression not constant (compiling source file C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\DerivedSources\WebCore\unified-sources\UnifiedSource-84c9f43f-6.cpp) [C:\cygwin\home\buildbot\worker\Windows-EWS\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]

The windows bot doesn't seem to update/regenerate DerivedSources properly.  This has happened a bunch of times when I've made changes to the protocol.  A clean build has always fixed the issue.

>> Source/WebInspectorUI/ChangeLog:12
>> +            Enable: [ ] ITP Debug Mode
> 
> If we have documentation for these things, then a link to that documentation from the settings page would be Awesome.
> 
> If not, shouldn't we document it?

We have the old blog post for the Device Settings menu <https://webkit.org/changing-page-settings-on-ios-using-web-inspector/>, as well as the section about ITP debug mode <https://webkit.org/intelligent-tracking-prevention-2-3/> and Ad Click Attribution debugging <https://webkit.org/blog/8943/privacy-preserving-ad-click-attribution-for-the-web/>.

Ideally, we'd "combine" those three into a new reference page.  Perhaps I can write one once/before this is in the hands of actual users :)

[EWS]

Committed r261103: <https://trac.webkit.org/changeset/261103>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 397934 [details].

[Radar WebKit Bug Importer]

<rdar://problem/62857406>

[Alex Christensen]

Comment on attachment 397934 [details]
Patch

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

> Source/WebCore/inspector/agents/InspectorPageAgent.cpp:452
> +    case Inspector::Protocol::Page::Setting::AdClickAttributionDebugModeEnabled:

https://build.webkit.org/builders/Apple%20Win%2010%20Release%20%28Build%29/builds/13252/steps/compile-webkit/logs/stdio