67732 2011-09-07 14:12:00 -0700 [Chromium] Web Inspector: Bug with console.log and popstate/hashchange events 2011-09-27 05:40:57 -0700 1 1 1 Unclassified WebKit History 528+ (Nightly build) PC Windows XP RESOLVED FIXED http://jsbin.com/owuyel P2 Minor --- 1 zachleatherman yurys apavlov bweinstein eric fishd joepeck keishi loislo pfeldman pmuellr rik timothy yurys oldest_to_newest 463641 0 106634 zachleatherman 2011-09-07 14:12:00 -0700 Created attachment 106634 Global i holds the number of times popstate has fired. If I bind a simple event handler to the popstate event with a console.log, it will execute the console.log multiple times per onpopstate event. The onpopstate event fires only once, it just triggers console.log multiple times. Looks like the same for hashchange. Chrome 14.0.835.157 beta-m 473550 1 eric 2011-09-26 17:05:48 -0700 I'm told Darin is mr. popstate (but I could be misinformed). 473552 2 eric 2011-09-26 17:06:40 -0700 I see. So this isn't a popstate bug, this is an interaction with the web inspector? In which case, Pavel is your man. 473746 3 yurys 2011-09-27 00:35:14 -0700 Looks like a bug in web inspector. After navigation two original messages are printed before the actual PopStateEvent. 473800 4 108817 yurys 2011-09-27 02:19:38 -0700 Created attachment 108817 Reduced test case This is a Chromium-specific issue. The problem here is that when back button is pushed we receive OnNavigatingToPendingEntry in the browser and send Detach followed by Attach to the corresponding DevToolsAgent to make sure our front-end is connected to correct RenderViewHost. In case of navigation back to the same URL with a different fragment we probably shouldn't do this as RVH doesn't change in that case. 473860 5 yurys 2011-09-27 05:09:28 -0700 Filed Chromium bug: http://code.google.com/p/chromium/issues/detail?id=98210 473861 6 108827 yurys 2011-09-27 05:11:47 -0700 Created attachment 108827 Patch 473875 7 yurys 2011-09-27 05:39:28 -0700 Committed r96100: <http://trac.webkit.org/changeset/96100> 473877 8 yurys 2011-09-27 05:40:57 -0700 Chromium patch will be landed after WebKit r96100 is rolled. 106634 2011-09-07 14:12:00 -0700 2011-09-07 14:12:00 -0700 Global i holds the number of times popstate has fired. popstate-test.html text/html 689 zachleatherman PCFET0NUWVBFIGh0bWw+IA0KPGh0bWw+IA0KICAgIDxoZWFkPg0KICAgICAgICA8c2NyaXB0Pg0K ICAgICAgICAoZnVuY3Rpb24oKSB7DQogICAgICAgICAgICB3aW5kb3cub25oYXNoY2hhbmdlID0g ZnVuY3Rpb24oZSkNCiAgICAgICAgICAgIHsNCiAgICAgICAgICAgICAgICBjb25zb2xlLmxvZygn aGFzaGNoYW5nZScsIGUsIGxvY2F0aW9uLmhhc2gpOw0KICAgICAgICAgICAgfTsNCg0KICAgICAg ICAgICAgaSA9IDA7DQogICAgICAgICAgICB3aW5kb3cub25wb3BzdGF0ZSA9IGZ1bmN0aW9uKGUp DQogICAgICAgICAgICB7DQogICAgICAgICAgICAgICAgaSsrOw0KICAgICAgICAgICAgICAgIGNv bnNvbGUubG9nKCdwb3BzdGF0ZScsIGUsIGUudGltZVN0YW1wLCBuZXcgRGF0ZSk7DQogICAgICAg ICAgICB9Ow0KICAgICAgICB9KSgpOw0KICAgICAgICA8L3NjcmlwdD4NCiAgICA8L2hlYWQ+DQo8 Ym9keT4NCiAgICA8b2w+DQogICAgICAgIDxsaT5PcGVuIGNvbnNvbGUuPC9saT4NCiAgICAgICAg PGxpPkNsaWNrIHRoaXM6IDxhIGhyZWY9IiNvbmUiPiNvbmU8L2E+PC9saT4NCiAgICAgICAgPGxp PlB1c2ggdGhlIGJhY2sgYnV0dG9uLjwvbGk+DQogICAgICAgIDxsaT5Ib3cgbWFueSBwb3BzdGF0 ZSBsb2cgc3RhdGVtZW50cyBkbyB5b3Ugc2VlPzwvbGk+DQogICAgPC9vbD4NCjwvYm9keT4NCjwv aHRtbD4= 108817 2011-09-27 02:19:38 -0700 2011-09-27 02:19:38 -0700 Reduced test case back-navigation.html text/html 227 yurys PHNjcmlwdD4NCiAgY29uc29sZS5sb2coIkxvYWRlZCAiICsgRGF0ZS5ub3coKSk7DQo8L3Njcmlw dD4NCjxvbD4NCiAgPGxpPk9wZW4gY29uc29sZS48L2xpPg0KICA8bGk+Q2xpY2sgdGhpczogPGEg aHJlZj0iI29uZSI+I29uZTwvYT48L2xpPg0KICA8bGk+UHVzaCB0aGUgYmFjayBidXR0b24uPC9s aT4NCiAgPGxpPkhvdyBtYW55IGxvZyByZWNvcmRzIGRvIHlvdSBzZWU/PC9saT4NCjwvb2w+DQo= 108827 2011-09-27 05:11:47 -0700 2011-09-27 05:31:30 -0700 Patch bug-67732-20110927161145.patch text/plain 2894 yurys U3VidmVyc2lvbiBSZXZpc2lvbjogOTU5MzcKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvY2hy b21pdW0vQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9jaHJvbWl1bS9DaGFuZ2VMb2cKaW5kZXgg MzQyOWE2NWM5MTcwYWU3ZGI5OWNhOTljOWFhZmY5YzBiZGYzNTlkZC4uMDhiNTljMWE4Yzg2YjA5 YjUxZjM4MmZjNmVjNTFmMGFhMjY1MzU1YSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9jaHJv bWl1bS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdC9jaHJvbWl1bS9DaGFuZ2VMb2cKQEAg LTEsMyArMSwxOSBAQAorMjAxMS0wOS0yNyAgWXVyeSBTZW1pa2hhdHNreSAgPHl1cnlzQGNocm9t aXVtLm9yZz4KKworICAgICAgICBbQ2hyb21pdW1dIFdlYiBJbnNwZWN0b3I6IEJ1ZyB3aXRoIGNv bnNvbGUubG9nIGFuZCBwb3BzdGF0ZS9oYXNoY2hhbmdlIGV2ZW50cworICAgICAgICBodHRwczov L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Njc3MzIKKworICAgICAgICBBZGRlZCBX ZWJLaXQgcGFydCBvZiBhIG5ldyBpbnRlcmFjdGl2ZSBVSSB0ZXN0IGZvciB0aGUgYnVnIHdpdGgg ZHVwbGljYXRlZCBjb25zb2xlIG1lc3NhZ2VzIGFmdGVyIG5hdmlnYXRpb24gYmFjay4KKworICAg ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIHNyYy9qcy9UZXN0 cy5qczoKKyAgICAgICAgKC5UZXN0U3VpdGUucHJvdG90eXBlLnRlc3RDb25zb2xlT25OYXZpZ2F0 ZUJhY2suZmlyc3RDb25zb2xlTWVzc2FnZVJlY2VpdmVkKToKKyAgICAgICAgKC5UZXN0U3VpdGUu cHJvdG90eXBlLnRlc3RDb25zb2xlT25OYXZpZ2F0ZUJhY2suZGlkQ2xpY2tMaW5rKToKKyAgICAg ICAgKC5UZXN0U3VpdGUucHJvdG90eXBlLnRlc3RDb25zb2xlT25OYXZpZ2F0ZUJhY2suZGlkTmF2 aWdhdGVCYWNrKToKKyAgICAgICAgKC5UZXN0U3VpdGUucHJvdG90eXBlLnRlc3RDb25zb2xlT25O YXZpZ2F0ZUJhY2suZGlkQ29tcGxldGVOYXZpZ2F0aW9uKToKKyAgICAgICAgKC5UZXN0U3VpdGUu cHJvdG90eXBlLnRlc3RDb25zb2xlT25OYXZpZ2F0ZUJhY2spOgorCiAyMDExLTA5LTI0ICBBZGFt IEJhcnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CiAKICAgICAgICAgQWx3YXlzIGVuYWJsZSBFTkFC TEUoT0ZGTElORV9XRUJfQVBQTElDQVRJT05TKQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9j aHJvbWl1bS9zcmMvanMvVGVzdHMuanMgYi9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3NyYy9qcy9U ZXN0cy5qcwppbmRleCAzYzZkYWUwYjVmNzI4ZjMwNGFjOGIxZjZlODY1OTQxMGZiMmU4NGY4Li5h MTI1MGY1NDVhYzZhMTYwMzEyNTM1OWE0ZTcyOTRiODE0OWE5ZjEwIDEwMDY0NAotLS0gYS9Tb3Vy Y2UvV2ViS2l0L2Nocm9taXVtL3NyYy9qcy9UZXN0cy5qcworKysgYi9Tb3VyY2UvV2ViS2l0L2No cm9taXVtL3NyYy9qcy9UZXN0cy5qcwpAQCAtNTM2LDYgKzUzNiw0MCBAQCBUZXN0U3VpdGUucHJv dG90eXBlLnRlc3ROZXR3b3JrVGltaW5nID0gZnVuY3Rpb24oKQogfTsKIAogCitUZXN0U3VpdGUu cHJvdG90eXBlLnRlc3RDb25zb2xlT25OYXZpZ2F0ZUJhY2sgPSBmdW5jdGlvbigpCit7CisgICAg aWYgKFdlYkluc3BlY3Rvci5jb25zb2xlLm1lc3NhZ2VzLmxlbmd0aCA9PT0gMSkKKyAgICAgICAg Zmlyc3RDb25zb2xlTWVzc2FnZVJlY2VpdmVkLmNhbGwodGhpcyk7CisgICAgZWxzZQorICAgICAg ICBXZWJJbnNwZWN0b3IuY29uc29sZS5hZGRFdmVudExpc3RlbmVyKFdlYkluc3BlY3Rvci5Db25z b2xlTW9kZWwuRXZlbnRzLk1lc3NhZ2VBZGRlZCwgZmlyc3RDb25zb2xlTWVzc2FnZVJlY2VpdmVk LCB0aGlzKTsKKworICAgIGZ1bmN0aW9uIGZpcnN0Q29uc29sZU1lc3NhZ2VSZWNlaXZlZCgpIHsK KyAgICAgICAgdGhpcy5ldmFsdWF0ZUluQ29uc29sZV8oImNsaWNrTGluaygpOyIsIGRpZENsaWNr TGluay5iaW5kKHRoaXMpKTsKKyAgICB9CisKKyAgICBmdW5jdGlvbiBkaWRDbGlja0xpbmsoKSB7 CisgICAgICAgIC8vIENoZWNrIHRoYXQgdGhlcmUgYXJlIG5vIG5ldyBtZXNzYWdlcyhjb21tYW5k IGlzIG5vdCBhIG1lc3NhZ2UpLgorICAgICAgICB0aGlzLmFzc2VydEVxdWFscygxLCBXZWJJbnNw ZWN0b3IuY29uc29sZS5tZXNzYWdlcy5sZW5ndGgpOworICAgICAgICB0aGlzLmFzc2VydEVxdWFs cygxLCBXZWJJbnNwZWN0b3IuY29uc29sZS5tZXNzYWdlc1swXS50b3RhbFJlcGVhdENvdW50KTsK KyAgICAgICAgdGhpcy5ldmFsdWF0ZUluQ29uc29sZV8oImhpc3RvcnkuYmFjaygpOyIsIGRpZE5h dmlnYXRlQmFjay5iaW5kKHRoaXMpKTsKKyAgICB9CisKKyAgICBmdW5jdGlvbiBkaWROYXZpZ2F0 ZUJhY2soKQorICAgIHsKKyAgICAgICAgLy8gTWFrZSBzdXJlIG5hdmlnYXRpb24gY29tcGxldGVk IGFuZCBwb3NzaWJsZSBjb25zb2xlIG1lc3NhZ2VzIHdlcmUgcHVzaGVkLgorICAgICAgICB0aGlz LmV2YWx1YXRlSW5Db25zb2xlXygidm9pZCAwOyIsIGRpZENvbXBsZXRlTmF2aWdhdGlvbi5iaW5k KHRoaXMpKTsKKyAgICB9CisKKyAgICBmdW5jdGlvbiBkaWRDb21wbGV0ZU5hdmlnYXRpb24oKSB7 CisgICAgICAgIHRoaXMuYXNzZXJ0RXF1YWxzKDEsIFdlYkluc3BlY3Rvci5jb25zb2xlLm1lc3Nh Z2VzLmxlbmd0aCk7CisgICAgICAgIHRoaXMuYXNzZXJ0RXF1YWxzKDEsIFdlYkluc3BlY3Rvci5j b25zb2xlLm1lc3NhZ2VzWzBdLnRvdGFsUmVwZWF0Q291bnQpOworICAgICAgICB0aGlzLnJlbGVh c2VDb250cm9sKCk7CisgICAgfQorCisgICAgdGhpcy50YWtlQ29udHJvbCgpOworfTsKKworCiBU ZXN0U3VpdGUucHJvdG90eXBlLnRlc3RTaGFyZWRXb3JrZXIgPSBmdW5jdGlvbigpCiB7CiAgICAg ZnVuY3Rpb24gZGlkRXZhbHVhdGVJbkNvbnNvbGUocmVzdWx0VGV4dCkgewo=