Bug 58794

Summary: Web Inspector: Hyperlink Auditing (ping) requests are not shown in network panel
Product: WebKit Reporter: Vsevolod Vlasov <vsevik>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, commit-queue, japhet, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch draft (No tests)
none
Patch
none
Patch (pings in DRT enabled)
eric: review+, commit-queue: commit-queue-
Archive of layout-test-results from cr-jail-4
none
Patch with fixes
none
Patch (rebaselined)
pfeldman: review+, commit-queue: commit-queue-
Patch (rebaselined again)
pfeldman: review+, commit-queue: commit-queue-
Patch (rebaselined) none

Description Vsevolod Vlasov 2011-04-18 09:53:39 PDT
Hyperlink Auditing (ping) requests are not shown in network panel.
http://www.whatwg.org/specs/web-apps/current-work/multipage/links.html#hyperlink-auditing
Comment 1 Vsevolod Vlasov 2011-04-18 09:55:58 PDT
Created attachment 90046 [details]
Patch draft (No tests)
Comment 2 Vsevolod Vlasov 2011-04-22 15:17:32 PDT
Created attachment 90775 [details]
Patch
Comment 3 Nate Chapin 2011-04-25 15:26:12 PDT
Comment on attachment 90775 [details]
Patch

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

This is awesome :)

I don't know the inspector well enough to review the implementation, but one quick test comment:

> LayoutTests/platform/chromium/test_expectations.txt:676
> +// Chromium DumpRenderTree does not send hyperlink auditing (ping) requests
> +WONTFIX SKIP : http/tests/inspector/network/ping.html = FAIL
> +

See http://trac.webkit.org/browser/trunk/LayoutTests/http/tests/navigation/ping-same-origin.html for how to enable pings in DRT.
Comment 4 Vsevolod Vlasov 2011-04-26 02:54:39 PDT
Created attachment 91085 [details]
Patch (pings in DRT enabled)

Thanks Nate,

This test works now in chromium now.
It is strange though that it worked in safari even without this setting enabled.
Comment 5 Nate Chapin 2011-04-26 12:56:58 PDT
(In reply to comment #4)
> Created an attachment (id=91085) [details]
> Patch (pings in DRT enabled)
> 
> Thanks Nate,
> 
> This test works now in chromium now.
> It is strange though that it worked in safari even without this setting enabled.

It appears the mac ports at least are now defaulting to hyperlink auditing enabled.  We should eventually switch chromium DRT's default setting to match chromium itself (though we should still force the preference on in this layout test, since not every platform defaults hyperlink auditing to on).
Comment 6 Eric Seidel (no email) 2011-04-26 16:20:14 PDT
Comment on attachment 91085 [details]
Patch (pings in DRT enabled)

This seems fine.
Comment 7 Joseph Pecoraro 2011-04-26 16:23:11 PDT
Comment on attachment 91085 [details]
Patch (pings in DRT enabled)

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

Neat patch.

> Source/WebCore/inspector/front-end/Resource.js:857
> +        return "text/ping" == this.requestContentType();

Nit (style): Almost always use "===" in JavaScript. We encourage it in Web Inspector code.
Comment 8 WebKit Commit Bot 2011-04-26 22:29:49 PDT
Comment on attachment 91085 [details]
Patch (pings in DRT enabled)

Rejecting attachment 91085 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-4', 'build-..." exit_code: 2

Last 500 characters of output:
ests/xmlhttprequest ................................................................................................................................................................................
http/tests/xmlhttprequest/web-apps ...............
http/tests/xmlhttprequest/workers ...........
http/tests/xmlviewer .
http/tests/xmlviewer/dumpAsText ............
736.44s total testing time

23391 test cases (99%) succeeded
1 test case (<1%) had incorrect layout
14 test cases (<1%) had stderr output

Full output: http://queues.webkit.org/results/8514094
Comment 9 WebKit Commit Bot 2011-04-26 22:29:53 PDT
Created attachment 91232 [details]
Archive of layout-test-results from cr-jail-4

The attached test failures were seen while running run-webkit-tests on the commit-queue.
Bot: cr-jail-4  Port: Mac  Platform: Mac OS X 10.6.6
Comment 10 Vsevolod Vlasov 2011-04-27 03:01:43 PDT
Created attachment 91261 [details]
Patch with fixes

This seems to fix it.
Comment 11 Vsevolod Vlasov 2011-04-28 03:29:36 PDT
Created attachment 91460 [details]
Patch (rebaselined)
Comment 12 WebKit Commit Bot 2011-04-28 10:50:58 PDT
Comment on attachment 91460 [details]
Patch (rebaselined)

Rejecting attachment 91460 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-8', 'apply-..." exit_code: 2

Last 500 characters of output:
h fuzz 3.
patching file LayoutTests/http/tests/inspector/network/ping-expected.txt
patching file LayoutTests/http/tests/inspector/network/ping.html
patching file Source/WebCore/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
Original content of Source/WebCore/English.lproj/localizedStrings.js mismatches at /mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply line 274.

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Pavel Feldman', u'--fo..." exit_code: 9

Full output: http://queues.webkit.org/results/8519443
Comment 13 Vsevolod Vlasov 2011-04-29 03:03:19 PDT
Created attachment 91658 [details]
Patch (rebaselined again)
Comment 14 WebKit Commit Bot 2011-04-29 03:06:14 PDT
Comment on attachment 91658 [details]
Patch (rebaselined again)

Rejecting attachment 91658 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-4', 'apply-..." exit_code: 2

Last 500 characters of output:
angeLog
patching file LayoutTests/http/tests/inspector/network/ping-expected.txt
patching file LayoutTests/http/tests/inspector/network/ping.html
patching file Source/WebCore/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
Original content of Source/WebCore/English.lproj/localizedStrings.js mismatches at /mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply line 274.

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Pavel Feldman', u'--fo..." exit_code: 255

Full output: http://queues.webkit.org/results/8514990
Comment 15 Vsevolod Vlasov 2011-04-29 04:16:15 PDT
Created attachment 91663 [details]
Patch (rebaselined)
Comment 16 WebKit Commit Bot 2011-04-29 04:43:40 PDT
Comment on attachment 91663 [details]
Patch (rebaselined)

Clearing flags on attachment: 91663

Committed r85316: <http://trac.webkit.org/changeset/85316>
Comment 17 WebKit Commit Bot 2011-04-29 04:43:47 PDT
All reviewed patches have been landed.  Closing bug.