Comment #6 on issue 6105 by [email protected]: REGRESSION: Save  
webpage as Web Page, Complete destroys user-inputted filename
http://code.google.com/p/chromium/issues/detail?id=6105

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=8569

------------------------------------------------------------------------
r8569 | [email protected] | 2009-01-23 11:42:33 -0800 (Fri, 23 Jan 2009) |  
19 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/download/save_package.cc?r1=8569&r2=8568
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/shell_dialogs.cc?r1=8569&r2=8568
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/win_util.cc?r1=8569&r2=8568
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/win_util.h?r1=8569&r2=8568

Fix a bug when saving web pages that have a period in their title.

The save dialog code attempts to generate a file extension based on
the suggested name provided to it, which in the case of saving a web
page is the title. If that title contains a '.' character, it treats
anything that follows as the file extension and appends it to the name
the user selects.

For example, saving a page with the title "Google Inc. - A search engine"
and the user choosing the save name as "Google" will result in the saved
file being named "Google. - A search engine.htm" when it should be
"Google.htm".

In the case of saving a web page, we can ignore trying to figure out
the file extension since we already know it will be ".htm".

BUG=6105 (http://crbug.com/6105)

Review URL: http://codereview.chromium.org/18699
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to