Change OmniboxEditModel to always use virtual URL instead of actual URL
commit23781675b5a63c3ba5d1ddf13be9dd78e1c2a1e5
authorblundell <blundell@chromium.org>
Thu, 30 Jul 2015 19:42:33 +0000 (30 12:42 -0700)
committerCommit bot <commit-bot@chromium.org>
Thu, 30 Jul 2015 19:43:04 +0000 (30 19:43 +0000)
tree24362c656fb241b00963aef82f9678162ffbb260
parentaf6e857fabbb950795ad49a9d3a992148fcddf61
Change OmniboxEditModel to always use virtual URL instead of actual URL

OmniboxEditModel typically manipulates the virtual URL (as defined in the
comments to NavigationEntry::GetVirtualURL()). However, there is one callsite
where it uses the actual URL (as defined in the comments to
NavigationEntry::GetURL()). Per the comment above this usage, it appears that
this usage is intended to be the virtual URL (aka the visible URL). This CL
makes the change.

To understand how the code change relates to the above description, note that
ChromeOmniboxClient::GetURL() returns the virtual URL of the visible
NavigationEntry (via WebContents::GetURL(), which itself is the same as
WebContents::GetVisibleURL()).

Review URL: https://codereview.chromium.org/1260703003

Cr-Commit-Position: refs/heads/master@{#341173}
chrome/browser/ui/omnibox/omnibox_edit_model.cc