Rename NavigationHandle::GetRequestContextType to request_context_type.

Rename NavigationHandle::GetRequestContextType to request_context_type.

At some point during discussions of the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing just a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

[carlosk, 2016-12-14 02:04:01]

Description was changed from

==========
Rename NavigationHandle::request_context_type to GetRequestContextType.

At one point during discussions at the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing more than a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
==========

to

==========
Rename NavigationHandle::request_context_type to GetRequestContextType.

At one point during discussions at the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing more than a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[carlosk, 2016-12-14 02:05:16]

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

[commit-bot: I haz the power, 2016-12-14 02:05:42]

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

[carlosk, 2016-12-14 02:07:48]

The CQ bit was unchecked by carlosk@chromium.org

[carlosk, 2016-12-14 02:08:35]

Patchset #1 (id:1) has been deleted

[carlosk, 2016-12-14 02:08:43]

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

[commit-bot: I haz the power, 2016-12-14 02:09:33]

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

[carlosk, 2016-12-14 03:17:30]

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

[carlosk, 2016-12-14 03:17:31]

nasko@: PTAL.

[commit-bot: I haz the power, 2016-12-14 03:18:00]

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

[commit-bot: I haz the power, 2016-12-14 03:18:01]

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

[carlosk, 2016-12-15 01:51:08]

Description was changed from

==========
Rename NavigationHandle::request_context_type to GetRequestContextType.

At one point during discussions at the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing more than a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Rename NavigationHandle::GetRequestContextType to request_context_type.

At some point during discussions of the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing just a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[carlosk, 2016-12-15 01:51:28]

Description was changed from

==========
Rename NavigationHandle::GetRequestContextType to request_context_type.

At some point during discussions of the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing just a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Rename NavigationHandle::GetRequestContextType to request_context_type.

At some point during discussions of the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing just a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[carlosk, 2016-12-15 01:52:19]

Updated description to reflect reality, which is the complete opposite of what I
said before. :)

[nasko, 2016-12-15 18:45:35]

LGTM

[carlosk, 2016-12-15 18:54:06]

The CQ bit was checked by carlosk@chromium.org

[commit-bot: I haz the power, 2016-12-15 18:54:31]

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-12-15 20:11:45]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481828046076220,
"parent_rev": "40587799088bbbd066fb6add99d5d44926e05adb", "commit_rev":
"d0531ffc4b1e1591f9f249191b9d586103106869"}

[commit-bot: I haz the power, 2016-12-15 20:12:10]

Description was changed from

==========
Rename NavigationHandle::GetRequestContextType to request_context_type.

At some point during discussions of the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing just a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Rename NavigationHandle::GetRequestContextType to request_context_type.

At some point during discussions of the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing just a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2577583002
==========

[commit-bot: I haz the power, 2016-12-15 20:12:10]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-12-15 20:15:30]

Description was changed from

==========
Rename NavigationHandle::GetRequestContextType to request_context_type.

At some point during discussions of the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing just a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2577583002
==========

to

==========
Rename NavigationHandle::GetRequestContextType to request_context_type.

At some point during discussions of the browser side mixed-content move
patch (https://crrev.com/1905033002) it was suggested that this getter
was doing just a simple value return and so it should be renamed
accordingly. This spin off change does that.

BUG=576270
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/a954e94f97ea0721925ceac05764e57e020f9493
Cr-Commit-Position: refs/heads/master@{#438905}
==========

[commit-bot: I haz the power, 2016-12-15 20:15:31]

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