7747 2006-03-12 18:43:46 -0800 REGRESSION: Background tab/window auto-refresh in GMail will take focus. 2006-03-22 16:03:17 -0800 1 1 1 Unclassified WebKit DOM 420+ Mac OS X 10.4 RESOLVED FIXED HasReduction, InRadar P1 Major --- 1 adele webkit-unassigned alice.barraclough darin ggaren jon mpeaston zynode oldest_to_newest 36057 0 adele 2006-03-12 18:43:46 -0800 This is not the same bug as http://bugzilla.opendarwin.org/enter_bug.cgi?product=WebKit "gmail website steals tab focus". This behavior has changed recently. It used to be that if you started to load GMail, and then went to another tab, that when it finished loading, GMail would take the focus (by calling window.focus). This is definitely different from Firefox but not nearly as annoying as the new behavior. Now, when GMail does refresh loads in a background tab or window, it also steals focus. So you can leave the tab/window after its fully loaded, go to another tab and do something else for a minute, and as the refresh in the GMail tab/window occurs, the focus will change out from under you. This does not occur in Safari 2.0.3. 36066 1 ddkilzer 2006-03-12 20:02:17 -0800 (In reply to comment #0) > This is not the same bug as > http://bugzilla.opendarwin.org/enter_bug.cgi?product=WebKit > "gmail website steals tab focus". That would be Bug 3779. :) 36117 2 adele 2006-03-13 11:41:06 -0800 This regression occurred between r13256 and r13261 36217 3 darin 2006-03-14 04:30:22 -0800 All the changes between those versions were things I did. Two bugs I fixed were bug 3308 and bug 7422 Here's a guess: It's possible this regression occurred because I fixed window.open() to properly find existing frames by name. Perhaps GMail calls focus() on a frame, but before it was having difficulty finding the frame. Thus I could have made the symptom of bug 3779 worse by fixing a bug in window.open(). If so, then it means the behavior before might have been better when pop-up blocking was enabled. Some ways to test this theory would be: 1) to change KJS::WindowFunc::callAsFunction to call tree()->child() instead of tree()->find() in the one place at the start related to pop-up blocking or 2) to test older versions of Safari with pop-up blocking turned off. 36553 4 jon 2006-03-16 07:44:09 -0800 I'm seeing this bug in a broader context. For example, at the Ars forums, clicking a link for a forum or topic will result in focus being shifting to the last tab or window in the order stack. So, open the Ars forums in a tab and then open another tab or window, which can have anything in it. Then go back to the Ars tab and click a forum or topic link. The view should switch to the new tab or window. Also, I've narrowed this down to the 13258 change as being the first version that will display this bug. 13257 doesn't build and 256 doesn't have this bug. I also think the severity of this bug should be increased since it affects multiple sites whenever you have multiple tabs or windows open. 36563 5 ggaren 2006-03-16 11:07:51 -0800 Bumping to major. 36599 6 alice.barraclough 2006-03-16 17:55:48 -0800 <rdar://problem/4481594> 37357 7 mitz 2006-03-22 11:54:19 -0800 *** Bug 7915 has been marked as a duplicate of this bug. *** 37359 8 7236 mitz 2006-03-22 12:24:27 -0800 Created attachment 7236 Reduction The problem seems to be very simple: FrameTree::find() does not match the null name with a frame that has a name. I think it should do just that. 37360 9 7237 mitz 2006-03-22 12:26:33 -0800 Created attachment 7237 Reduction Corrected link text. 37363 10 7238 mitz 2006-03-22 13:52:48 -0800 Created attachment 7238 Patch 37364 11 7238 adele 2006-03-22 13:59:26 -0800 Comment on attachment 7238 Patch nice work, mitz! r=me. 37376 12 mitz 2006-03-22 16:03:17 -0800 *** Bug 7920 has been marked as a duplicate of this bug. *** 7236 2006-03-22 12:24:27 -0800 2006-03-22 12:26:33 -0800 Reduction 7747 reduction.html text/html 250 mitz PHNjcmlwdD4KICAgIHdpbmRvdy5uYW1lID0gImZvbyI7Cjwvc2NyaXB0PgpPcGVuIGEgbmV3IHRh YiBvciB3aW5kb3cgYW5kIGxvYWQgYW4gSFRNTCBwYWdlIGluIGl0LCB0aGVuIGNvbWUgYmFjayBo ZXJlIGFuZCBjbGljayB0aGUgbGluay4KPGEgaHJlZj0iaHR0cDovL3d3dy53ZWJraXQub3JnLyI+ VGhpcyB3aWxsIG9wZW4gaW4gYW4gZXhpc3RpbmcsIGJ1dCBkaWZmZXJlbnQgdGFiL3dpbmRvdyAo aWYgdGhlcmUgaXMgb25lKS48L2E+Cg== 7237 2006-03-22 12:26:33 -0800 2006-03-22 12:26:33 -0800 Reduction 7747 reduction.html text/html 248 mitz PHNjcmlwdD4KICAgIHdpbmRvdy5uYW1lID0gImZvbyI7Cjwvc2NyaXB0PgpPcGVuIGEgbmV3IHRh YiBvciB3aW5kb3cgYW5kIGxvYWQgYW4gSFRNTCBwYWdlIGluIGl0LCB0aGVuIGNvbWUgYmFjayBo ZXJlIGFuZCBjbGljayB0aGUgbGluay4KPGEgaHJlZj0iaHR0cDovL3d3dy53ZWJraXQub3JnLyI+ VGhpcyB3aWxsIGZvY3VzIGFuIGV4aXN0aW5nLCBidXQgZGlmZmVyZW50IHRhYi93aW5kb3cgKGlm IHRoZXJlIGlzIG9uZSkuPC9hPgo= 7238 2006-03-22 13:52:48 -0800 2006-03-22 13:59:26 -0800 Patch 7747_r1.patch text/plain 5214 mitz SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAxMzQzNCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTggQEAKKzIwMDYtIyMtIyMgIE1pdHogUGV0dGVsICA8b3BlbmRhcndpbi5vcmdA bWl0enBldHRlbC5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK KyAgICAgICAgLSBmaXggaHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNn aT9pZD03NzQ3CisgICAgICAgICAgUkVHUkVTU0lPTjogQmFja2dyb3VuZCB0YWIvd2luZG93IGF1 dG8tcmVmcmVzaCBpbiBHTWFpbCB3aWxsIHRha2UgZm9jdXMuCisKKyAgICAgICAgKiBtYW51YWwt dGVzdHMvbmFtZWQtd2luZG93LWJsYW5rLXRhcmdldC5odG1sOiBBZGRlZC4KKyAgICAgICAgKiBt YW51YWwtdGVzdHMvcmVzb3VyY2VzL25hbWVkLXdpbmRvdy1ibGFuay10YXJnZXQtc3RlcDIuaHRt bDogQWRkZWQuCisgICAgICAgICogbWFudWFsLXRlc3RzL3Jlc291cmNlcy9uYW1lZC13aW5kb3ct YmxhbmstdGFyZ2V0LXN0ZXAzLmh0bWw6IEFkZGVkLgorICAgICAgICAqIG1hbnVhbC10ZXN0cy9y ZXNvdXJjZXMvbmFtZWQtd2luZG93LWJsYW5rLXRhcmdldC1zdGVwNC5odG1sOiBBZGRlZC4KKyAg ICAgICAgKiBwYWdlL0ZyYW1lVHJlZS5jcHA6CisgICAgICAgIChXZWJDb3JlOjpGcmFtZVRyZWU6 OmZpbmQpOiBJZiB0aGUgZ2l2ZW4gbmFtZSBpcyBlbXB0eSwganVzdCByZXR1cm4gb3VyIGZyYW1l LAorICAgICAgICBldmVuIGlmIGl0IGhhcyBhIG5hbWUuCisKIDIwMDYtMDMtMjIgIERhcmluIEFk bGVyICA8ZGFyaW5AYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEFkZWxlLgpJbmRl eDogV2ViQ29yZS9tYW51YWwtdGVzdHMvbmFtZWQtd2luZG93LWJsYW5rLXRhcmdldC5odG1sCj09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT0KLS0tIFdlYkNvcmUvbWFudWFsLXRlc3RzL25hbWVkLXdpbmRvdy1ibGFuay10YXJn ZXQuaHRtbAkocmV2aXNpb24gMCkKKysrIFdlYkNvcmUvbWFudWFsLXRlc3RzL25hbWVkLXdpbmRv dy1ibGFuay10YXJnZXQuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwxOSBAQAorPCFET0NU WVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIgorICAgICAgICAiaHR0 cDovL3d3dy53My5vcmcvVFIvaHRtbDQvc3RyaWN0LmR0ZCI+Cis8aHRtbCBsYW5nPSJlbiI+Cis8 aGVhZD4KKzxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0Ij4KKyAgICB3aW5kb3cubmFtZSA9 ICJmb28iOyAgICAvLyBhbnkgbm9uLWVtcHR5IG5hbWUgd2lsbCBkbyBoZXJlCis8L3NjcmlwdD4K KzwvaGVhZD4KKzxib2R5PgorPHA+PGI+QlVHIElEOjwvYj4gPGEgaHJlZj0iaHR0cDovL2J1Z3pp bGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNnaT9pZD03NzQ3Ij5CdWd6aWxsYSBidWcgNzc0 NzwvYT4gUkVHUkVTU0lPTjogQmFja2dyb3VuZCB0YWIvd2luZG93IGF1dG8tcmVmcmVzaCBpbiBH TWFpbCB3aWxsIHRha2UgZm9jdXMuPC9wPgorCis8cCBpZD0idGVzdCIgc3R5bGU9ImJhY2tncm91 bmQtY29sb3I6c2t5Ymx1ZTsgcGFkZGluZzozcHg7Ij48Yj5TVEVQUyBUTyBURVNUOjwvYj4gPGJy PgorU1RFUCAxOiBDbG9zZSBhbGwgb3RoZXIgU2FmYXJpIHdpbmRvd3MgYW5kIHRhYnMuPGJyPgor U1RFUCAyOiBPcGVuIDxhIGhyZWY9InJlc291cmNlcy9uYW1lZC13aW5kb3ctYmxhbmstdGFyZ2V0 LXN0ZXAyLmh0bWwiPnRoaXMgbGluazwvYT4gaW4gYSBuZXcgd2luZG93IGluIGZyb250IG9mIHRo aXMgd2luZG93IGFuZCBmb2xsb3cgdGhlIGluc3RydWN0aW9ucyBpbiBpdC4gICAKKzwvcD4KKwor Cis8L2JvZHk+Cis8L2h0bWw+CkluZGV4OiBXZWJDb3JlL21hbnVhbC10ZXN0cy9yZXNvdXJjZXMv bmFtZWQtd2luZG93LWJsYW5rLXRhcmdldC1zdGVwMi5odG1sCj09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNv cmUvbWFudWFsLXRlc3RzL3Jlc291cmNlcy9uYW1lZC13aW5kb3ctYmxhbmstdGFyZ2V0LXN0ZXAy Lmh0bWwJKHJldmlzaW9uIDApCisrKyBXZWJDb3JlL21hbnVhbC10ZXN0cy9yZXNvdXJjZXMvbmFt ZWQtd2luZG93LWJsYW5rLXRhcmdldC1zdGVwMi5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCwwICsx LDE5IEBACis8IURPQ1RZUEUgSFRNTCBQVUJMSUMgIi0vL1czQy8vRFREIEhUTUwgNC4wMS8vRU4i CisgICAgICAgICJodHRwOi8vd3d3LnczLm9yZy9UUi9odG1sNC9zdHJpY3QuZHRkIj4KKzxodG1s IGxhbmc9ImVuIj4KKzxoZWFkPgorPC9oZWFkPgorPGJvZHk+Cis8cD48Yj5CVUcgSUQ6PC9iPiA8 YSBocmVmPSJodHRwOi8vYnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hvd19idWcuY2dpP2lkPTc3 NDciPkJ1Z3ppbGxhIGJ1ZyA3NzQ3PC9hPiBSRUdSRVNTSU9OOiBCYWNrZ3JvdW5kIHRhYi93aW5k b3cgYXV0by1yZWZyZXNoIGluIEdNYWlsIHdpbGwgdGFrZSBmb2N1cy48L3A+CisKKzxwIGlkPSJ0 ZXN0IiBzdHlsZT0iYmFja2dyb3VuZC1jb2xvcjpza3libHVlOyBwYWRkaW5nOjNweDsiPjxiPlNU RVBTIFRPIFRFU1QgKGNvbnRpbnVlZCk6PC9iPiA8YnI+CitTVEVQIDM6IENsaWNrIDxhIHRhcmdl dD0iZm9vIiBocmVmPSJuYW1lZC13aW5kb3ctYmxhbmstdGFyZ2V0LXN0ZXAzLmh0bWwiPnRoaXMg bGluazwvYT4uCis8L3A+CisKKzxwIGlkPSJmYWlsdXJlIiBzdHlsZT0iYmFja2dyb3VuZC1jb2xv cjojRkYzMzAwOyBwYWRkaW5nOjNweDsiPjxiPlRFU1QgRkFJTDo8L2I+ICAKK0lmIHRoaXMgd2lu ZG93IGlzIGZyb250bW9zdCBhZnRlciBTVEVQIDQsIHRoZW4gdGhlIHRlc3QgRkFJTEVELgorPC9w PgorCisKKzwvYm9keT4KKzwvaHRtbD4KSW5kZXg6IFdlYkNvcmUvbWFudWFsLXRlc3RzL3Jlc291 cmNlcy9uYW1lZC13aW5kb3ctYmxhbmstdGFyZ2V0LXN0ZXAzLmh0bWwKPT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g V2ViQ29yZS9tYW51YWwtdGVzdHMvcmVzb3VyY2VzL25hbWVkLXdpbmRvdy1ibGFuay10YXJnZXQt c3RlcDMuaHRtbAkocmV2aXNpb24gMCkKKysrIFdlYkNvcmUvbWFudWFsLXRlc3RzL3Jlc291cmNl cy9uYW1lZC13aW5kb3ctYmxhbmstdGFyZ2V0LXN0ZXAzLmh0bWwJKHJldmlzaW9uIDApCkBAIC0w LDAgKzEsMTQgQEAKKzwhRE9DVFlQRSBIVE1MIFBVQkxJQyAiLS8vVzNDLy9EVEQgSFRNTCA0LjAx Ly9FTiIKKyAgICAgICAgImh0dHA6Ly93d3cudzMub3JnL1RSL2h0bWw0L3N0cmljdC5kdGQiPgor PGh0bWwgbGFuZz0iZW4iPgorPGhlYWQ+Cis8L2hlYWQ+Cis8Ym9keT4KKzxwPjxiPkJVRyBJRDo8 L2I+IDxhIGhyZWY9Imh0dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2luLm9yZy9zaG93X2J1Zy5jZ2k/ aWQ9Nzc0NyI+QnVnemlsbGEgYnVnIDc3NDc8L2E+IFJFR1JFU1NJT046IEJhY2tncm91bmQgdGFi L3dpbmRvdyBhdXRvLXJlZnJlc2ggaW4gR01haWwgd2lsbCB0YWtlIGZvY3VzLjwvcD4KKworPHAg aWQ9InRlc3QiIHN0eWxlPSJiYWNrZ3JvdW5kLWNvbG9yOnNreWJsdWU7IHBhZGRpbmc6M3B4OyI+ PGI+U1RFUFMgVE8gVEVTVDo8L2I+IDxicj4KK1NURVAgNDogQ2xpY2sgPGEgaHJlZj0ibmFtZWQt d2luZG93LWJsYW5rLXRhcmdldC1zdGVwNC5odG1sIj50aGlzIGxpbms8L2E+LiAgIAorPC9wPgor Cis8L2JvZHk+Cis8L2h0bWw+CkluZGV4OiBXZWJDb3JlL21hbnVhbC10ZXN0cy9yZXNvdXJjZXMv bmFtZWQtd2luZG93LWJsYW5rLXRhcmdldC1zdGVwNC5odG1sCj09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNv cmUvbWFudWFsLXRlc3RzL3Jlc291cmNlcy9uYW1lZC13aW5kb3ctYmxhbmstdGFyZ2V0LXN0ZXA0 Lmh0bWwJKHJldmlzaW9uIDApCisrKyBXZWJDb3JlL21hbnVhbC10ZXN0cy9yZXNvdXJjZXMvbmFt ZWQtd2luZG93LWJsYW5rLXRhcmdldC1zdGVwNC5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCwwICsx LDE0IEBACis8IURPQ1RZUEUgSFRNTCBQVUJMSUMgIi0vL1czQy8vRFREIEhUTUwgNC4wMS8vRU4i CisgICAgICAgICJodHRwOi8vd3d3LnczLm9yZy9UUi9odG1sNC9zdHJpY3QuZHRkIj4KKzxodG1s IGxhbmc9ImVuIj4KKzxoZWFkPgorPC9oZWFkPgorPGJvZHk+Cis8cD48Yj5CVUcgSUQ6PC9iPiA8 YSBocmVmPSJodHRwOi8vYnVnemlsbGEub3BlbmRhcndpbi5vcmcvc2hvd19idWcuY2dpP2lkPTc3 NDciPkJ1Z3ppbGxhIGJ1ZyA3NzQ3PC9hPiBSRUdSRVNTSU9OOiBCYWNrZ3JvdW5kIHRhYi93aW5k b3cgYXV0by1yZWZyZXNoIGluIEdNYWlsIHdpbGwgdGFrZSBmb2N1cy48L3A+CisKKzxwIGlkPSJz dWNjZXNzIiBzdHlsZT0iYmFja2dyb3VuZC1jb2xvcjpwYWxlZ3JlZW47IHBhZGRpbmc6M3B4OyI+ PGI+VEVTVCBQQVNTOjwvYj4gCitJZiB0aGlzIHdpbmRvdyBpcyBmcm9udG1vc3QgYWZ0ZXIgU1RF UCA0LCB0aGUgdGVzdCBQQVNTRUQuCis8L3A+CisKKzwvYm9keT4KKzwvaHRtbD4KSW5kZXg6IFdl YkNvcmUvcGFnZS9GcmFtZVRyZWUuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvcGFnZS9GcmFt ZVRyZWUuY3BwCShyZXZpc2lvbiAxMzQzNCkKKysrIFdlYkNvcmUvcGFnZS9GcmFtZVRyZWUuY3Bw CSh3b3JraW5nIGNvcHkpCkBAIC0xNjYsNyArMTY2LDcgQEAgRnJhbWUqIEZyYW1lVHJlZTo6Y2hp bGQoY29uc3QgQXRvbWljU3RyaQogCiBGcmFtZSogRnJhbWVUcmVlOjpmaW5kKGNvbnN0IEF0b21p Y1N0cmluZyYgbmFtZSkgY29uc3QKIHsKLSAgICBpZiAobmFtZSA9PSAiX3NlbGYiIHx8IG5hbWUg PT0gIl9jdXJyZW50IikKKyAgICBpZiAobmFtZSA9PSAiX3NlbGYiIHx8IG5hbWUgPT0gIl9jdXJy ZW50IiB8fCBuYW1lLmlzRW1wdHkoKSkKICAgICAgICAgcmV0dXJuIG1fdGhpc0ZyYW1lOwogICAg IAogICAgIGlmIChuYW1lID09ICJfdG9wIikK