13009 2007-03-07 23:17:49 -0800 REGRESSION (r20042): Console spews "CGContextGetType: invalid context" non-stop on web site 2007-03-08 19:15:53 -0800 1 1 1 Unclassified WebKit Layout and Rendering 523.x (Safari 3) Mac OS X 10.4 RESOLVED FIXED http://tf1.lci.fr/ InRadar, NeedsReduction, Regression P1 Normal --- 1 ddkilzer webkit-unassigned bruceq darin dev+webkit mrowe oldest_to_newest 19876 0 ddkilzer 2007-03-07 23:17:49 -0800 Summary: The following message spews non-stop to the console when the URL (http://tf1.lci.fr/) is loaded: CGContextGetType: invalid context This is a fairly recent regression (within the last 24 hours). Steps to reproduce: 1. Open WebKit debug build. 2. Access URL: http://tf1.lci.fr/ 3. Wait for page to load. Expected results: No "CGContextGetType: invalid context" messages should be printed to the console. Actual results: The "CGContextGetType: invalid context" messages are printed to the console non-stop until the page is closed. Regression: This wasn't happening with debug builds less than 24 hours ago. 19877 1 ddkilzer 2007-03-07 23:20:22 -0800 Note that the URL is from Bug 12980. 19880 2 slewis 2007-03-07 23:38:41 -0800 <rdar://problem/5048788> 19881 3 ddkilzer 2007-03-07 23:51:09 -0800 Of course, I can't reproduce it now. Probably a particular ad was being displayed. If anyone sees this again, please save a webarchive of the page. 19884 4 ddkilzer 2007-03-08 00:05:17 -0800 (In reply to comment #3) > Of course, I can't reproduce it now. Probably a particular ad was being > displayed. I was missing a step in the steps to reproduce: 1. Open WebKit debug build. 2. Access URL: http://tf1.lci.fr/ 3. Wait for page to load. 4. Hide Safari. As soon as Safari is hidden, the message starts spewing to the console and never stops until the browser window is closed. 19885 5 ddkilzer 2007-03-08 00:19:30 -0800 This is a regression from r20042 which was the fix for Bug 12515. 19889 6 ddkilzer 2007-03-08 00:59:12 -0800 I'm also seeing these console messages sporadically (on this URL, for example: http://www.flickr.com/search/?q=cute+kittens&s=int): Not a JPEG file: starts with 0x7b 0x35 Not a JPEG file: starts with 0x7b 0x35 Not a JPEG file: starts with 0x7b 0xbd Not a JPEG file: starts with 0x7b 0x35 Not a JPEG file: starts with 0x7b 0xcb Not a JPEG file: starts with 0x7b 0x35 Not a JPEG file: starts with 0x7b 0x35 Not a JPEG file: starts with 0x7b 0x35 Not a JPEG file: starts with 0x7b 0x35 I'm not sure if this is caused by r20042 or not, though. 19892 7 ddkilzer 2007-03-08 01:33:18 -0800 (In reply to comment #6) > I'm also seeing these console messages sporadically (on this URL, for example: > http://www.flickr.com/search/?q=cute+kittens&s=int): > > Not a JPEG file: starts with 0x7b 0x35 > [...] > > I'm not sure if this is caused by r20042 or not, though. This appears to have been a different issue. Please ignore it. 19893 8 ddkilzer 2007-03-08 01:35:04 -0800 (In reply to comment #7) > This appears to have been a different issue. Please ignore it. And is fixed as of r20057. 19930 9 dev+webkit 2007-03-08 14:03:14 -0800 (In reply to comment #4) > > As soon as Safari is hidden, the message starts spewing to the console and > never stops until the browser window is closed. > The website http://weplayanything.com spews this before anything is even rendered, without having to hide Safari. 19937 10 13549 bruceq 2007-03-08 14:45:56 -0800 Created attachment 13549 Checks for null ptr before calling WKCGContextIsBitmapContext. A fairly trivial (low-risk) fix for this issue 19938 11 13549 darin 2007-03-08 14:47:08 -0800 Comment on attachment 13549 Checks for null ptr before calling WKCGContextIsBitmapContext. r=me Do you know why there's a null context? 19833 12 mrowe 2007-03-08 19:15:53 -0800 Landed in r20077. 13549 2007-03-08 14:45:56 -0800 2007-03-08 14:47:08 -0800 Checks for null ptr before calling WKCGContextIsBitmapContext. CGContextGetType_console_spew_patch.txt text/plain 2015 bruceq SW5kZXg6IFdlYktpdC9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViS2l0L0NoYW5nZUxvZwko cmV2aXNpb24gMjAwNjcpCisrKyBXZWJLaXQvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0x LDMgKzEsMTQgQEAKKzIwMDctMDMtMDggIEJydWNlIFEgSGFtbW9uZCAgPGJydWNlcUBhcHBsZS5v bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGaXgg Zm9yIGJ1ZyBpZCAxMzAwOSAtIENvbnNvbGUgc3Bld3MKKyAgICAgICAgICAgICJDR0NvbnRleHRH ZXRUeXBlOiBpbnZhbGlkIGNvbnRleHQiIG5vbi1zdG9wIG9uIHdlYiBzaXRlCisKKyAgICAgICAg KiBQbHVnaW5zL1dlYkJhc2VOZXRzY2FwZVBsdWdpblZpZXcubW06CisgICAgICAgICgtW1dlYkJh c2VOZXRzY2FwZVBsdWdpblZpZXcgc2F2ZUFuZFNldE5ld1BvcnRTdGF0ZUZvclVwZGF0ZTpdKToK KyAgICAgICAgKC1bV2ViQmFzZU5ldHNjYXBlUGx1Z2luVmlldyByZXN0b3JlUG9ydFN0YXRlOl0p OgorCiAyMDA3LTAzLTA4ICBUaW1vdGh5IEhhdGNoZXIgIDx0aW1vdGh5QGFwcGxlLmNvbT4KIAog ICAgICAgICBSZXZpZXdlZCBieSBKb2huLgpJbmRleDogV2ViS2l0L1BsdWdpbnMvV2ViQmFzZU5l dHNjYXBlUGx1Z2luVmlldy5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJLaXQvUGx1Z2lucy9XZWJCYXNl TmV0c2NhcGVQbHVnaW5WaWV3Lm1tCShyZXZpc2lvbiAyMDA2NykKKysrIFdlYktpdC9QbHVnaW5z L1dlYkJhc2VOZXRzY2FwZVBsdWdpblZpZXcubW0JKHdvcmtpbmcgY29weSkKQEAgLTM5Nyw3ICsz OTcsNyBAQCBzdGF0aWMgVUludDMyIFFEUGl4ZWxGb3JtYXRGcm9tQ0dCaXRtYXBJCiAKICAgICAg ICAgICAgIENHQ29udGV4dFJlZiBjdXJyZW50Q29udGV4dCA9IChDR0NvbnRleHRSZWYpW1tOU0dy YXBoaWNzQ29udGV4dCBjdXJyZW50Q29udGV4dF0gZ3JhcGhpY3NQb3J0XTsKICAgICAgICAgICAg IC8vIElmIHRoZSBjdXJyZW50IGNvbnRleHQgaXMgYW4gb2Zmc2NyZWVuIGJpdG1hcCwgdGhlbiB3 ZSBjcmVhdGUgYSBHV29ybGQgZm9yIGl0Ci0gICAgICAgICAgICBib29sIG9mZlNjcmVlbkNvbnRl eHQgPSBXS0NHQ29udGV4dElzQml0bWFwQ29udGV4dChjdXJyZW50Q29udGV4dCk7CisgICAgICAg ICAgICBib29sIG9mZlNjcmVlbkNvbnRleHQgPSBjdXJyZW50Q29udGV4dCAmJiBXS0NHQ29udGV4 dElzQml0bWFwQ29udGV4dChjdXJyZW50Q29udGV4dCk7CiAgICAgICAgICAgICBpZiAob2ZmU2Ny ZWVuQ29udGV4dCkgewogICAgICAgICAgICAgICAgIENHQml0bWFwSW5mbyBjb250ZXh0Qml0bWFw SW5mbyA9IENHQml0bWFwQ29udGV4dEdldEJpdG1hcEluZm8oY3VycmVudENvbnRleHQpOwogICAg ICAgICAgICAgICAgIEdXb3JsZFB0ciBwT2ZmU2NyZWVuR1dvcmxkOwpAQCAtNjAyLDcgKzYwMiw3 IEBAIHN0YXRpYyBVSW50MzIgUURQaXhlbEZvcm1hdEZyb21DR0JpdG1hcEkKICAgICAgICAgICAg IAogICAgICAgICAgICAgLy8gSWYgdGhlIGN1cnJlbnQgY29udGV4dCBpcyBhbiBvZmZzY3JlZW4g Yml0bWFwLCB0aGVuIHdlIG5lZWQgdG8KICAgICAgICAgICAgIC8vIGRpc3Bvc2UgaXRzIEdXb3Js ZCBhbmQgcmVzdG9yZSB0aGUgV2luZG93J3MgR1dvcmxkCi0gICAgICAgICAgICBpZiAoV0tDR0Nv bnRleHRJc0JpdG1hcENvbnRleHQoY3VycmVudENvbnRleHQpKSB7CisgICAgICAgICAgICBpZiAo Y3VycmVudENvbnRleHQgJiYgV0tDR0NvbnRleHRJc0JpdG1hcENvbnRleHQoY3VycmVudENvbnRl eHQpKSB7CiAgICAgICAgICAgICAgICAgR1dvcmxkUHRyIGN1ckdXb3JsZDsKICAgICAgICAgICAg ICAgICBHZXRHV29ybGQoJmN1ckdXb3JsZCwgTlVMTCk7CiAgICAgICAgICAgICAgICAgRGlzcG9z ZUdXb3JsZChjdXJHV29ybGQpOwo=