[Extensions] Merge AddPendingTaskToDispatchEvent with AddPendingTask.
This CL integrates previous refactoring changes to have a single add task API for the task queues.
Bug: 915814
Change-Id: I1098d0785df50c025bff6c74bc28515d86b88ed8
Reviewed-on: https://chromium-review.googlesource.com/c/1380822
Commit-Queue: David Bertoni <[email protected]>
Reviewed-by: Stuart Langley <[email protected]>
Reviewed-by: Ben Wells <[email protected]>
Reviewed-by: Istiaque Ahmed <[email protected]>
Cr-Commit-Position: refs/heads/master@{#617843}diff --git a/extensions/browser/lazy_background_task_queue.cc b/extensions/browser/lazy_background_task_queue.cc
index bfc6afc..69f0db3 100644
--- a/extensions/browser/lazy_background_task_queue.cc
+++ b/extensions/browser/lazy_background_task_queue.cc
@@ -82,21 +82,14 @@
return false;
}
-void LazyBackgroundTaskQueue::AddPendingTaskToDispatchEvent(
- const LazyContextId& context_id,
- LazyContextTaskQueue::PendingTask task) {
- AddPendingTask(context_id.browser_context(), context_id.extension_id(),
- std::move(task));
-}
-
-void LazyBackgroundTaskQueue::AddPendingTask(
- content::BrowserContext* browser_context,
- const std::string& extension_id,
- PendingTask task) {
+void LazyBackgroundTaskQueue::AddPendingTask(const LazyContextId& context_id,
+ PendingTask task) {
if (ExtensionsBrowserClient::Get()->IsShuttingDown()) {
std::move(task).Run(nullptr);
return;
}
+ const ExtensionId& extension_id = context_id.extension_id();
+ content::BrowserContext* const browser_context = context_id.browser_context();
PendingTasksList* tasks_list = nullptr;
PendingTasksKey key(browser_context, extension_id);
auto it = pending_tasks_.find(key);
