[ew] Move operation observers from QuotaUtil to FileSystemBackend.

[ew] Move operation observers from QuotaUtil to FileSystemBackend.

Soon the storage::WatcherManager interface instantiations will want to be an
observer for operations. However, not all backends which have
storage::WatcherManager have storage::QuotaUtil.

To resolve this issue, this CL moves the observer lists one level up to the
storage::FileSystemBackend.

TEST=Just refactoring. Current tests should pass.
BUG=261491
Committed: https://crrev.com/110ffbe98fd296a6e77328d0b963813bf5635533
Cr-Commit-Position: refs/heads/master@{#294367}

[mtomasz, 2014-09-09 04:49:37]

mtomasz@chromium.org changed reviewers:
+ tzik@chromium.org

[mtomasz, 2014-09-09 04:56:17]

On 2014/09/09 04:49:37, mtomasz wrote:
> mailto:mtomasz@chromium.org changed reviewers:
> + mailto:tzik@chromium.org

(+cc @kinuko)
@tzik: PTAL.

It compiles, but I think there should be direct includes to
storage/browser/fileapi/task_runner_bound_observer_list.h in many header files,
but DEPS restrictions would not let me do that. I noticed that there are some
forwards to skip DEPS:

https://code.google.com/p/chromium/codesearch#chromium/src/webkit/browser/fil...

Let me know if I can keep it as it is, or add a DEPS exception, or create some
forward like above.

Thanks.

[tzik, 2014-09-09 05:15:17]

On 2014/09/09 04:56:17, mtomasz wrote:
> On 2014/09/09 04:49:37, mtomasz wrote:
> > mailto:mtomasz@chromium.org changed reviewers:
> > + mailto:tzik@chromium.org
> 
> (+cc @kinuko)
> @tzik: PTAL.
> 
> It compiles, but I think there should be direct includes to
> storage/browser/fileapi/task_runner_bound_observer_list.h in many header
files,
> but DEPS restrictions would not let me do that. I noticed that there are some
> forwards to skip DEPS:
> 
>
https://code.google.com/p/chromium/codesearch#chromium/src/webkit/browser/fil...
> 
> Let me know if I can keep it as it is, or add a DEPS exception, or create some
> forward like above.
> 
> Thanks.

These files are moving from webkit/ to storage/, as you may know, and these
forwarding headers will be removed on completion.
I think it's OK to use these forwarding headers for now, since the bulk #include
rewriting is not yet started.

[tzik, 2014-09-09 07:00:34]

lgtm

[mtomasz, 2014-09-09 09:19:10]

On 2014/09/09 07:00:34, tzik wrote:
> lgtm

@tzik: Shall I create a webkit/browser/fileapi/task_runner_bound_observer_list.h
forward to storage/browser/fileapi/task_runner_bound_observer_list.h or just
commit as it is?

[tzik, 2014-09-09 14:01:08]

On 2014/09/09 09:19:10, mtomasz wrote:
> On 2014/09/09 07:00:34, tzik wrote:
> > lgtm
> 
> @tzik: Shall I create a
webkit/browser/fileapi/task_runner_bound_observer_list.h
> forward to storage/browser/fileapi/task_runner_bound_observer_list.h or just
> commit as it is?

We already have one:
https://chromium.googlesource.com/chromium/src/+/master/webkit/browser/fileap...

[mtomasz, 2014-09-10 01:18:10]

On 2014/09/09 14:01:08, tzik wrote:
> On 2014/09/09 09:19:10, mtomasz wrote:
> > On 2014/09/09 07:00:34, tzik wrote:
> > > lgtm
> > 
> > @tzik: Shall I create a
> webkit/browser/fileapi/task_runner_bound_observer_list.h
> > forward to storage/browser/fileapi/task_runner_bound_observer_list.h or just
> > commit as it is?
> 
> We already have one:
>
https://chromium.googlesource.com/chromium/src/+/master/webkit/browser/fileap...

Sure, missed it.

[mtomasz, 2014-09-10 01:21:02]

mtomasz@chromium.org changed reviewers:
+ phajdan.jr@chromium.org

[mtomasz, 2014-09-10 01:21:17]

@phajdan.jr: PTAL at content/public/test/*. Thanks.

[Paweł Hajdan Jr., 2014-09-11 09:13:38]

content/public/test LGTM

[mtomasz, 2014-09-11 09:14:05]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2014-09-11 09:14:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/549413003/20001

[commit-bot: I haz the power, 2014-09-11 09:19:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-11 09:19:16]

Try jobs failed on following builders:
  ios_rel_device_ninja on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[mtomasz, 2014-09-11 09:36:43]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2014-09-11 09:37:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/549413003/40001

[commit-bot: I haz the power, 2014-09-11 10:36:43]

Committed patchset #3 (id:40001) as 85ffe8b87dc23dc2d9e5e177e21d447dda9475a7

[commit-bot: I haz the power, 2014-09-11 10:39:56]

Patchset 3 (id:??) landed as
https://crrev.com/110ffbe98fd296a6e77328d0b963813bf5635533
Cr-Commit-Position: refs/heads/master@{#294367}