69674 2011-10-07 14:57:50 -0700 Web Inspector: HTTP headers merged incorrect in Chrome 2011-10-27 15:38:52 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 0 lsong vsevik apavlov bweinstein joepeck keishi loislo pfeldman pmuellr rik timothy vsevik webkit.review.bot yurys oldest_to_newest 480330 0 lsong 2011-10-07 14:57:50 -0700 Multiple headers are combined into one with the '\n' separator, instead of a comma. See: http://trac.webkit.org/browser/trunk/Source/WebKit/chromium/src/WebHTTPLoadInfo.cpp?rev=96981#L108 It is correctly combined with comma in Safari 5.1. HTTP 1.1 Section 4.2 It MUST be possible to combine the multiple header fields into one "field-name: field-value" pair, without changing the semantics of the message, by appending each subsequent field-value to the first, each separated by a comma. Look at the following Cache-Control header: Response Headersview source Cache-Control:no-cache private no-store must-revalidate max-stale=0 post-check=0 pre-check=0 Content-Length:4352 Content-Type:text/html; charset=utf-8 Date:Fri, 07 Oct 2011 21:40:46 GMT Expires:Mon, 26 Jul 1997 05:00:00 GMT P3P:P3P - policyref="http://www.adfusion.com/w3c/adfusion.xml", CP="NON DSP COR CURa TIA" Pragma:no-cache Server:Microsoft-IIS/6.0 Set-Cookie:AF=CID=6f698bbf-1b81-4c5e-ba2b-5134f86d0952; expires=Sat, 07-Apr-2012 21:40:46 GMT; path=/ X-AspNet-Version:2.0.50727 X-Powered-By:ASP.NET Response Headersview parsed HTTP/1.1 200 OK Date: Fri, 07 Oct 2011 21:40:46 GMT Server: Microsoft-IIS/6.0 P3P: P3P - policyref="http://www.adfusion.com/w3c/adfusion.xml", CP="NON DSP COR CURa TIA" X-Powered-By: ASP.NET X-AspNet-Version: 2.0.50727 Pragma: no-cache Set-Cookie: AF=CID=6f698bbf-1b81-4c5e-ba2b-5134f86d0952; expires=Sat, 07-Apr-2012 21:40:46 GMT; path=/ Cache-Control: no-cache Cache-Control: private Cache-Control: no-store Cache-Control: must-revalidate Cache-Control: max-stale=0 Cache-Control: post-check=0 Cache-Control: pre-check=0 Expires: Mon, 26 Jul 1997 05:00:00 GMT Content-Type: text/html; charset=utf-8 Content-Length: 4352 You can reproduce this on www.cnn.com, in the network panel, look for http://www.adfusion.com/adfusion.partnersite/categoryhtml.aspx?userfeedguid=c97e9baf-d238-42e0-bdd2-10cbd46e8b7c&pid=1&clickTag=http://ads.cnn.com/event.ng/Type%3dclick%26FlightID%3d352757%26AdID%3d484596%26TargetID%3d77299%26Segments%3d730,2247,2743,2823,3285,3430,9496,9779,9781,9853,10381,16113,17173,17251,18517,18982,19419,21497,30363,30550,30582,30708,31936,31939,32530,32594,32749,32922,33852,35306,35605,36376,38203,39377,39378,39416,39671,40253,41850,42838,45557,45604,45611,45799,46163,46694,47603,48618,48619,48897,49715,50069,50091,50778,50779,51392,51756,51876,51962,52195,52207,52256,52448,52649,52765,52922,53055,53151,53232,53347,53359,53423,53424,53478,53577,53582,53583,53595,53647,53835,53961,53974,54095,54114,54178,54251,54319,54404,54534,54567,54633,54637,54665,54668,54675,54746,54747,54867,54888,54895,54897%26Values%3d1588%26Redirect%3d 491797 1 112722 vsevik 2011-10-27 12:07:42 -0700 Created attachment 112722 Patch 492025 2 112722 webkit.review.bot 2011-10-27 15:38:48 -0700 Comment on attachment 112722 Patch Clearing flags on attachment: 112722 Committed r98651: <http://trac.webkit.org/changeset/98651> 492026 3 webkit.review.bot 2011-10-27 15:38:52 -0700 All reviewed patches have been landed. Closing bug. 112722 2011-10-27 12:07:42 -0700 2011-10-27 15:38:48 -0700 Patch bug-69674-20111027230740.patch text/plain 1473 vsevik U3VidmVyc2lvbiBSZXZpc2lvbjogOTg1ODAKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvY2hy b21pdW0vQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9jaHJvbWl1bS9DaGFuZ2VMb2cKaW5kZXgg NmEyZDZmNjVjMjEwMDViYmQ0MmRkMmM1YTNjMjYxYzAwMWU1ZTgyMC4uYWE1N2FiMjc3NDEwMzJi YjBiNTAzYzA2MmZkYmI5OTY2YjMzOGY1NiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9jaHJv bWl1bS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdC9jaHJvbWl1bS9DaGFuZ2VMb2cKQEAg LTEsMyArMSwxMyBAQAorMjAxMS0xMC0yNyAgVnNldm9sb2QgVmxhc292ICA8dnNldmlrQGNocm9t aXVtLm9yZz4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiBIVFRQIGhlYWRlcnMgbWVyZ2VkIGlu Y29ycmVjdCBpbiBDaHJvbWUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTY5Njc0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK KyAgICAgICAgKiBzcmMvV2ViSFRUUExvYWRJbmZvLmNwcDoKKyAgICAgICAgKFdlYktpdDo6YWRk SGVhZGVyKToKKwogMjAxMS0xMC0yNyAgRGF2ZSBNaWNoYWVsICA8ZG1pY2hhZWxAY2hyb21pdW0u b3JnPgogCiAgICAgICAgIENocm9taXVtIHVpX3Rlc3RzIFdvcmtlclRlc3QuV29ya2VyTWVzc2Fn ZVBvcnRbR0NdIHdlcmUgYnJva2VuIGJ5IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL2F0dGFjaG1l bnQuY2dpP2lkPTExMjM0MgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9jaHJvbWl1bS9zcmMv V2ViSFRUUExvYWRJbmZvLmNwcCBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vc3JjL1dlYkhUVFBM b2FkSW5mby5jcHAKaW5kZXggN2EwN2RkZTJjZmIxOWRkZDUxNjkyMDBmY2YyOTNmMmZhNWViNzlj ZC4uMDQyYzY2OWEyZWE2MjgyOTBkN2IwOWZmMjk4OWFjOGU5YmZiYzI0OCAxMDA2NDQKLS0tIGEv U291cmNlL1dlYktpdC9jaHJvbWl1bS9zcmMvV2ViSFRUUExvYWRJbmZvLmNwcAorKysgYi9Tb3Vy Y2UvV2ViS2l0L2Nocm9taXVtL3NyYy9XZWJIVFRQTG9hZEluZm8uY3BwCkBAIC0xMDUsNyArMTA1 LDcgQEAgc3RhdGljIHZvaWQgYWRkSGVhZGVyKEhUVFBIZWFkZXJNYXAqIG1hcCwgY29uc3QgV2Vi U3RyaW5nJiBuYW1lLCBjb25zdCBXZWJTdHJpbmcKIHsKICAgICBwYWlyPEhUVFBIZWFkZXJNYXA6 Oml0ZXJhdG9yLCBib29sPiByZXN1bHQgPSBtYXAtPmFkZChuYW1lLCB2YWx1ZSk7CiAgICAgaWYg KCFyZXN1bHQuc2Vjb25kKQotICAgICAgICByZXN1bHQuZmlyc3QtPnNlY29uZCArPSAiXG4iICsg U3RyaW5nKHZhbHVlKTsKKyAgICAgICAgcmVzdWx0LmZpcnN0LT5zZWNvbmQgKz0gIiwgIiArIFN0 cmluZyh2YWx1ZSk7CiB9CiAKIHZvaWQgV2ViSFRUUExvYWRJbmZvOjphZGRSZXF1ZXN0SGVhZGVy KGNvbnN0IFdlYlN0cmluZyYgbmFtZSwgY29uc3QgV2ViU3RyaW5nJiB2YWx1ZSkK