 Chromium Code Reviews
 Chromium Code Reviews Issue 1163963004:
  [Omnibox] Changing scheme from https:// to http:// results in DCHECK (Always).  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 1163963004:
  [Omnibox] Changing scheme from https:// to http:// results in DCHECK (Always).  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: chrome/browser/autocomplete/url_index_private_data.cc | 
| diff --git a/chrome/browser/autocomplete/url_index_private_data.cc b/chrome/browser/autocomplete/url_index_private_data.cc | 
| index 2b1138003c83206c9044e2b228579104a3ebeee2..246df95c0bf2e50b74f543b8ffde6ff45d607033 100644 | 
| --- a/chrome/browser/autocomplete/url_index_private_data.cc | 
| +++ b/chrome/browser/autocomplete/url_index_private_data.cc | 
| @@ -1289,11 +1289,12 @@ URLIndexPrivateData::AddHistoryMatch::AddHistoryMatch( | 
| // If the iterator doesn't work, assume an offset of 0. | 
| if (!iter.Init()) | 
| continue; | 
| - // Find the first word start. | 
| + // Find the first word start. If the iterator didn't find a word break, set | 
| + // an offset of term size. For an instance, the offset for "://" should be | 
| 
Mark P
2015/06/08 18:13:16
nit: For an instance -> For example
 
Pritam Nikam
2015/06/09 10:03:53
Done.
 | 
| + // 3, indicating that the word-part is missing. | 
| while (iter.Advance() && !iter.IsWord()) {} | 
| - if (iter.IsWord()) | 
| - lower_terms_to_word_starts_offsets_[i] = iter.prev(); | 
| - // Else: the iterator didn't find a word break. Assume an offset of 0. | 
| + | 
| + lower_terms_to_word_starts_offsets_[i] = iter.prev(); | 
| } | 
| } |