Commit 666de7af authored by Andrew Shulaev's avatar Andrew Shulaev Committed by Commit Bot

Convert base::Closure to base::OnceClosure

Context: https://docs.google.com/document/d/1J4N3V5UmzZL9BZGNrn95E4xJu4cz3ynbg21zXR-J_qA/edit#

Bug: 1007635
Change-Id: I28ba29b8360f030f62945b351555cbea69127482
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2435641Reviewed-by: default avatarEric Seckler <eseckler@chromium.org>
Commit-Queue: Andrew Shulaev <ddrone@google.com>
Cr-Commit-Position: refs/heads/master@{#811241}
parent 9b3fdb53
...@@ -76,8 +76,9 @@ class ChromeTracingDelegateBrowserTest : public InProcessBrowserTest { ...@@ -76,8 +76,9 @@ class ChromeTracingDelegateBrowserTest : public InProcessBrowserTest {
} }
void TriggerPreemptiveScenario( void TriggerPreemptiveScenario(
const base::Closure& on_started_finalization_callback) { base::OnceClosure on_started_finalization_callback) {
on_started_finalization_callback_ = on_started_finalization_callback; on_started_finalization_callback_ =
std::move(on_started_finalization_callback);
trigger_handle_ = trigger_handle_ =
content::BackgroundTracingManager::GetInstance()->RegisterTriggerType( content::BackgroundTracingManager::GetInstance()->RegisterTriggerType(
"test"); "test");
...@@ -131,12 +132,12 @@ class ChromeTracingDelegateBrowserTest : public InProcessBrowserTest { ...@@ -131,12 +132,12 @@ class ChromeTracingDelegateBrowserTest : public InProcessBrowserTest {
if (!on_started_finalization_callback_.is_null()) { if (!on_started_finalization_callback_.is_null()) {
content::GetUIThreadTaskRunner({})->PostTask( content::GetUIThreadTaskRunner({})->PostTask(
FROM_HERE, on_started_finalization_callback_); FROM_HERE, std::move(on_started_finalization_callback_));
} }
} }
std::unique_ptr<base::RunLoop> wait_for_upload_; std::unique_ptr<base::RunLoop> wait_for_upload_;
base::Closure on_started_finalization_callback_; base::OnceClosure on_started_finalization_callback_;
int receive_count_; int receive_count_;
int started_finalizations_count_; int started_finalizations_count_;
content::BackgroundTracingManager::TriggerHandle trigger_handle_; content::BackgroundTracingManager::TriggerHandle trigger_handle_;
...@@ -148,7 +149,7 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTest, ...@@ -148,7 +149,7 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTest,
EXPECT_TRUE(StartPreemptiveScenario( EXPECT_TRUE(StartPreemptiveScenario(
content::BackgroundTracingManager::NO_DATA_FILTERING)); content::BackgroundTracingManager::NO_DATA_FILTERING));
TriggerPreemptiveScenario(base::Closure()); TriggerPreemptiveScenario(base::OnceClosure());
WaitForUpload(); WaitForUpload();
...@@ -181,7 +182,7 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTest, ...@@ -181,7 +182,7 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTest,
EXPECT_TRUE(StartPreemptiveScenario( EXPECT_TRUE(StartPreemptiveScenario(
content::BackgroundTracingManager::NO_DATA_FILTERING)); content::BackgroundTracingManager::NO_DATA_FILTERING));
TriggerPreemptiveScenario(base::Closure()); TriggerPreemptiveScenario(base::OnceClosure());
WaitForUpload(); WaitForUpload();
......
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