Current ServiceWorkerVersion::StopWorker() takes StatusCallback as args, but this method always runs it with SERVICE_WORKER_OK. To simplify this, change it to OnceClosure.
i like turtles]
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd commit 0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd Author: Yuki Yamada <yukiy@google.com> Date: Fri Sep 15 02:15:23 2017 Make ServiceWorkerVersion::StopWorker() take OnceClosure This CL changes the argument of ServiceWorkerVersion::StopWorker(). It took StatusCallback but just always returned SERVICE_WORKER_OK. To simplify this, change it to OnceClosure. Bug: 764595 Change-Id: Ib74ff2a13ebdad382f0df667a3f4b542108f15a3 Reviewed-on: https://chromium-review.googlesource.com/664520 Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Reviewed-by: Makoto Shimazu <shimazu@chromium.org> Reviewed-by: Matt Falkenhagen <falken@chromium.org> Commit-Queue: Yuki Yamada <yukiy@google.com> Cr-Commit-Position: refs/heads/master@{#502132} [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/devtools/protocol/service_worker_handler.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/devtools/service_worker_devtools_agent_host.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/payments/payment_app_content_unittest_base.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_browsertest.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_context_core.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_context_wrapper.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_dispatcher_host.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_internals_ui.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_internals_ui.h [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_job_unittest.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_register_job.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_registration.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_version.cc [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_version.h [modify] https://crrev.com/0982ddb4d4f612cf5078dd2b1ac835142b5bfbbd/content/browser/service_worker/service_worker_version_unittest.cc
Comment 1 by 0499...@schoolsnet.act.edu.au
, Sep 13 2017