diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-07 05:02:08 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-07 05:02:08 +0000 |
| commit | 4b6d916eb655addd9b2682333a649615dd45f6ec (patch) | |
| tree | 9e4df42d766c46bb6581d1376036788887bf2ebc /chrome/common/content_settings_pattern_parser.h | |
| parent | 78a8649e21a194710127fdb659b83e709f02e0e1 (diff) | |
| download | chromium_src-4b6d916eb655addd9b2682333a649615dd45f6ec.zip chromium_src-4b6d916eb655addd9b2682333a649615dd45f6ec.tar.gz chromium_src-4b6d916eb655addd9b2682333a649615dd45f6ec.tar.bz2 | |
This change the native omnibox to look and behave a bit more like the HTML Instant Extended one.
The popup positioning logic now has to be done at a bit of a higher level since it needs to know the window width as well as just the omnibox width. I added a new function in LocationBar::Delegate to get this that's implemented by the toolbar view (the lowest-level UI component that knows about the window width as well as the omnibox width).
More information must now be passed to the popup, so I added a new OmniboxViewDelegate that's implemented by the LocationBar to give the popup its context. This cleaned up some dependencies where the popup had to know about the location bar and did things silly things like this:
location_bar_->GetWidget()->GetNativeView()
R=pkasting@chromium.org
Review URL: https://codereview.chromium.org/15745031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204715 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common/content_settings_pattern_parser.h')
0 files changed, 0 insertions, 0 deletions