views omnibox popup: Setup the parent Widget, instead of the parent NativeView.

views omnibox popup: Setup the parent Widget, instead of the parent NativeView.

In views-desktop world, the omnibox popup should be parented to the widget the
location-bar belongs to. This fixes pointer events on the popup in touchui.

BUG=none
TEST=manually

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=95838

[sky, 2011-08-08 16:35:28]

http://codereview.chromium.org/7584024/diff/14/chrome/browser/ui/views/autoco...
File chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc
(right):

http://codereview.chromium.org/7584024/diff/14/chrome/browser/ui/views/autoco...
chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc:360:
params.parent_widget = location_bar_->GetWidget();
Should InitParams parent_widget be a const* Widget?

[sadrul, 2011-08-08 17:05:33]

http://codereview.chromium.org/7584024/diff/14/chrome/browser/ui/views/autoco...
File chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc
(right):

http://codereview.chromium.org/7584024/diff/14/chrome/browser/ui/views/autoco...
chrome/browser/ui/views/autocomplete/autocomplete_popup_contents_view.cc:360:
params.parent_widget = location_bar_->GetWidget();
On 2011/08/08 16:35:28, sky wrote:
> Should InitParams parent_widget be a const* Widget?

I think that'll be very difficult, considering NWViews::hosting_widget_ is set
to parent_widget, and it needs to be non-const (e.g. for GetCompositor).

[sky, 2011-08-08 17:24:52]

Ok, LGTM