GitLab

This CL fixes issues I missed while reviewing
https://crrev.com/c/2467899.

* Use base::SequenceChecker instead of making thread-hostile checks. The
  quota client doesn't need to run on the IO thread, it just needs to
  run on the same sequence as NativeIOContext.

* Replace the deprecated DISALLOW_COPY_AND_ASSIGN() macro with deleted
  copy constructor and assignment operator.

* Remove qualifier (storage::QuotaClient::) from QuotaClient callback
  types. This reduces churn when QuotaClient becomes a mojo interface.

* Remove unused GetClientTypeFromOwner() declaration.

* Remove self-#include in header.

No functional changes are introduced.

Bug: 1137788, 1016065
Change-Id: I34f59e128e877358d16f197bc6e6f0ac79de5df3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2570318
Commit-Queue: Jarryd Goodman <jarrydg@chromium.org>
Reviewed-by: Jarryd Goodman <jarrydg@chromium.org>
Auto-Submit: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#833862}