Chromium Code Reviews| Index: chrome/browser/autocomplete/autocomplete_edit.cc |
| =================================================================== |
| --- chrome/browser/autocomplete/autocomplete_edit.cc (revision 25735) |
| +++ chrome/browser/autocomplete/autocomplete_edit.cc (working copy) |
| @@ -456,6 +456,10 @@ |
| return; |
| } |
| + // TODO(suzhe): It shouldn't affect inline autocomplete text at all. |
|
Peter Kasting
2009/09/10 19:30:27
Nit: The substance here is fine, the grammar is a
|
| + // Instead, the edit should deal with it separately in Observe(), and |
| + // save and restore it properly around setting/resetting temporary text. |
| + // |
| // Handle changes to inline autocomplete text. Don't make changes if the user |
| // is showing temporary text. Making display changes would be obviously |
| // wrong; making changes to the inline_autocomplete_text_ itself turns out to |