37567 2010-04-14 01:34:45 -0700 REGRESSION: hyperlink text styled with ::first-letter text-transform shows up in wrong color 2010-05-11 23:05:04 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) PC OS X 10.5 RESOLVED FIXED http://dev.w3.org/html5/markup/figcaption.html#figcaption P2 Normal --- 1 mike hyatt hyatt oldest_to_newest 212295 0 mike 2010-04-14 01:34:45 -0700 r57509 1. visit http://dev.w3.org/html5/markup/figcaption.html#figcaption 2. click and follow the "Phrasing content" hyperlink 3. go back to http://dev.w3.org/html5/markup/figcaption.html#figcaption You should notice that the letter "P" is rendered in the default, non-visited-link color, while the remaining "hrasing content" text shows up as expected in visited-link color. relevant source excerpt: <p class="elem-mdl"> <span><a href="terminology.html#phrasing-content">phrasing content</a></span> accompanying CSS excerpt: .elem-mdl::first-letter { text-transform: uppercase; } I get expected behavior (all of the hyperlink text in visited-link color) in stable Safari and in all other browsers tested (Minefield, latest Chrome dev-channel for OSX). 212617 1 53375 hyatt 2010-04-14 15:29:56 -0700 Created attachment 53375 This fixes the bug. I need to write a test case. I noticed a couple of other bugs with pseudo elements and :visited though besides this one. I'll fix those in a followup. 212930 2 hyatt 2010-04-15 10:56:12 -0700 I'd like to write a good test for this, but https://bugs.webkit.org/show_bug.cgi?id=24021 blocks me. I'm going to land testless and then see if I can't get 24021 ferried into the tree (the patch in the bug looks good to me). After that lands, pseudo element tests will be possible. 212932 3 53451 hyatt 2010-04-15 10:57:50 -0700 Created attachment 53451 Patch 212933 4 53451 andersca 2010-04-15 10:59:01 -0700 Comment on attachment 53451 Patch r=me 212942 5 hyatt 2010-04-15 11:22:44 -0700 Fixed in r57657. 224427 6 ap 2010-05-11 23:05:04 -0700 This has caused bug 38891. 53375 2010-04-14 15:29:56 -0700 2010-04-15 10:57:50 -0700 This fixes the bug. patch.txt text/plain 1690 hyatt SW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRlckJsb2NrLmNwcAo9PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBX ZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJCbG9jay5jcHAJKHJldmlzaW9uIDU3NDc5KQorKysgV2Vi Q29yZS9yZW5kZXJpbmcvUmVuZGVyQmxvY2suY3BwCSh3b3JraW5nIGNvcHkpCkBAIC00NzMyLDkg KzQ3MzIsNyBAQCB2b2lkIFJlbmRlckJsb2NrOjp1cGRhdGVGaXJzdExldHRlcigpCiAgICAgICAg ICAgICAvLyBjb25zdHJ1Y3QgdGV4dCBmcmFnbWVudCBmb3IgdGhlIGZpcnN0IGxldHRlcgogICAg ICAgICAgICAgUmVuZGVyVGV4dEZyYWdtZW50KiBsZXR0ZXIgPSAKICAgICAgICAgICAgICAgICBu ZXcgKHJlbmRlckFyZW5hKCkpIFJlbmRlclRleHRGcmFnbWVudChyZW1haW5pbmdUZXh0LT5ub2Rl KCkgPyByZW1haW5pbmdUZXh0LT5ub2RlKCkgOiByZW1haW5pbmdUZXh0LT5kb2N1bWVudCgpLCBv bGRUZXh0LmdldCgpLCAwLCBsZW5ndGgpOwotICAgICAgICAgICAgUmVmUHRyPFJlbmRlclN0eWxl PiBuZXdTdHlsZSA9IFJlbmRlclN0eWxlOjpjcmVhdGUoKTsKLSAgICAgICAgICAgIG5ld1N0eWxl LT5pbmhlcml0RnJvbShwc2V1ZG9TdHlsZSk7Ci0gICAgICAgICAgICBsZXR0ZXItPnNldFN0eWxl KG5ld1N0eWxlLnJlbGVhc2UoKSk7CisgICAgICAgICAgICBsZXR0ZXItPnNldFN0eWxlKHBzZXVk b1N0eWxlKTsKICAgICAgICAgICAgIGZpcnN0TGV0dGVyLT5hZGRDaGlsZChsZXR0ZXIpOwogCiAg ICAgICAgICAgICB0ZXh0T2JqLT5kZXN0cm95KCk7CkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9z dHlsZS9SZW5kZXJTdHlsZS5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9yZW5kZXJpbmcvc3R5 bGUvUmVuZGVyU3R5bGUuY3BwCShyZXZpc2lvbiA1NzUyOCkKKysrIFdlYkNvcmUvcmVuZGVyaW5n L3N0eWxlL1JlbmRlclN0eWxlLmNwcAkod29ya2luZyBjb3B5KQpAQCAtMjExLDggKzIxMSwxNyBA QCB2b2lkIFJlbmRlclN0eWxlOjpzZXRIYXNQc2V1ZG9TdHlsZShQc2V1CiAKIFJlbmRlclN0eWxl KiBSZW5kZXJTdHlsZTo6Z2V0Q2FjaGVkUHNldWRvU3R5bGUoUHNldWRvSWQgcGlkKSBjb25zdAog ewotICAgIGlmICghbV9jYWNoZWRQc2V1ZG9TdHlsZSB8fCBzdHlsZVR5cGUoKSAhPSBOT1BTRVVE TykKKyAgICBBU1NFUlQoc3R5bGVUeXBlKCkgIT0gVklTSVRFRF9MSU5LKTsKKworICAgIGlmICgh bV9jYWNoZWRQc2V1ZG9TdHlsZSkKKyAgICAgICAgcmV0dXJuIDA7CisKKyAgICBpZiAoc3R5bGVU eXBlKCkgIT0gTk9QU0VVRE8pIHsKKyAgICAgICAgaWYgKHBpZCA9PSBWSVNJVEVEX0xJTkspCisg ICAgICAgICAgICByZXR1cm4gbV9jYWNoZWRQc2V1ZG9TdHlsZS0+c3R5bGVUeXBlKCkgPT0gVklT SVRFRF9MSU5LID8gbV9jYWNoZWRQc2V1ZG9TdHlsZS5nZXQoKSA6IDA7CiAgICAgICAgIHJldHVy biAwOworICAgIH0KKwogICAgIFJlbmRlclN0eWxlKiBwcyA9IG1fY2FjaGVkUHNldWRvU3R5bGUu Z2V0KCk7CiAgICAgd2hpbGUgKHBzICYmIHBzLT5zdHlsZVR5cGUoKSAhPSBwaWQpCiAgICAgICAg IHBzID0gcHMtPm1fY2FjaGVkUHNldWRvU3R5bGUuZ2V0KCk7Cg== 53451 2010-04-15 10:57:50 -0700 2010-04-15 10:59:01 -0700 Patch patch.txt text/plain 2458 hyatt SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA1NzY1NCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTUgQEAKKzIwMTAtMDQtMTUgIERhdmlkIEh5YXR0ICA8aHlhdHRAYXBwbGUuY29t PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIGh0dHBz Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zNzU2NywgOmZpcnN0LWxldHRlciBp bnNpZGUgYSA6dmlzaXRlZCBsaW5rIGlzIHdyb25nIGNvbG9yLiAgTWFrZSBzdXJlCisgICAgICAg IHRoYXQgdGhlIHBzZXVkbyBzdHlsZSBjYWNoaW5nIGFsbG93cyB2aXNpdGVkIGxpbmsgc3R5bGVz IHRvIGhhbmcgb2ZmIG90aGVyIHBzZXVkbyBzdHlsZXMuCisKKyAgICAgICAgKiByZW5kZXJpbmcv UmVuZGVyQmxvY2suY3BwOgorICAgICAgICAoV2ViQ29yZTo6UmVuZGVyQmxvY2s6OnVwZGF0ZUZp cnN0TGV0dGVyKToKKyAgICAgICAgKiByZW5kZXJpbmcvc3R5bGUvUmVuZGVyU3R5bGUuY3BwOgor ICAgICAgICAoV2ViQ29yZTo6UmVuZGVyU3R5bGU6OmdldENhY2hlZFBzZXVkb1N0eWxlKToKKwog MjAxMC0wNC0xNSAgWWFhciBTY2huaXRtYW4gIDx5YWFyQGNocm9taXVtLm9yZz4KIAogICAgICAg ICBSZXZpZXdlZCBieSBOYXRlIENoYXBpbi4KSW5kZXg6IFdlYkNvcmUvcmVuZGVyaW5nL1JlbmRl ckJsb2NrLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJCbG9jay5j cHAJKHJldmlzaW9uIDU3NDc5KQorKysgV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyQmxvY2suY3Bw CSh3b3JraW5nIGNvcHkpCkBAIC00NzMyLDkgKzQ3MzIsNyBAQCB2b2lkIFJlbmRlckJsb2NrOjp1 cGRhdGVGaXJzdExldHRlcigpCiAgICAgICAgICAgICAvLyBjb25zdHJ1Y3QgdGV4dCBmcmFnbWVu dCBmb3IgdGhlIGZpcnN0IGxldHRlcgogICAgICAgICAgICAgUmVuZGVyVGV4dEZyYWdtZW50KiBs ZXR0ZXIgPSAKICAgICAgICAgICAgICAgICBuZXcgKHJlbmRlckFyZW5hKCkpIFJlbmRlclRleHRG cmFnbWVudChyZW1haW5pbmdUZXh0LT5ub2RlKCkgPyByZW1haW5pbmdUZXh0LT5ub2RlKCkgOiBy ZW1haW5pbmdUZXh0LT5kb2N1bWVudCgpLCBvbGRUZXh0LmdldCgpLCAwLCBsZW5ndGgpOwotICAg ICAgICAgICAgUmVmUHRyPFJlbmRlclN0eWxlPiBuZXdTdHlsZSA9IFJlbmRlclN0eWxlOjpjcmVh dGUoKTsKLSAgICAgICAgICAgIG5ld1N0eWxlLT5pbmhlcml0RnJvbShwc2V1ZG9TdHlsZSk7Ci0g ICAgICAgICAgICBsZXR0ZXItPnNldFN0eWxlKG5ld1N0eWxlLnJlbGVhc2UoKSk7CisgICAgICAg ICAgICBsZXR0ZXItPnNldFN0eWxlKHBzZXVkb1N0eWxlKTsKICAgICAgICAgICAgIGZpcnN0TGV0 dGVyLT5hZGRDaGlsZChsZXR0ZXIpOwogCiAgICAgICAgICAgICB0ZXh0T2JqLT5kZXN0cm95KCk7 CkluZGV4OiBXZWJDb3JlL3JlbmRlcmluZy9zdHlsZS9SZW5kZXJTdHlsZS5jcHAKPT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PQotLS0gV2ViQ29yZS9yZW5kZXJpbmcvc3R5bGUvUmVuZGVyU3R5bGUuY3BwCShyZXZpc2lvbiA1 NzUyOCkKKysrIFdlYkNvcmUvcmVuZGVyaW5nL3N0eWxlL1JlbmRlclN0eWxlLmNwcAkod29ya2lu ZyBjb3B5KQpAQCAtMjExLDggKzIxMSwxNyBAQCB2b2lkIFJlbmRlclN0eWxlOjpzZXRIYXNQc2V1 ZG9TdHlsZShQc2V1CiAKIFJlbmRlclN0eWxlKiBSZW5kZXJTdHlsZTo6Z2V0Q2FjaGVkUHNldWRv U3R5bGUoUHNldWRvSWQgcGlkKSBjb25zdAogewotICAgIGlmICghbV9jYWNoZWRQc2V1ZG9TdHls ZSB8fCBzdHlsZVR5cGUoKSAhPSBOT1BTRVVETykKKyAgICBBU1NFUlQoc3R5bGVUeXBlKCkgIT0g VklTSVRFRF9MSU5LKTsKKworICAgIGlmICghbV9jYWNoZWRQc2V1ZG9TdHlsZSkKKyAgICAgICAg cmV0dXJuIDA7CisKKyAgICBpZiAoc3R5bGVUeXBlKCkgIT0gTk9QU0VVRE8pIHsKKyAgICAgICAg aWYgKHBpZCA9PSBWSVNJVEVEX0xJTkspCisgICAgICAgICAgICByZXR1cm4gbV9jYWNoZWRQc2V1 ZG9TdHlsZS0+c3R5bGVUeXBlKCkgPT0gVklTSVRFRF9MSU5LID8gbV9jYWNoZWRQc2V1ZG9TdHls ZS5nZXQoKSA6IDA7CiAgICAgICAgIHJldHVybiAwOworICAgIH0KKwogICAgIFJlbmRlclN0eWxl KiBwcyA9IG1fY2FjaGVkUHNldWRvU3R5bGUuZ2V0KCk7CiAgICAgd2hpbGUgKHBzICYmIHBzLT5z dHlsZVR5cGUoKSAhPSBwaWQpCiAgICAgICAgIHBzID0gcHMtPm1fY2FjaGVkUHNldWRvU3R5bGUu Z2V0KCk7Cg==