file_manager: Remove the last reference of BrowserThread::FILE

The browser test was calling

  content::RunAllPendingInMessageLoop(content::BrowserThread::FILE)

which no longer makes sense since FILE thread is no longer used in
the test since crrev.com/484191. The test kept passing after the
change hence the I didn't notice until gab@ kindly pointed out.

This patch rewrites the file content checking logic with RunLoop
so that there is no need to flush tasks.

BUG=689520
TEST=The browser test still passes

Change-Id: I72983b2f3c4a0b4d3b22a2eca3835bb08ddf574c
Reviewed-on: https://chromium-review.googlesource.com/618067
Reviewed-by: Yuki Awano <yawano@chromium.org>
Commit-Queue: Satoru Takabayashi <satorux@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495089}
1 file changed