20117 2008-07-20 12:46:35 -0700 setBaseAndExtent fails to reverse the current selection 2008-07-21 22:52:34 -0700 1 1 1 Unclassified WebKit DOM 525.x (Safari 3.1) All All RESOLVED FIXED http://nick-santos.com/tests/rangetest.html P4 Normal --- 1 nicksantos ap justin.garcia oldest_to_newest 86490 0 nicksantos 2008-07-20 12:46:35 -0700 Repro steps: 1) Create a range with Javascript and set the base and extent as: range.setBaseAndExtent(start, 0, end, 0); and select it. 2) Creacte a range with Javascript and set the base and extent as: range.setBaseAndExtent(end, 0, start, 0); and select it. Expected result: The current selection should be reversed: end-to-start. Actual result: The current selection does not change when the second range is selected. See the attached link for a test case. Notice that if you clear the selection between the two steps, everything works as expected. 86549 1 22401 ap 2008-07-21 03:55:22 -0700 Created attachment 22401 proposed fix 86572 2 22401 mitz 2008-07-21 08:31:18 -0700 Comment on attachment 22401 proposed fix r=me 86623 3 ap 2008-07-21 22:52:34 -0700 Committed revision 35284. 22401 2008-07-21 03:55:22 -0700 2008-07-21 08:31:18 -0700 proposed fix 20117r1_patch.txt text/plain 4241 ap SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAzNTI2MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTQgQEAKKzIwMDgtMDctMjEgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEB3ZWJr aXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDExNworICAgICAgICBz ZXRCYXNlQW5kRXh0ZW50IGZhaWxzIHRvIHJldmVyc2UgdGhlIGN1cnJlbnQgc2VsZWN0aW9uCisK KyAgICAgICAgVGVzdDogZWRpdGluZy9zZWxlY3Rpb24vc2V0QmFzZUFuZEV4dGVudC1yZXZlcnQt c2VsZWN0aW9uLmh0bWwKKworICAgICAgICAqIGVkaXRpbmcvU2VsZWN0aW9uLmg6IChXZWJDb3Jl OjpvcGVyYXRvcj09KTogQ29tcGFyZSBkaXJlY3Rpb24sIHRvby4KKwogMjAwOC0wNy0yMSAgQW5h dG9saSBQYXBpcm92c2tpICA8YXBhcGlyb3Zza2lAbWFjLmNvbT4KIAogICAgICAgICBSZXZpZXdl ZCBieSBEYXZlIEh5YXR0LgpJbmRleDogV2ViQ29yZS9lZGl0aW5nL1NlbGVjdGlvbi5oCj09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT0KLS0tIFdlYkNvcmUvZWRpdGluZy9TZWxlY3Rpb24uaAkocmV2aXNpb24gMzUyNTApCisr KyBXZWJDb3JlL2VkaXRpbmcvU2VsZWN0aW9uLmgJKHdvcmtpbmcgY29weSkKQEAgLTExNSw3ICsx MTUsNyBAQCBwcml2YXRlOgogCiBpbmxpbmUgYm9vbCBvcGVyYXRvcj09KGNvbnN0IFNlbGVjdGlv biYgYSwgY29uc3QgU2VsZWN0aW9uJiBiKQogewotICAgIHJldHVybiBhLnN0YXJ0KCkgPT0gYi5z dGFydCgpICYmIGEuZW5kKCkgPT0gYi5lbmQoKSAmJiBhLmFmZmluaXR5KCkgPT0gYi5hZmZpbml0 eSgpICYmIGEuZ3JhbnVsYXJpdHkoKSA9PSBiLmdyYW51bGFyaXR5KCk7CisgICAgcmV0dXJuIGEu c3RhcnQoKSA9PSBiLnN0YXJ0KCkgJiYgYS5lbmQoKSA9PSBiLmVuZCgpICYmIGEuYWZmaW5pdHko KSA9PSBiLmFmZmluaXR5KCkgJiYgYS5ncmFudWxhcml0eSgpID09IGIuZ3JhbnVsYXJpdHkoKSAm JiBhLmlzQmFzZUZpcnN0KCkgPT0gYi5pc0Jhc2VGaXJzdCgpOwogfQogCiBpbmxpbmUgYm9vbCBv cGVyYXRvciE9KGNvbnN0IFNlbGVjdGlvbiYgYSwgY29uc3QgU2VsZWN0aW9uJiBiKQpJbmRleDog TGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL0NoYW5nZUxv ZwkocmV2aXNpb24gMzUyNjEpCisrKyBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29w eSkKQEAgLTEsMyArMSwxMyBAQAorMjAwOC0wNy0yMSAgQWxleGV5IFByb3NrdXJ5YWtvdiAgPGFw QHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIwMTE3CisgICAg ICAgIHNldEJhc2VBbmRFeHRlbnQgZmFpbHMgdG8gcmV2ZXJzZSB0aGUgY3VycmVudCBzZWxlY3Rp b24KKworICAgICAgICAqIGVkaXRpbmcvc2VsZWN0aW9uL3NldEJhc2VBbmRFeHRlbnQtcmV2ZXJ0 LXNlbGVjdGlvbi1leHBlY3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAqIGVkaXRpbmcvc2VsZWN0 aW9uL3NldEJhc2VBbmRFeHRlbnQtcmV2ZXJ0LXNlbGVjdGlvbi5odG1sOiBBZGRlZC4KKwogMjAw OC0wNy0yMSAgQW5hdG9saSBQYXBpcm92c2tpICA8YXBhcGlyb3Zza2lAbWFjLmNvbT4KIAogICAg ICAgICBSZXZpZXdlZCBieSBEYXZlIEh5YXR0LgpJbmRleDogTGF5b3V0VGVzdHMvZWRpdGluZy9z ZWxlY3Rpb24vc2V0QmFzZUFuZEV4dGVudC1yZXZlcnQtc2VsZWN0aW9uLWV4cGVjdGVkLnR4dAo9 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9lZGl0aW5nL3NlbGVjdGlvbi9zZXRCYXNlQW5kRXh0 ZW50LXJldmVydC1zZWxlY3Rpb24tZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0 VGVzdHMvZWRpdGluZy9zZWxlY3Rpb24vc2V0QmFzZUFuZEV4dGVudC1yZXZlcnQtc2VsZWN0aW9u LWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSw2IEBACitUZXN0IGZvciBidWcg MjAxMTc6IHNldEJhc2VBbmRFeHRlbnQgZmFpbHMgdG8gcmV2ZXJzZSB0aGUgY3VycmVudCBzZWxl Y3Rpb24uCisKK0RpdiBYCitEaXYgWQorRGl2IFoKK1BBU1MKClByb3BlcnR5IGNoYW5nZXMgb246 IExheW91dFRlc3RzL2VkaXRpbmcvc2VsZWN0aW9uL3NldEJhc2VBbmRFeHRlbnQtcmV2ZXJ0LXNl bGVjdGlvbi1leHBlY3RlZC50eHQKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpOYW1lOiBzdm46bWltZS10eXBlCiAgICsg dGV4dC9wbGFpbgpOYW1lOiBzdm46ZW9sLXN0eWxlCiAgICsgbmF0aXZlCgpJbmRleDogTGF5b3V0 VGVzdHMvZWRpdGluZy9zZWxlY3Rpb24vc2V0QmFzZUFuZEV4dGVudC1yZXZlcnQtc2VsZWN0aW9u Lmh0bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZWRpdGluZy9zZWxlY3Rpb24vc2V0QmFz ZUFuZEV4dGVudC1yZXZlcnQtc2VsZWN0aW9uLmh0bWwJKHJldmlzaW9uIDApCisrKyBMYXlvdXRU ZXN0cy9lZGl0aW5nL3NlbGVjdGlvbi9zZXRCYXNlQW5kRXh0ZW50LXJldmVydC1zZWxlY3Rpb24u aHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwzMyBAQAorPGh0bWw+CisgIDxoZWFkPgorICAg IDxzY3JpcHQ+CisgICAgICB2YXIgb3V0cHV0ID0gJyc7CisgICAKKyAgICAgIGZ1bmN0aW9uIHNl bGVjdE5vZGVzKGEsIGIpIHsKKyAgICAgICAgd2luZG93LmdldFNlbGVjdGlvbigpLnNldEJhc2VB bmRFeHRlbnQoYSwgMSwgYiwgMSk7CisgICAgICAgIG91dHB1dCArPSB3aW5kb3cuZ2V0U2VsZWN0 aW9uKCkuYW5jaG9yTm9kZS5wYXJlbnROb2RlLmlkICsgJyAnOyAgIAorICAgICAgfQorCisgICAg ICBpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgICAgICBsYXlvdXRUZXN0Q29u dHJvbGxlci5kdW1wQXNUZXh0KCk7CisKKyAgICAgIGZ1bmN0aW9uIGRvSXQoKSB7CisgICAgICAg IHRyeSB7CisgICAgICAgICAgdmFyIHggPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgneCcpOwor ICAgICAgICAgIHZhciB5ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ3knKTsKKyAgICAgICAg ICBzZWxlY3ROb2Rlcyh4LCB5KTsKKyAgICAgICAgICBzZWxlY3ROb2Rlcyh5LCB4KTsKKyAgICAg ICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgncmVzdWx0JykuaW5uZXJIVE1MID0gKG91dHB1 dCA9PSAieCB5ICIgPyAiUEFTUyIgOiAoIkZBSUw6ICIgKyBvdXRwdXQpKTsKKyAgICAgICAgfSBj YXRjaCAoZSkgeyBhbGVydChlKTsgfQorICAgICAgfQorICAgIDwvc2NyaXB0PgorICA8L2hlYWQ+ CisgIDxib2R5IG9ubG9hZD0iZG9JdCgpIj4KKyAgICA8cD5UZXN0IGZvciA8YSBocmVmPSJodHRw czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjAxMTciPmJ1ZyAyMDExNzwvYT46 CisgICAgc2V0QmFzZUFuZEV4dGVudCBmYWlscyB0byByZXZlcnNlIHRoZSBjdXJyZW50IHNlbGVj dGlvbi48L3A+CisgICAgPGRpdiBpZD0neCc+RGl2IFg8L2Rpdj4KKyAgICA8ZGl2IGlkPSd5Jz5E aXYgWTwvZGl2PgorICAgIDxkaXYgaWQ9J3onPkRpdiBaPC9kaXY+CisgICAgPGRpdiBpZD0ncmVz dWx0Jz5GQUlMIChzY3JpcHQgZGlkbid0IHJ1bik8L2Rpdj4KKyAgPC9ib2R5PgorPC9odG1sPgoK UHJvcGVydHkgY2hhbmdlcyBvbjogTGF5b3V0VGVzdHMvZWRpdGluZy9zZWxlY3Rpb24vc2V0QmFz ZUFuZEV4dGVudC1yZXZlcnQtc2VsZWN0aW9uLmh0bWwKX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpOYW1lOiBzdm46bWlt ZS10eXBlCiAgICsgdGV4dC9odG1sCgo=