Comment #1 on issue 16147 by [email protected]: Rapid navigation between two bookmarks can cause tab to close http://code.google.com/p/chromium/issues/detail?id=16147
crbug.com/12745 is related to this issue. Both of these problems are related to one assumption in RenderViewHostManager that gets violated in two ways: The assumption is in ::ShouldClosePage, and it is this: a) If there is a pending_render_view_host_, then what's taking place is a cross-site-navigation. b) If there is no render_view_host_, then what's taking place is the tab being closed. (a) is violated in bug 12745, where a download from a chrome:// url leaves a pending_render_view_ around and subsequent attempts to shutdown the browser can fail because the RVHM thinks a cross-site-navigation is underway. This issue is additionally masked because it only occurs in cases where the current RV has an unload handler registered. Otherwise the browser would have done a quick shutdown. (b) is violated with this (16147) bug, where rapid navigations can cause a CancelPending() before a ShouldClose_ACK arrives at the RVHM, which results in the RVHM closing the tab, even though a cross-site-navigation *is* underway. Additionally, it seems to me that the RVHM's model is based on the idea that it is managing exactly one transition, but this bug shows that it may have to manage multiple navigations that come in before it can complete the first. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ Automated mail from issue updates at http://crbug.com/ Subscription options: http://groups.google.com/group/chromium-bugs -~----------~----~----~----~------~----~------~--~---
