diff options
| author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-13 22:21:08 +0000 |
|---|---|---|
| committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-13 22:21:08 +0000 |
| commit | c97d12626e9273d595c6e75d20f70f536de86ee1 (patch) | |
| tree | f43efa0bd7c14c0ce1eae88a5ea558adeaa9a104 /chrome/browser/ui/views/find_bar_host.cc | |
| parent | d959de93dd9e4be04231a86d8c5b19f2038b988a (diff) | |
| download | chromium_src-c97d12626e9273d595c6e75d20f70f536de86ee1.zip chromium_src-c97d12626e9273d595c6e75d20f70f536de86ee1.tar.gz chromium_src-c97d12626e9273d595c6e75d20f70f536de86ee1.tar.bz2 | |
content: Make GetViewBounds() return a rect rather than using an out param.
NOTE: This was a TODO for @ben.
BUG=98716
R=jam@chromium.org
TBR=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10534106
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141995 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/views/find_bar_host.cc')
| -rw-r--r-- | chrome/browser/ui/views/find_bar_host.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/chrome/browser/ui/views/find_bar_host.cc b/chrome/browser/ui/views/find_bar_host.cc index aefb407c..7e6810e 100644 --- a/chrome/browser/ui/views/find_bar_host.cc +++ b/chrome/browser/ui/views/find_bar_host.cc @@ -334,8 +334,7 @@ void FindBarHost::GetWidgetPositionNative(gfx::Rect* avoid_overlapping_rect) { gfx::Rect frame_rect = host()->GetTopLevelWidget()->GetWindowScreenBounds(); content::WebContentsView* tab_view = find_bar_controller_->tab_contents()->web_contents()->GetView(); - gfx::Rect webcontents_rect; - tab_view->GetViewBounds(&webcontents_rect); + gfx::Rect webcontents_rect = tab_view->GetViewBounds(); avoid_overlapping_rect->Offset(0, webcontents_rect.y() - frame_rect.y()); } |