[fsp] Extract ObservedEntry and ObservedEntries to a separate file.

[fsp] Extract ObservedEntry and ObservedEntries to a separate file.

Before they were part of the interface, which should be clean and simple.
Moreover, this CL lets pass the list of changes via ProvidedFileSystemObserver
which will be required soon in the Service class.

TEST=unit_tests, browser_tests: *FileSystemProvider*
BUG=248427
Committed: https://crrev.com/b9bac4b512b768e2d17e9199cc1a66997c55aaf3
Cr-Commit-Position: refs/heads/master@{#300071}

[mtomasz, 2014-10-17 03:47:04]

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

[mtomasz, 2014-10-17 03:47:13]

@fukino: PTAL. Thanks!

[fukino, 2014-10-17 04:51:02]

Thanks! lgtm with a nit.

https://codereview.chromium.org/663713002/diff/20001/chrome/browser/chromeos/...
File
chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h
(right):

https://codereview.chromium.org/663713002/diff/20001/chrome/browser/chromeos/...
chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h:18:
#include "chrome/browser/chromeos/file_system_provider/observed_entry.h"
nit: unneeded header?

[mtomasz, 2014-10-17 05:29:43]

Thanks!

https://codereview.chromium.org/663713002/diff/20001/chrome/browser/chromeos/...
File
chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h
(right):

https://codereview.chromium.org/663713002/diff/20001/chrome/browser/chromeos/...
chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h:18:
#include "chrome/browser/chromeos/file_system_provider/observed_entry.h"
On 2014/10/17 04:51:02, fukino wrote:
> nit: unneeded header?

ObservedEntry was previously declared in the same file, now it's moved away, so
we need an include. See #204.

[mtomasz, 2014-10-17 05:29:47]

The CQ bit was checked by mtomasz@chromium.org

[commit-bot: I haz the power, 2014-10-17 05:30:31]

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

[fukino, 2014-10-17 05:36:08]

On 2014/10/17 05:29:43, mtomasz wrote:
> Thanks!
> 
>
https://codereview.chromium.org/663713002/diff/20001/chrome/browser/chromeos/...
> File
> chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h
> (right):
> 
>
https://codereview.chromium.org/663713002/diff/20001/chrome/browser/chromeos/...
>
chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h:18:
> #include "chrome/browser/chromeos/file_system_provider/observed_entry.h"
> On 2014/10/17 04:51:02, fukino wrote:
> > nit: unneeded header?
> 
> ObservedEntry was previously declared in the same file, now it's moved away,
so
> we need an include. See #204.

Oh, I overlooked #204. Thank you for explanation!

[commit-bot: I haz the power, 2014-10-17 06:22:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-17 06:23:34]

Patchset 2 (id:??) landed as
https://crrev.com/b9bac4b512b768e2d17e9199cc1a66997c55aaf3
Cr-Commit-Position: refs/heads/master@{#300071}