Fix future build break from upstream renaming patch....

Fix future build break from upstream renaming patch.

R=wtc
BUG=None
TEST=It compiles!  It must work.


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

[wtc, 2009-05-22 15:45:07]

LGTM.

I don't understand how you can rename our method invocations
before renaming the method upstream.  Are you going to check
in this CL after the next webkit integration?

http://codereview.chromium.org/115702/diff/1/2
File webkit/glue/resource_handle_impl.cc (right):

http://codereview.chromium.org/115702/diff/1/2#newcode414
Line 414: //               This is not more obviously wrong, but I'm making
progress
Two nits:

1. Is "not" a typo?

2. not more wrong than what?  The mainDocumentURL method
that is no longer in the code below?

[abarth-chromium, 2009-05-22 16:51:27]

On 2009/05/22 15:45:07, wtc wrote:
> I don't understand how you can rename our method invocations
> before renaming the method upstream.  Are you going to check
> in this CL after the next webkit integration?

I'm going to land it right after the integration builder goes red.  I just
wanted to have the patch reviewed and ready before hand.

[wtc, 2009-05-22 16:56:04]

I see.  You may want to edit the Description of the CL
because the current description will be confusing after
you check this in :-)

[abarth-chromium, 2009-05-22 17:02:14]

On 2009/05/22 16:56:04, wtc wrote:
> I see.  You may want to edit the Description of the CL
> because the current description will be confusing after
> you check this in :-)

I edited it a bit.  It's still a build break from the future because the build
break will never be in our tree...  Plus, that sounds funny.  :)

[wtc, 2009-05-22 17:04:40]

OK! :-)