Use the most recently used download path when displaying a "save as" dialog....

Use the most recently used download path when displaying a "save as" dialog.
Reset the download path when the user clears the download history.

BUG=293
(http://code.google.com/p/chromium/issues/detail?id=293)

[Alex Mendes da Costa, 2008-11-14 07:40:09]

Hi, please review this fix for issue 293.  I'm a Google employee.  Do I still
have to fill out the consent form and add my name to the AUTHORS file?  I
haven't done either yet.  Thanks!

[Elliot Glaysher, 2008-11-14 18:12:12]

On 2008/11/14 07:40:09, Alex Mendes da Costa wrote:
> Hi, please review this fix for issue 293.  I'm a Google employee.  Do I still
> have to fill out the consent form and add my name to the AUTHORS file?  I
> haven't done either yet.  Thanks!

1) Google employees are already covered; you don't have to do anything.
2) I'm not the person to review this change. I have no idea why that bug was
assigned to me. I've added jcampan@chromium.org since he's done more
modifications to download_manager.{cc,h} than any other person since Google
Chrome was publicly released.

[jcampan, 2008-11-19 00:18:14]

LGTM

Jay

[Paul Godavari, 2008-11-19 00:24:50]

http://codereview.chromium.org/10743/diff/1/3
File chrome/browser/download/download_manager.cc (right):

http://codereview.chromium.org/10743/diff/1/3#newcode408
Line 408: // name failed.
Please line up these booleans and comments, it's horrible to read like this.