Updates:
        Status: Assigned
        Owner: [email protected]

Comment #8 on issue 18024 by tony.chromium: autocomplete edit view DCHECK
http://code.google.com/p/chromium/issues/detail?id=18024

I looked into this a bit.  To trigger, navigate to the full URL, then paste  
in the
partial URL and type "m" (to trigger the inline autocomplete).

match.inline_autocomplete_offset = 67, because that's how long
"http://skypher.com/SkyLined/Repro/Chrome/carpet%20bombing/repro.htm"; is.

match.fill_into_edit.length() = 66, because that's how long
"http://skypher.com/SkyLined/Repro/Chrome/carpet bombing/repro.html" is.

match.fill_into_edit is unescaped because that's what net::FormatUrl does  
(according
to the header).

So, I'm not sure what this code is supposed to do.  Peter, can you comment?

My guess is we should DCHECK against match.destination_url.length() instead  
of
match.fill_into_edit.length(), but I'm not really sure.

--
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