Simplify RenderFrameImpl::ResolveOpener()

Simplify RenderFrameImpl::ResolveOpener() by removing the |opener_view_routing_id| optional param.

In the prior patch set, we removed the last user of that param.

BUG=627852
Committed: https://crrev.com/3b5a21f39ee24c516759e7ed9a7ed92b9157c266
Cr-Commit-Position: refs/heads/master@{#434015}

[ncarter (slow), 2016-11-15 00:36:00]

The CQ bit was checked by nick@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-15 00:37:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-15 00:43:14]

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

[commit-bot: I haz the power, 2016-11-15 00:43:15]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ncarter (slow), 2016-11-16 17:48:03]

The CQ bit was checked by nick@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-16 17:48:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ncarter (slow), 2016-11-16 17:52:27]

Description was changed from

==========
RenderFrameImpl::ResolveOpener() -- remove view logic, and clean up callers.

BUG=627852
==========

to

==========
Simplify RenderFrameImpl::ResolveOpener() by removing the
|opener_view_routing_id| optional param.

In the prior patch set, we removed the last user of that param.

BUG=627852
==========

[ncarter (slow), 2016-11-16 17:53:07]

nick@chromium.org changed reviewers:
+ alexmos@chromium.org

[ncarter (slow), 2016-11-16 17:53:08]

Alex, please review (note that this depends on the other CL I also sent you)

[alexmos, 2016-11-16 18:33:19]

LGTM, really happy to see this cleanup!

[commit-bot: I haz the power, 2016-11-16 19:04:31]

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

[commit-bot: I haz the power, 2016-11-16 19:04:32]

Dry run: This issue passed the CQ dry run.

[ncarter (slow), 2016-11-17 19:45:02]

The CQ bit was checked by nick@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-17 19:46:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-17 21:50:07]

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

[commit-bot: I haz the power, 2016-11-17 21:50:07]

Dry run: This issue passed the CQ dry run.

[ncarter (slow), 2016-11-22 22:01:28]

The CQ bit was checked by nick@chromium.org

[ncarter (slow), 2016-11-22 22:01:28]

The patchset sent to the CQ was uploaded after l-g-t-m from alexmos@chromium.org
Link to the patchset: https://codereview.chromium.org/2499053003/#ps40001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-11-22 22:02:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-22 23:07:14]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1479852088440610,
"parent_rev": "b5ea3b46faaa8313a792059758d134a787e33b5f", "commit_rev":
"2fb3bb7abfdafbf970bf4897c73b13c8546c196f"}

[commit-bot: I haz the power, 2016-11-22 23:07:43]

Description was changed from

==========
Simplify RenderFrameImpl::ResolveOpener() by removing the
|opener_view_routing_id| optional param.

In the prior patch set, we removed the last user of that param.

BUG=627852
==========

to

==========
Simplify RenderFrameImpl::ResolveOpener() by removing the
|opener_view_routing_id| optional param.

In the prior patch set, we removed the last user of that param.

BUG=627852
==========

[commit-bot: I haz the power, 2016-11-22 23:07:44]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-22 23:09:59]

Description was changed from

==========
Simplify RenderFrameImpl::ResolveOpener() by removing the
|opener_view_routing_id| optional param.

In the prior patch set, we removed the last user of that param.

BUG=627852
==========

to

==========
Simplify RenderFrameImpl::ResolveOpener() by removing the
|opener_view_routing_id| optional param.

In the prior patch set, we removed the last user of that param.

BUG=627852

Committed: https://crrev.com/3b5a21f39ee24c516759e7ed9a7ed92b9157c266
Cr-Commit-Position: refs/heads/master@{#434015}
==========

[commit-bot: I haz the power, 2016-11-22 23:10:00]

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