146254 2015-06-23 15:33:14 -0700 Web Inspector: TextEditor scroll position not correctly restored when switching tabs 2015-06-23 17:28:21 -0700 1 1 1 Unclassified WebKit Web Inspector 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 joepeck joepeck commit-queue graouts joepeck jonowells mattbaker nvasilyev timothy webkit-bug-importer oldest_to_newest 1104208 0 joepeck 2015-06-23 15:33:14 -0700 * SUMMARY TextEditor scroll position not correctly restored when switching tabs. * STEPS TO REPRODUCE 1. Load <http://apple.com> 2. Show page source (Web Inspector, Resources Tab, Main Resource selected) 3. Scroll down the text editor 4. Switch to the Elements tab 5. Switch back to the Resources tab => correct resource is showing, but scroll position is now (0,0) instead of what it was before switching 1104209 1 joepeck 2015-06-23 15:33:25 -0700 <rdar://problem/21176592> 1104221 2 255443 joepeck 2015-06-23 15:50:33 -0700 Created attachment 255443 [PATCH] Proposed Fix 1104260 3 255443 commit-queue 2015-06-23 17:28:18 -0700 Comment on attachment 255443 [PATCH] Proposed Fix Clearing flags on attachment: 255443 Committed r185891: <http://trac.webkit.org/changeset/185891> 1104261 4 commit-queue 2015-06-23 17:28:21 -0700 All reviewed patches have been landed. Closing bug. 255443 2015-06-23 15:50:33 -0700 2015-06-23 17:28:18 -0700 [PATCH] Proposed Fix scroll-position.patch text/plain 1767 joepeck ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IGZlNTQ4MGIuLmZmYjFmZjggMTAwNjQ0Ci0tLSBh L1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkluc3BlY3Rv clVJL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE3IEBACiAyMDE1LTA2LTIzICBKb3NlcGggUGVjb3Jh cm8gIDxwZWNvcmFyb0BhcHBsZS5jb20+CiAKKyAgICAgICAgV2ViIEluc3BlY3RvcjogVGV4dEVk aXRvciBzY3JvbGwgcG9zaXRpb24gbm90IGNvcnJlY3RseSByZXN0b3JlZCB3aGVuIHN3aXRjaGlu ZyB0YWJzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0x NDYyNTQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAq IFVzZXJJbnRlcmZhY2UvVmlld3MvVGV4dEVkaXRvci5qczoKKyAgICAgICAgKFdlYkluc3BlY3Rv ci5UZXh0RWRpdG9yLnByb3RvdHlwZS51cGRhdGVMYXlvdXQpOgorICAgICAgICBXb3JrYXJvdW5k IGEgbGFyZ2VyIGlzc3VlIHdpdGggQ29udGVudFZpZXcgcmVzdG9yYXRpb24gc28gdGhhdAorICAg ICAgICBUZXh0RWRpdG9ycyByZXN0b3JlIHRoZWlyIHNjcm9sbCBwb3NpdGlvbiBhcyBleHBlY3Rl ZC4KKworMjAxNS0wNi0yMyAgSm9zZXBoIFBlY29yYXJvICA8cGVjb3Jhcm9AYXBwbGUuY29tPgor CiAgICAgICAgIFdlYiBJbnNwZWN0b3I6IFNvbWUgYnJpZWYgcHJldmlld3MgYXJlIGluY29ycmVj dGx5IHRyZWF0ZWQgYXMgbG9zc2xlc3MKICAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv c2hvd19idWcuY2dpP2lkPTE0NjI0NwogCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViSW5zcGVjdG9y VUkvVXNlckludGVyZmFjZS9WaWV3cy9UZXh0RWRpdG9yLmpzIGIvU291cmNlL1dlYkluc3BlY3Rv clVJL1VzZXJJbnRlcmZhY2UvVmlld3MvVGV4dEVkaXRvci5qcwppbmRleCBkMzM1NzQzLi4zOGYz OGY3IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVyZmFjZS9WaWV3 cy9UZXh0RWRpdG9yLmpzCisrKyBiL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNl L1ZpZXdzL1RleHRFZGl0b3IuanMKQEAgLTUwNiw3ICs1MDYsMTEgQEAgV2ViSW5zcGVjdG9yLlRl eHRFZGl0b3IgPSBjbGFzcyBUZXh0RWRpdG9yIGV4dGVuZHMgV2ViSW5zcGVjdG9yLk9iamVjdAog CiAgICAgdXBkYXRlTGF5b3V0KGZvcmNlKQogICAgIHsKLSAgICAgICAgdGhpcy5fY29kZU1pcnJv ci5yZWZyZXNoKCk7CisgICAgICAgIC8vIEZJWE1FOiA8aHR0cHM6Ly93ZWJraXQub3JnL2IvMTQ2 MjU2PiBXZWIgSW5zcGVjdG9yOiBOZXN0ZWQgQ29udGVudEJyb3dzZXJzIC8gQ29udGVudFZpZXdD b250YWluZXJzIGNhdXNlIHRvbyBtYW55IENvbnRlbnRWaWV3IHVwZGF0ZXMKKyAgICAgICAgLy8g SWRlYWxseSB3ZSB3b3VsZCBub3QgZ2V0IGFuIHVwZGF0ZUxheW91dCBjYWxsIGlmIHdlIGFyZSBu b3QgdmlzaWJsZS4gV2Ugc2hvdWxkIHJlc3RydWN0dXJlIENvbnRlbnRWaWV3CisgICAgICAgIC8v IHNob3cvaGlkZSByZXN0b3JhdGlvbiB0byByZWR1Y2UgZHVwbGljYXRlZCB3b3JrIGFuZCBzb2x2 ZSB0aGlzIGluIHRoZSBwcm9jZXNzLgorICAgICAgICBpZiAodGhpcy5fdmlzaWJsZSkKKyAgICAg ICAgICAgIHRoaXMuX2NvZGVNaXJyb3IucmVmcmVzaCgpOwogICAgIH0KIAogICAgIHNob3duKCkK