12619 2007-02-05 13:05:15 -0800 Non-abbreviated XPaths don't work 2007-02-15 01:06:34 -0800 1 1 1 Unclassified WebKit XML 420+ Mac OS X 10.4 RESOLVED FIXED P2 Normal --- 12584 1 ap ap oldest_to_newest 27329 0 ap 2007-02-05 13:05:15 -0800 It seems to be an issue in the parser that garbage is sent to XPath::Step ctor instead of correct Axis ids. 27330 1 12941 ap 2007-02-05 13:06:07 -0800 Created attachment 12941 test case 27338 2 ap 2007-02-05 13:40:00 -0800 I have a fix for this issue (which is caused by a missing break in a switch statement in Parser::lex()), but my test case still doesn't work correctly. WebKit and Firefox/Opera seem to have different ideas about what a root node is (in Firefox, it's the document element, while in WebKit, it's the Document itself). Going to investigate this further. 26892 3 12960 ap 2007-02-05 21:48:39 -0800 Created attachment 12960 proposed fix 26893 4 12960 ggaren 2007-02-05 21:50:57 -0800 Comment on attachment 12960 proposed fix r=me 26894 5 ap 2007-02-05 21:53:19 -0800 Committed revision 19425. 12941 2007-02-05 13:06:07 -0800 2007-02-05 13:06:07 -0800 test case ancestor-axis.html text/html 627 ap PGh0bWw+Cjxib2R5Pgo8cD5UZXN0IGZvciA8YSBocmVmPSJodHRwOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD0xMjYxOSI+YnVnIDEyNjE5PC9hPjoKTm9uLWFiYnJldmlhdGVkIFhQ YXRocyBkb24ndCB3b3JrLjwvcD4KCiAgPGRpdiBpZD0iYSI+PC9kaXY+Cgo8c2NyaXB0PgogIGlm ICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCiAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5k dW1wQXNUZXh0KCk7CgogIHRyeSB7CiAgICByZXN1bHQgPSBkb2N1bWVudC5ldmFsdWF0ZSgiLy9k aXYvYW5jZXN0b3I6OioiLCBkb2N1bWVudC5ib2R5LCBudWxsLCBYUGF0aFJlc3VsdC5VTk9SREVS RURfTk9ERV9TTkFQU0hPVF9UWVBFLCBudWxsKTsKCiAgICBpZiAoMiA9PSByZXN1bHQuc25hcHNo b3RMZW5ndGgpCiAgICAgIGRvY3VtZW50LndyaXRlKCJTVUNDRVNTIik7CiAgICBlbHNlCiAgICAg IGRvY3VtZW50LndyaXRlKCJGQUlMVVJFOiAiICsgcmVzdWx0LnNuYXBzaG90TGVuZ3RoICsgIiBy ZXN1bHQgbm9kZXMgKHNob3VsZCBiZSAyKSIpOwogIH0gY2F0Y2ggKGV4KSB7CiAgICBkb2N1bWVu dC53cml0ZSgiRkFJTFVSRTogIiArIGV4KTsKICB9Cjwvc2NyaXB0Pgo8L2JvZHk+CjwvaHRtbD4K 12960 2007-02-05 21:48:39 -0800 2007-02-05 21:50:57 -0800 proposed fix 12619r1_patch.txt text/plain 3987 ap SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHJldmlzaW9uIDE5NDI0KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMDctMDItMDYgIEFsZXhleSBQcm9za3VyeWFr b3YgIDxhcEB3ZWJraXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp LgorCisgICAgICAgIGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEyNjE5 CisgICAgICAgIE5vbi1hYmJyZXZpYXRlZCBYUGF0aHMgZG9uJ3Qgd29yaworCisgICAgICAgICog ZmFzdC94cGF0aC9hbmNlc3Rvci1heGlzLWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICog ZmFzdC94cGF0aC9hbmNlc3Rvci1heGlzLmh0bWw6IEFkZGVkLgorCiAyMDA3LTAyLTA1ICBEYXZp ZCBLaWx6ZXIgIDxkZGtpbHplckB3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERh cmluLgpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC94cGF0aC9hbmNlc3Rvci1heGlzLWV4cGVjdGVk LnR4dAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L3hwYXRoL2FuY2VzdG9yLWF4aXMt ZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvZmFzdC94cGF0aC9hbmNl c3Rvci1heGlzLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwzIEBACitUZXN0 IGZvciBidWcgMTI2MTk6IE5vbi1hYmJyZXZpYXRlZCBYUGF0aHMgZG9uJ3Qgd29yay4KKworU1VD Q0VTUwoKUHJvcGVydHkgY2hhbmdlcyBvbjogTGF5b3V0VGVzdHMvZmFzdC94cGF0aC9hbmNlc3Rv ci1heGlzLWV4cGVjdGVkLnR4dApfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCk5hbWU6IHN2bjptaW1lLXR5cGUKICAgKyB0 ZXh0L3BsYWluCk5hbWU6IHN2bjplb2wtc3R5bGUKICAgKyBuYXRpdmUKCkluZGV4OiBMYXlvdXRU ZXN0cy9mYXN0L3hwYXRoL2FuY2VzdG9yLWF4aXMuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU ZXN0cy9mYXN0L3hwYXRoL2FuY2VzdG9yLWF4aXMuaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91 dFRlc3RzL2Zhc3QveHBhdGgvYW5jZXN0b3ItYXhpcy5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCww ICsxLDI0IEBACis8aHRtbD4KKzxib2R5PgorPHA+VGVzdCBmb3IgPGEgaHJlZj0iaHR0cDovL2J1 Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI2MTkiPmJ1ZyAxMjYxOTwvYT46CitOb24t YWJicmV2aWF0ZWQgWFBhdGhzIGRvbid0IHdvcmsuPC9wPgorCisgIDxkaXYgaWQ9ImEiPjwvZGl2 PgorCis8c2NyaXB0PgorICBpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgIGxh eW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsKKworICB0cnkgeworICAgIHJlc3VsdCA9 IGRvY3VtZW50LmV2YWx1YXRlKCIvL2Rpdi9hbmNlc3Rvcjo6KiIsIGRvY3VtZW50LmJvZHksIG51 bGwsIFhQYXRoUmVzdWx0LlVOT1JERVJFRF9OT0RFX1NOQVBTSE9UX1RZUEUsIG51bGwpOworCisg ICAgaWYgKDIgPT0gcmVzdWx0LnNuYXBzaG90TGVuZ3RoKQorICAgICAgZG9jdW1lbnQud3JpdGUo IlNVQ0NFU1MiKTsKKyAgICBlbHNlCisgICAgICBkb2N1bWVudC53cml0ZSgiRkFJTFVSRTogIiAr IHJlc3VsdC5zbmFwc2hvdExlbmd0aCArICIgcmVzdWx0IG5vZGVzIChzaG91bGQgYmUgMikiKTsK KyAgfSBjYXRjaCAoZXgpIHsKKyAgICBkb2N1bWVudC53cml0ZSgiRkFJTFVSRTogIiArIGV4KTsK KyAgfQorPC9zY3JpcHQ+Cis8L2JvZHk+Cis8L2h0bWw+CgpQcm9wZXJ0eSBjaGFuZ2VzIG9uOiBM YXlvdXRUZXN0cy9mYXN0L3hwYXRoL2FuY2VzdG9yLWF4aXMuaHRtbApfX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCk5hbWU6 IHN2bjptaW1lLXR5cGUKICAgKyB0ZXh0L2h0bWwKCkluZGV4OiBXZWJDb3JlL0NoYW5nZUxvZwo9 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09Ci0tLSBXZWJDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTk0MjQpCisrKyBXZWJD b3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDA3LTAyLTA2 ICBBbGV4ZXkgUHJvc2t1cnlha292ICA8YXBAd2Via2l0Lm9yZz4KKworICAgICAgICBSZXZpZXdl ZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBodHRwOi8vYnVncy53ZWJraXQub3JnL3No b3dfYnVnLmNnaT9pZD0xMjYxOQorICAgICAgICBOb24tYWJicmV2aWF0ZWQgWFBhdGhzIGRvbid0 IHdvcmsKKworICAgICAgICBUZXN0OiBmYXN0L3hwYXRoL2FuY2VzdG9yLWF4aXMuaHRtbAorCisg ICAgICAgICogeG1sL1hQYXRoUGFyc2VyLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlhQYXRoOjpQ YXJzZXI6OmxleCk6IEZpeCBheGlzIG5hbWUgcGFyc2luZyB0byBhY3R1YWxseSB3b3JrLgorICAg ICAgICAqIHhtbC9YUGF0aFN0ZXAuY3BwOgorICAgICAgICAoV2ViQ29yZTo6WFBhdGg6OlN0ZXA6 Om5vZGVUZXN0TWF0Y2hlcyk6IEZpeCAiKiIgbm9kZSB0ZXN0IHRvIGFjdHVhbGx5IHdvcmsuCisK IDIwMDctMDItMDYgIE1hcmsgUm93ZSAgPG1yb3dlQGFwcGxlLmNvbT4KIAogICAgICAgICBSb2xs IG91dCBpbmNvbXBsZXRlIHN1cHBvcnQgZm9yIGZvbnQtc3RyZXRjaCAocjE5MzUwKSBhdCBEYXZl IEh5YXR0J3MgcmVxdWVzdC4KSW5kZXg6IFdlYkNvcmUveG1sL1hQYXRoUGFyc2VyLmNwcAo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBXZWJDb3JlL3htbC9YUGF0aFBhcnNlci5jcHAJKHJldmlzaW9uIDE5NDA2KQor KysgV2ViQ29yZS94bWwvWFBhdGhQYXJzZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC00MjMsNiAr NDIzLDcgQEAgaW50IFBhcnNlcjo6bGV4KHZvaWQqIGRhdGEpCiAgICAgc3dpdGNoICh0b2sudHlw ZSkgewogICAgICAgICBjYXNlIEFYSVNOQU1FOgogICAgICAgICAgICAgeXlsdmFsLT5heGlzID0g dG9rLmF4aXM7CisgICAgICAgICAgICBicmVhazsKICAgICAgICAgY2FzZSBNVUxPUDoKICAgICAg ICAgY2FzZSBSRUxPUDoKICAgICAgICAgICAgIHl5bHZhbC0+bnVtb3AgPSB0b2subnVtb3A7Cklu ZGV4OiBXZWJDb3JlL3htbC9YUGF0aFN0ZXAuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUveG1s L1hQYXRoU3RlcC5jcHAJKHJldmlzaW9uIDE5NDA2KQorKysgV2ViQ29yZS94bWwvWFBhdGhTdGVw LmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTk4LDcgKzE5OCw3IEBAIE5vZGVWZWN0b3IgU3RlcDo6 bm9kZVRlc3RNYXRjaGVzKGNvbnN0IE4KICAgICAgICAgICAgICAgICAobV9uYW1lc3BhY2VVUkku aXNFbXB0eSgpIHx8IG1fbmFtZXNwYWNlVVJJID09IG5vZGUtPm5hbWVzcGFjZVVSSSgpKSkKICAg ICAgICAgICAgICAgICBtYXRjaGVzLmFwcGVuZChub2RlKTsKICAgICAgICAgfQotICAgICAgICBy ZXR1cm4gbm9kZXM7CisgICAgICAgIHJldHVybiBtYXRjaGVzOwogICAgIH0gZWxzZSBpZiAobV9u b2RlVGVzdCA9PSAidGV4dCgpIikgewogICAgICAgICBIYXNoU2V0PE5vZGUqPiBub2RlU2V0Owog ICAgICAgICBmb3IgKHVuc2lnbmVkIGkgPSAwOyBpIDwgbm9kZXMuc2l6ZSgpOyBpKyspIHsK