Simplify HomeCardGestureManager::UpdateScrollState()

athena/home/home_card_impl.cc

 // Copyright 2014 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 "athena/home/home_card_impl.h"
 
 #include <cmath>
 #include <limits>
 
 #include "athena/common/container_priorities.h"
@@ skipping 139 matching lines @@
     // the home card.
     // TODO(mukai): make it so after the detailed UI has been fixed.
     main_view_ = new AthenaStartPageView(view_delegate);
     AddChildView(main_view_);
   }
 
   void SetStateProgress(HomeCard::State from_state,
                         HomeCard::State to_state,
                         float progress) {
     // TODO(mukai): not clear the focus, but simply close the virtual keyboard.
-    if (from_state != HomeCard::VISIBLE_CENTERED ||
-        to_state != HomeCard::VISIBLE_CENTERED) {
-      GetFocusManager()->ClearFocus();
-    }
+    GetFocusManager()->ClearFocus();
+
     if (from_state == HomeCard::VISIBLE_CENTERED)
       main_view_->SetLayoutState(1.0f - progress);
     else if (to_state == HomeCard::VISIBLE_CENTERED)
       main_view_->SetLayoutState(progress);
     UpdateShadow(true);
   }
 
   void SetStateWithAnimation(HomeCard::State state) {
     UpdateShadow(state != HomeCard::VISIBLE_MINIMIZED);
     if (state == HomeCard::VISIBLE_CENTERED)
@@ skipping 215 matching lines @@
     home_card_view_->SetStateWithAnimation(state_);
     layout_manager_->Layout(true);
   }
 }
 
 void HomeCardImpl::OnGestureProgressed(
     State from_state, State to_state, float progress) {
   if (from_state == VISIBLE_MINIMIZED || to_state == VISIBLE_MINIMIZED) {
     minimized_home_->layer()->SetVisible(true);
     float opacity =
         (from_state == VISIBLE_MINIMIZED) ? 1.0f - progress : progress;

[Jun Mukai, 2014/09/02 08:22:42]

Instead this could be rewritten as:
opacity = (to_state == VISIBLE_MINIMIZED) ? progress : 1.0f - progress;
?

     minimized_home_->layer()->SetOpacity(opacity);
   }
   gfx::Rect screen_bounds =
       home_card_widget_->GetNativeWindow()->GetRootWindow()->bounds();
   home_card_widget_->SetBounds(gfx::Tween::RectValueBetween(
       progress,
       GetBoundsForState(screen_bounds, from_state),
       GetBoundsForState(screen_bounds, to_state)));
 
   home_card_view_->SetStateProgress(from_state, to_state, progress);
@@ skipping 36 matching lines @@
   instance = NULL;
 }
 
 // static
 HomeCard* HomeCard::Get() {
   DCHECK(instance);
   return instance;
 }
 
 }  // namespace athena