diff options
author | finnur@google.com <finnur@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-08-06 03:14:11 +0000 |
---|---|---|
committer | finnur@google.com <finnur@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-08-06 03:14:11 +0000 |
commit | e5d478012a5ee0765d35fd62e5090d07f5ae7dde (patch) | |
tree | cd5a7a4113961f155df38cafae9f807707ba5886 /chrome/renderer/render_widget.cc | |
parent | e7b055b2192125a85b07bcacca312d01357f9e96 (diff) | |
download | chromium_src-e5d478012a5ee0765d35fd62e5090d07f5ae7dde.zip chromium_src-e5d478012a5ee0765d35fd62e5090d07f5ae7dde.tar.gz chromium_src-e5d478012a5ee0765d35fd62e5090d07f5ae7dde.tar.bz2 |
Fix 1303709: Crash: Switching between tabs with a Find box open on both tabs nulls the Esc handler
This is a temporary fix until we fix this properly:
http://b/issue?id=1307173
This change makes sure that when we Register as the Esc handler, we store who was registered at the time the Find box was opened for the first time (as opposed to always restoring to what RegisterAccelerator tells us was registered before us).
Also, if we have already been unregistered, we don't restore the old value.
This prevents us from restoring to another tab's handler when the tab registering gets focus notification before the tab *un*registering receives blur notification (through FocusWillChange).
BUG=1303709
TEST=Manually perform the steps listed in the bug and make sure it doesn't crash. Also make sure that the browser's Esc handler still works after closing the Find box.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@421 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/render_widget.cc')
0 files changed, 0 insertions, 0 deletions