FasterReload: remove default WebFrameLoadType from WebFrame::reload*

FasterReload: remove default WebFrameLoadType from WebFrame::reload*

Now, there are two callses that use WebFrameLoadType::Reload to call
WebFrame::reloadWithOverrideURL(). These callers can be modified to use
WebFrameLoadType::ReloadMainResoure insteads.

Also, this change removes default argument from WebFrame::reload() and
WebFrame::reloadWithOverrideURL() so to encourage using a right reload
type for each caller respectively. Also this will be better in terms of
coding style since default argument for virtual method is not recommended
in chromium.

BUG=670196
Committed: https://crrev.com/4b39f0247b96125c94a71dd3323ea16b8deac182
Cr-Commit-Position: refs/heads/master@{#438737}

[Takashi Toyoshima, 2016-12-14 09:02:31]

Description was changed from

==========
FasterReload: remove default WebFrameLoadType from WebFrame::reload*

Now, there are two callses that use WebFrameLoadType::Reload to call
WebFrame::reloadWithOverrideURL(). These callers can be modified to use
WebFrameLoadType::ReloadMainResoure insteads.

Also, this change removes default argument from WebFrame::reload() and
WebFrame::reloadWithOverrideURL() so to encourage using a right reload
type for each caller respectively. Also this will be better in terms of
coding style since default argument for virtual method is not recommended
in chromium.

BUG=670196
==========

to

==========
FasterReload: remove default WebFrameLoadType from WebFrame::reload*

Now, there are two callses that use WebFrameLoadType::Reload to call
WebFrame::reloadWithOverrideURL(). These callers can be modified to use
WebFrameLoadType::ReloadMainResoure insteads.

Also, this change removes default argument from WebFrame::reload() and
WebFrame::reloadWithOverrideURL() so to encourage using a right reload
type for each caller respectively. Also this will be better in terms of
coding style since default argument for virtual method is not recommended
in chromium.

BUG=670196
==========

[Takashi Toyoshima, 2016-12-14 09:02:31]

toyoshim@chromium.org changed reviewers:
+ tkent@chromium.org

[Takashi Toyoshima, 2016-12-14 09:03:13]

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

[Takashi Toyoshima, 2016-12-14 09:03:21]

ptal

[commit-bot: I haz the power, 2016-12-14 09:03:22]

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-12-14 12:36:05]

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

[commit-bot: I haz the power, 2016-12-14 12:36:06]

Dry run: This issue passed the CQ dry run.

[tkent, 2016-12-14 23:01:30]

lgtm

[Takashi Toyoshima, 2016-12-15 04:23:31]

The CQ bit was checked by toyoshim@chromium.org

[commit-bot: I haz the power, 2016-12-15 04:23:54]

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 04:27:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481775811217980, "parent_rev":
"3fa9478a3cf91dc2c322b195e5887bd3f6e5dbb4", "commit_rev":
"7cb592e3d571183e41be47f49fc2e33143645af6"}

[commit-bot: I haz the power, 2016-12-15 04:27:47]

Description was changed from

==========
FasterReload: remove default WebFrameLoadType from WebFrame::reload*

Now, there are two callses that use WebFrameLoadType::Reload to call
WebFrame::reloadWithOverrideURL(). These callers can be modified to use
WebFrameLoadType::ReloadMainResoure insteads.

Also, this change removes default argument from WebFrame::reload() and
WebFrame::reloadWithOverrideURL() so to encourage using a right reload
type for each caller respectively. Also this will be better in terms of
coding style since default argument for virtual method is not recommended
in chromium.

BUG=670196
==========

to

==========
FasterReload: remove default WebFrameLoadType from WebFrame::reload*

Now, there are two callses that use WebFrameLoadType::Reload to call
WebFrame::reloadWithOverrideURL(). These callers can be modified to use
WebFrameLoadType::ReloadMainResoure insteads.

Also, this change removes default argument from WebFrame::reload() and
WebFrame::reloadWithOverrideURL() so to encourage using a right reload
type for each caller respectively. Also this will be better in terms of
coding style since default argument for virtual method is not recommended
in chromium.

BUG=670196

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

[commit-bot: I haz the power, 2016-12-15 04:27:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-15 04:29:38]

Description was changed from

==========
FasterReload: remove default WebFrameLoadType from WebFrame::reload*

Now, there are two callses that use WebFrameLoadType::Reload to call
WebFrame::reloadWithOverrideURL(). These callers can be modified to use
WebFrameLoadType::ReloadMainResoure insteads.

Also, this change removes default argument from WebFrame::reload() and
WebFrame::reloadWithOverrideURL() so to encourage using a right reload
type for each caller respectively. Also this will be better in terms of
coding style since default argument for virtual method is not recommended
in chromium.

BUG=670196

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

to

==========
FasterReload: remove default WebFrameLoadType from WebFrame::reload*

Now, there are two callses that use WebFrameLoadType::Reload to call
WebFrame::reloadWithOverrideURL(). These callers can be modified to use
WebFrameLoadType::ReloadMainResoure insteads.

Also, this change removes default argument from WebFrame::reload() and
WebFrame::reloadWithOverrideURL() so to encourage using a right reload
type for each caller respectively. Also this will be better in terms of
coding style since default argument for virtual method is not recommended
in chromium.

BUG=670196

Committed: https://crrev.com/4b39f0247b96125c94a71dd3323ea16b8deac182
Cr-Commit-Position: refs/heads/master@{#438737}
==========

[commit-bot: I haz the power, 2016-12-15 04:29:39]

Patchset 1 (id:??) landed as
https://crrev.com/4b39f0247b96125c94a71dd3323ea16b8deac182
Cr-Commit-Position: refs/heads/master@{#438737}