extensions: Get rid of RequestLocalFileSystemFunction::LocalFileSystemCallbackDispatcher

extensions: Get rid of RequestLocalFileSystemFunction::LocalFileSystemCallbackDispatcher

This indirection is no longer necessary. The code is merged to
RequestLocalFileSystemFunction. This is a mechanical change, except
that DCHECK(BrowserThread::CurrentlyOn(...)) is added in several places.

BUG=246091
TEST=Files.app works as before
R=hashimoto@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203807

[hashimoto, 2013-06-03 06:05:29]

lgtm

https://codereview.chromium.org/16316002/diff/2001/chrome/browser/chromeos/ex...
File chrome/browser/chromeos/extensions/file_manager/file_browser_private_api.cc
(right):

https://codereview.chromium.org/16316002/diff/2001/chrome/browser/chromeos/ex...
chrome/browser/chromeos/extensions/file_manager/file_browser_private_api.cc:610:
bool LogoutUserFunction::RunImpl() {
nit: In file_browser_private_api.cc, LogoutUserFunction comes before
RequestLocalFileSystemFunction. Could you move this to the correct position?

[satorux1, 2013-06-03 07:10:45]

https://codereview.chromium.org/16316002/diff/2001/chrome/browser/chromeos/ex...
File chrome/browser/chromeos/extensions/file_manager/file_browser_private_api.cc
(right):

https://codereview.chromium.org/16316002/diff/2001/chrome/browser/chromeos/ex...
chrome/browser/chromeos/extensions/file_manager/file_browser_private_api.cc:610:
bool LogoutUserFunction::RunImpl() {
On 2013/06/03 06:05:29, hashimoto wrote:
> nit: In file_browser_private_api.cc, LogoutUserFunction comes before
> RequestLocalFileSystemFunction. Could you move this to the correct position?

Done.

[hashimoto, 2013-06-03 07:22:17]

lgtm

[satorux1, 2013-06-04 01:23:46]

Committed patchset #3 manually as r203807 (presubmit successful).