Clean up the proxy code for handling frame navigation....

Clean up the proxy code for handling frame navigation.

When a frame navigates to a new page, the old context handle is disposed, and the global proxy object is detached from
the old context. A new context is created using the global proxy object. However, the security token is not created
because, at this point, the security origin is not updated to new URL.
When the new document is ready, V8Proxy updates the security token using the security origin of the document.

Our code relies on the fact that ScriptController::clearWindowShell be called only when the frame loads a page, but not at closing time.



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

[Mike Belshe, 2008-11-13 18:13:49]

lgtm

http://codereview.chromium.org/10687/diff/1/4
File webkit/port/bindings/v8/v8_proxy.cpp (right):

http://codereview.chromium.org/10687/diff/1/4#newcode1645
Line 1645: 
extra line

http://codereview.chromium.org/10687/diff/1/4#newcode1659
Line 1659: {   v8::HandleScope scope;
funky braces

[Feng Qian, 2008-11-13 21:30:18]

http://codereview.chromium.org/10687/diff/1/4
File webkit/port/bindings/v8/v8_proxy.cpp (right):

http://codereview.chromium.org/10687/diff/1/4#newcode1645
Line 1645: 
On 2008/11/13 18:13:50, mbelshe wrote:
> extra line
> 
> 

Done.

http://codereview.chromium.org/10687/diff/1/4#newcode1659
Line 1659: {   v8::HandleScope scope;
On 2008/11/13 18:13:50, mbelshe wrote:
> funky braces
> 
> 

Done.