CrOS: Fix file_watchers_ DCHECK in SelectFileDialogExtensionBrowserTest

CrOS: Fix file_watchers_ DCHECK in SelectFileDialogExtensionBrowserTest

The CrOS file picker closes the dialog immediately, without running the page unload handler.  This meant we were leaking file watcher objects, causing a DCHECK in SelectFileDialogExtensionBrowserTest.  Fixed by calling it manually when needed.

Also, we need to clean up the current directory's watcher, because we don't have a directory-change event in the unload handler that we can use to look up the directory path.

BUG=104692, 103491
TEST=browser_tests, also verify you can open a file with control-O

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110789

[James Cook, 2011-11-18 22:34:03]

Guys, please take a look.  This works on my linux_chromeos build for manually
picking a file with the dialog, opening/using/closing the control-M file manager
tab, and the browser_test SelectFileDialogExtensionBrowserTest.

[Dmitry Zvorygin, 2011-11-18 23:14:04]

On 2011/11/18 22:34:03, James Cook (Chromium) wrote:
> Guys, please take a look.  This works on my linux_chromeos build for manually
> picking a file with the dialog, opening/using/closing the control-M file
manager
> tab, and the browser_test SelectFileDialogExtensionBrowserTest.

LGTM

[rginda, 2011-11-18 23:39:55]

LGTM too