Fix app list position in windows

ui/views/bubble/bubble_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/bubble/bubble_delegate.h"
 
 #include "ui/base/animation/slide_animation.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/rect.h"
 #include "ui/native_theme/native_theme.h"
@@ skipping 213 matching lines @@
     GetWidget()->Close();
 }
 
 void BubbleDelegateView::OnWidgetBoundsChanged(Widget* widget,
                                                const gfx::Rect& new_bounds) {
   if (move_with_anchor() && anchor_widget() == widget)
     SizeToContents();
 }
 
 gfx::Rect BubbleDelegateView::GetAnchorRect() {
-  gfx::Rect anchor_bounds = anchor_view() ? anchor_view()->GetBoundsInScreen() :
-      gfx::Rect(anchor_point_, gfx::Size());
-  anchor_bounds.Inset(anchor_insets_);
-  return anchor_bounds;
+  if (anchor_view()) {
+    gfx::Rect anchor_bounds = anchor_view()->GetBoundsInScreen();
+    anchor_bounds.Inset(anchor_insets_);
+    return anchor_bounds;
+  } else {
+    return gfx::Rect(anchor_point_, gfx::Size());

[benwells, 2013/03/28 05:43:18]

nit: no need for the else here, as the if() has a return.

That is, do it like this:

if (anchor_view()) {
  ...
  return ...;
}

return ....;

+  }
 }
 
 void BubbleDelegateView::StartFade(bool fade_in) {
   fade_animation_.reset(new ui::SlideAnimation(this));
   fade_animation_->SetSlideDuration(kHideFadeDurationMS);
   fade_animation_->Reset(fade_in ? 0.0 : 1.0);
   if (fade_in) {
     original_opacity_ = 0;
     if (border_widget_)
       border_widget_->SetOpacity(original_opacity_);
@@ skipping 105 matching lines @@
 #if defined(OS_WIN) && !defined(USE_AURA)
 gfx::Rect BubbleDelegateView::GetBubbleClientBounds() const {
   gfx::Rect client_bounds(GetBubbleFrameView()->GetBoundsForClientView());
   client_bounds.Offset(
       border_widget_->GetWindowBoundsInScreen().OffsetFromOrigin());
   return client_bounds;
 }
 #endif
 
 }  // namespace views