Commit dfe784e5 authored by Nicolás Peña Moreno's avatar Nicolás Peña Moreno Committed by Commit Bot

Remove base::Bind() from chrome/renderer/net

This CL replaces base::Bind() with BindOnce or BindRepeating in the
chrome/renderer/net folder. Two occurrences are in parameters for
AddInterface(), which receives a repeating callback and it makes sense
to have repeating callbacks there. The other one is for a timer where
BindOnce suffices.

Bug: 1007641
Change-Id: Id2d3df19c699b881c492a1a3402bb60dcaa7dd3d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107803
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Reviewed-by: default avatarMatt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751123}
parent 49d6b260
......@@ -167,10 +167,11 @@ NetErrorHelper::NetErrorHelper(RenderFrame* render_frame)
!render_frame->IsHidden()));
render_frame->GetAssociatedInterfaceRegistry()->AddInterface(
base::Bind(&NetErrorHelper::OnNetworkDiagnosticsClientRequest,
base::Unretained(this)));
render_frame->GetAssociatedInterfaceRegistry()->AddInterface(base::Bind(
&NetErrorHelper::OnNavigationCorrectorRequest, base::Unretained(this)));
base::BindRepeating(&NetErrorHelper::OnNetworkDiagnosticsClientRequest,
base::Unretained(this)));
render_frame->GetAssociatedInterfaceRegistry()->AddInterface(
base::BindRepeating(&NetErrorHelper::OnNavigationCorrectorRequest,
base::Unretained(this)));
}
NetErrorHelper::~NetErrorHelper() {
......
......@@ -901,8 +901,8 @@ void NetErrorHelperCore::StartAutoReloadTimer() {
auto_reload_timer_->Stop();
auto_reload_timer_->Start(
FROM_HERE, delay,
base::Bind(&NetErrorHelperCore::AutoReloadTimerFired,
base::Unretained(this)));
base::BindOnce(&NetErrorHelperCore::AutoReloadTimerFired,
base::Unretained(this)));
}
void NetErrorHelperCore::AutoReloadTimerFired() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment