file manager: Prevent [ENTER] from completing the save-as dialog in bad directories

file manager: Prevent [ENTER] from completing the save-as dialog in bad directories

Previous code reset the disabled state of the ok button on-keypress of the filename input text box.  This overwrote the fact the the ok button may have been disabled because we're in the / or /media directories.

BUG=chromium-os:16276
TEST=manual testing on chromeos-chrome in linux

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

[rginda, 2011-06-07 23:35:35]

http://codereview.chromium.org/7108012/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/js/file_manager.js (left):

http://codereview.chromium.org/7108012/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/file_manager.js:1550: var selectable;
This was moved to updateOkButton_, where it should have been.

http://codereview.chromium.org/7108012/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/js/file_manager.js (right):

http://codereview.chromium.org/7108012/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/js/file_manager.js:1553: if
(this.dialogType_ == FileManager.DialogType.SELECT_SAVEAS_FILE) {
This snippet came from updateOkButton_.  It was accidentally put there when
updateOkButton_ was extracted from this function.

[achuithb, 2011-06-08 01:01:20]

On 2011/06/07 23:35:35, rginda wrote:
>
http://codereview.chromium.org/7108012/diff/1/chrome/browser/resources/file_m...
> File chrome/browser/resources/file_manager/js/file_manager.js (left):
> 
>
http://codereview.chromium.org/7108012/diff/1/chrome/browser/resources/file_m...
> chrome/browser/resources/file_manager/js/file_manager.js:1550: var selectable;
> This was moved to updateOkButton_, where it should have been.
> 
>
http://codereview.chromium.org/7108012/diff/1/chrome/browser/resources/file_m...
> File chrome/browser/resources/file_manager/js/file_manager.js (right):
> 
>
http://codereview.chromium.org/7108012/diff/1/chrome/browser/resources/file_m...
> chrome/browser/resources/file_manager/js/file_manager.js:1553: if
> (this.dialogType_ == FileManager.DialogType.SELECT_SAVEAS_FILE) {
> This snippet came from updateOkButton_.  It was accidentally put there when
> updateOkButton_ was extracted from this function.


Ok. LGTM.