Fix a back forward bug in ChromeFrame reported on the field. The bug occurs when multiple

Fix a back forward bug in ChromeFrame reported on the field. The bug occurs when multiple
navigations are performed within IE + CF with some originating from IE and others within Chrome.

The CF active document implementation had some code to reuse the active document instance in case
a new CF navigation occurred within the current tab. This was for optimization, i.e. to ensure that
we don't tear down and create a new instance of Chrome. This basically messed up the document destruction
order causing the IE tab history to get populated with information coming from deleted active document
instances at times.

In any case this optimization is no longer necessary as we have a proxy cache maintained in the automation
client in CF which ensures that we don't inadvarently tear down and create new chrome instances.

Fixes bug http://code.google.com/p/chromium/issues/detail?id=90424

BUG=90424
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97177

[grt (UTC plus 2), 2011-08-17 09:51:41]

LGTM.  I think thread_local.h is no longer needed by chrome_active_document.cc. 
Also, please re-try on win bots, as it looks like you hit one that hadn't been
clobbered yet following rsleevi's message a few days ago.

http://codereview.chromium.org/7627005/diff/1/chrome_frame/chrome_active_docu...
File chrome_frame/chrome_active_document.cc (right):

http://codereview.chromium.org/7627005/diff/1/chrome_frame/chrome_active_docu...
chrome_frame/chrome_active_document.cc:29: #include
"base/threading/thread_local.h"
I think this header is no longer needed.

[ananta, 2011-08-17 18:13:14]

http://codereview.chromium.org/7627005/diff/1/chrome_frame/chrome_active_docu...
File chrome_frame/chrome_active_document.cc (right):

http://codereview.chromium.org/7627005/diff/1/chrome_frame/chrome_active_docu...
chrome_frame/chrome_active_document.cc:29: #include
"base/threading/thread_local.h"
On 2011/08/17 09:51:41, grt wrote:
> I think this header is no longer needed.

Done. Removed a bunch of other unneeded headers as well.