diff options
| author | jamescook@chromium.org <jamescook@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-21 23:08:17 +0000 |
|---|---|---|
| committer | jamescook@chromium.org <jamescook@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-21 23:08:17 +0000 |
| commit | afbf65f026465e65136bb4de4d6b9946b4ec1c19 (patch) | |
| tree | ebeb569052b5af0f070af8335f3881e3b8906728 /views/view.h | |
| parent | 8f0987c4219757117f1cda4639357746f9a37892 (diff) | |
| download | chromium_src-afbf65f026465e65136bb4de4d6b9946b4ec1c19.zip chromium_src-afbf65f026465e65136bb4de4d6b9946b4ec1c19.tar.gz chromium_src-afbf65f026465e65136bb4de4d6b9946b4ec1c19.tar.bz2 | |
CrOS - Refactor file picker closing to avoid race conditions.
* Move ExtensionDialogObserver to separate header to reduce #include burden on clients of FileManagerDialog.
* Move file picker callback registry logic into FileManagerDialog, which is the owner of the listener pointers.
* Change fileBrowserPrivate.{selectFile,selectFiles,cancel} to immediately close the dialog window. Talked to mpcomplete@ about this, he thinks it's OK to tear it down in the middle of a function call. If this causes problems we can add a callback parameter to those functions and use it to call window.close().
* Added FileManagerDialogTest.SelectFileAndOpen to test picking a file.
BUG=chromium-os:16607
TEST=browser_tests FileManagerDialogTest.*
Review URL: http://codereview.chromium.org/7201007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89920 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/view.h')
0 files changed, 0 insertions, 0 deletions