diff options
| author | rahulk@google.com <rahulk@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-08-21 18:18:30 +0000 |
|---|---|---|
| committer | rahulk@google.com <rahulk@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-08-21 18:18:30 +0000 |
| commit | 025a16feb0c23574585de20a7bc6524a9a6a1e06 (patch) | |
| tree | 6199c376b691b50a4b31436a16e9e198a60e8931 /chrome/browser/vista_frame.cc | |
| parent | d862fd9d396126f3f06a8061412def6d61d156fd (diff) | |
| download | chromium_src-025a16feb0c23574585de20a7bc6524a9a6a1e06.zip chromium_src-025a16feb0c23574585de20a7bc6524a9a6a1e06.tar.gz chromium_src-025a16feb0c23574585de20a7bc6524a9a6a1e06.tar.bz2 | |
Revert 1158 as it is breaking ui_tests
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1165 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/vista_frame.cc')
| -rw-r--r-- | chrome/browser/vista_frame.cc | 55 |
1 files changed, 25 insertions, 30 deletions
diff --git a/chrome/browser/vista_frame.cc b/chrome/browser/vista_frame.cc index 18e3340..bdd3dbbd 100644 --- a/chrome/browser/vista_frame.cc +++ b/chrome/browser/vista_frame.cc @@ -249,30 +249,28 @@ void VistaFrame::Layout() { // If we are maxmized, the tab strip will be in line with the window // controls, so we need to make sure they don't overlap. int zoomed_offset = 0; - if (distributor_logo_) { - if(IsZoomed()) { - zoomed_offset = std::max(min_offset, kWindowControlsMinOffset); + if(IsZoomed()) { + zoomed_offset = std::max(min_offset, kWindowControlsMinOffset); - // Hide the distributor logo if we're zoomed. - distributor_logo_->SetVisible(false); - } else { - CSize distributor_logo_size; - distributor_logo_->GetPreferredSize(&distributor_logo_size); - - int logo_x; - // Because of Bug 1128173, our Window controls aren't actually flipped - // on Vista, yet all our math and layout presumes that they are. - if (frame_view_->UILayoutIsRightToLeft()) - logo_x = width - distributor_logo_size.cx; - else - logo_x = width - min_offset - distributor_logo_size.cx; - - distributor_logo_->SetVisible(true); - distributor_logo_->SetBounds(logo_x, - kDistributorLogoVerticalOffset, - distributor_logo_size.cx, - distributor_logo_size.cy); - } + // Hide the distributor logo if we're zoomed. + distributor_logo_->SetVisible(false); + } else { + CSize distributor_logo_size; + distributor_logo_->GetPreferredSize(&distributor_logo_size); + + int logo_x; + // Because of Bug 1128173, our Window controls aren't actually flipped + // on Vista, yet all our math and layout presumes that they are. + if (frame_view_->UILayoutIsRightToLeft()) + logo_x = width - distributor_logo_size.cx; + else + logo_x = width - min_offset - distributor_logo_size.cx; + + distributor_logo_->SetVisible(true); + distributor_logo_->SetBounds(logo_x, + kDistributorLogoVerticalOffset, + distributor_logo_size.cx, + distributor_logo_size.cy); } gfx::Rect tabstrip_bounds(tabstrip_x, @@ -450,13 +448,10 @@ void VistaFrame::Init() { frame_view_->AddChildView(off_the_record_image_); } - SkBitmap* image = rb.GetBitmapNamed(IDR_DISTRIBUTOR_LOGO); - if (!image->isNull()) { - distributor_logo_ = new ChromeViews::ImageView(); - frame_view_->AddViewToDropList(distributor_logo_); - distributor_logo_->SetImage(image); - frame_view_->AddChildView(distributor_logo_); - } + distributor_logo_ = new ChromeViews::ImageView(); + frame_view_->AddViewToDropList(distributor_logo_); + distributor_logo_->SetImage(rb.GetBitmapNamed(IDR_DISTRIBUTOR_LOGO)); + frame_view_->AddChildView(distributor_logo_); tab_contents_container_ = new TabContentsContainerView(); frame_view_->AddChildView(tab_contents_container_); |
