rsms/chromium-tabs — Issue #10: A bug likely exists in CTTabWindowController
Author: enloyee
Created: 2011-12-03 15:26:18+00:00
State: open

Hi!

There may be a bug in setUserOverlay: method of CTTabWindowController.

I always got an exception while dragging a tab to another window for merging them. I hacked
your code, and found that it was caused by [window makeFirstResponder:cachedContentView_].

In apple's document, they say that
"If this method is called explicitly, first send acceptsFirstResponder to responder, and do not call
makeFirstResponder: if acceptsFirstResponder returns NO."

So I added an if-statement to check whether [window acceptsFirstResponder] return YES, and
no exceptions shown any more.

But I am not so sure about this, and please make your own decision.

Best regards.