diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-31 19:38:13 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-31 19:38:13 +0000 |
| commit | 5883c27ad1e8832365ceca1c2e40078b2670049d (patch) | |
| tree | ed4f4f0d3e5f76bff272218bccccdb5c8ebc0c96 /o3d/converter | |
| parent | 1105c3ead05ab927e53a129691319a75a1ae1e15 (diff) | |
| download | chromium_src-5883c27ad1e8832365ceca1c2e40078b2670049d.zip chromium_src-5883c27ad1e8832365ceca1c2e40078b2670049d.tar.gz chromium_src-5883c27ad1e8832365ceca1c2e40078b2670049d.tar.bz2 | |
Minor autocomplete fixes while trying to track down a topcrash:
* Turn some DCHECKs into CHECKs (I don't think any of these will get hit)
* Ensure that URLsForDefaultMatch() doesn't leave any results lying around on the controller, since the popup isn't open (and those two states should be locked in step)
* Eliminate some old code in Move() that shouldn't have actually done anything anymore (since result() should already be in sync with the popup, and SetSelectedLine() is going to stop the controller)
* Use a pre-existing helper function at two places in the Edit for readability
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/180045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@24920 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'o3d/converter')
0 files changed, 0 insertions, 0 deletions