OOPIF: Fix cross-process navigations in new windows.

OOPIF: Fix cross-process navigations in new windows.

We were putting the wrong FrameTreeNode ID on the OpenURLParams
when a navigation was destined for a different WebContents.

BUG=464944
TEST=Shift click a link to https://chrome.google.com/webstore
     in --site-per-process mode.

Committed: https://crrev.com/b1a8a668fe3478ddb399eef33728914fe01a8a43
Cr-Commit-Position: refs/heads/master@{#319690}

[Charlie Reis, 2015-03-07 00:47:34]

creis@chromium.org changed reviewers:
+ nasko@chromium.org

[Charlie Reis, 2015-03-07 00:47:35]

Nasko, can you take a look?  This should get the try bot to go green.

[nasko, 2015-03-07 00:55:15]

LGTM

https://codereview.chromium.org/988953003/diff/1/content/browser/frame_host/n...
File content/browser/frame_host/navigator_impl.cc (right):

https://codereview.chromium.org/988953003/diff/1/content/browser/frame_host/n...
content/browser/frame_host/navigator_impl.cc:623: // Send the navigation to the
current FrameTreeNode if it's destined for a
nit: I find it more readable if there is an empty line above the comment.

[Charlie Reis, 2015-03-07 03:23:28]

Thanks.  Glad to see the linux_site_isolation try bot went green!  CQ'ing.

https://codereview.chromium.org/988953003/diff/1/content/browser/frame_host/n...
File content/browser/frame_host/navigator_impl.cc (right):

https://codereview.chromium.org/988953003/diff/1/content/browser/frame_host/n...
content/browser/frame_host/navigator_impl.cc:623: // Send the navigation to the
current FrameTreeNode if it's destined for a
On 2015/03/07 00:55:15, nasko wrote:
> nit: I find it more readable if there is an empty line above the comment.

Done.

[Charlie Reis, 2015-03-07 03:23:34]

The CQ bit was checked by creis@chromium.org

[Charlie Reis, 2015-03-07 03:23:35]

The patchset sent to the CQ was uploaded after l-g-t-m from nasko@chromium.org
Link to the patchset: https://codereview.chromium.org/988953003/#ps20001 (title:
"Fix spacing")

[commit-bot: I haz the power, 2015-03-07 03:24:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/988953003/20001

[commit-bot: I haz the power, 2015-03-07 05:25:03]

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

[commit-bot: I haz the power, 2015-03-07 05:25:04]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_TIMED_OUT, no build
URL)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)
Timed out jobs are not retried to avoid causing additional load on the builders
with large pending queues.

[Charlie Reis, 2015-03-09 16:43:38]

The CQ bit was checked by creis@chromium.org

[commit-bot: I haz the power, 2015-03-09 16:44:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/988953003/20001

[commit-bot: I haz the power, 2015-03-09 18:22:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-09 18:23:00]

Patchset 2 (id:??) landed as
https://crrev.com/b1a8a668fe3478ddb399eef33728914fe01a8a43
Cr-Commit-Position: refs/heads/master@{#319690}