Rename UpdateRendererStateForNavigate

Rename UpdateRendererStateForNavigate

The method name is a bit confusing, as it isn't updating any renderer. It makes a policy decision which RenderFrameHost should handle the navigation and returns it out. Dropping the "Renderer" part of the name makes it a bit better.

BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=267128

[nasko, 2014-04-29 21:21:37]

Hey Charlie,
As we discussed earlier, the name of UpdateRendererStateForNavigate can be
better. This is a CL to rename it to something a bit more meaningful. Can you
review it?
Thanks,
Nasko

[Charlie Reis, 2014-04-29 21:32:08]

Thanks, LGTM.

https://codereview.chromium.org/255293002/diff/1/content/browser/frame_host/r...
File content/browser/frame_host/render_frame_host_manager.cc (right):

https://codereview.chromium.org/255293002/diff/1/content/browser/frame_host/r...
content/browser/frame_host/render_frame_host_manager.cc:168:
UpdateStateForNavigate(entry);
This can fit on the previous line now, right?

[nasko, 2014-04-29 21:33:59]

The CQ bit was checked by nasko@chromium.org

[nasko, 2014-04-29 21:34:17]

Done.

https://codereview.chromium.org/255293002/diff/1/content/browser/frame_host/r...
File content/browser/frame_host/render_frame_host_manager.cc (right):

https://codereview.chromium.org/255293002/diff/1/content/browser/frame_host/r...
content/browser/frame_host/render_frame_host_manager.cc:168:
UpdateStateForNavigate(entry);
On 2014/04/29 21:32:09, Charlie Reis wrote:
> This can fit on the previous line now, right?

Done.

[commit-bot: I haz the power, 2014-04-29 21:36:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nasko@chromium.org/255293002/20001

[commit-bot: I haz the power, 2014-04-29 22:12:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-29 22:12:11]

Try jobs failed on following builders:
  win_chromium_x64_rel on tryserver.chromium

[nasko, 2014-04-29 22:12:47]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2014-04-29 22:16:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nasko@chromium.org/255293002/20001

[commit-bot: I haz the power, 2014-04-30 02:03:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-30 02:03:18]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel on tryserver.chromium

[nasko, 2014-04-30 05:15:21]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2014-04-30 05:15:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nasko@chromium.org/255293002/20001

[commit-bot: I haz the power, 2014-04-30 08:00:03]

Change committed as 267128