| Index: components/infobars/core/infobar_container.cc
|
| diff --git a/components/infobars/core/infobar_container.cc b/components/infobars/core/infobar_container.cc
|
| index b6831129e69a830c970b632e9baed4c66a298b86..90ba064d4e0b6dca083a14c0e7c0c1589c5e166d 100644
|
| --- a/components/infobars/core/infobar_container.cc
|
| +++ b/components/infobars/core/infobar_container.cc
|
| @@ -6,11 +6,11 @@
|
|
|
| #include <algorithm>
|
|
|
| +#include "base/auto_reset.h"
|
| #include "base/logging.h"
|
| #include "build/build_config.h"
|
| #include "components/infobars/core/infobar.h"
|
| #include "components/infobars/core/infobar_delegate.h"
|
| -#include "ui/gfx/animation/slide_animation.h"
|
|
|
| namespace infobars {
|
|
|
| @@ -20,7 +20,7 @@ InfoBarContainer::Delegate::~Delegate() {
|
| InfoBarContainer::InfoBarContainer(Delegate* delegate)
|
| : delegate_(delegate),
|
| infobar_manager_(NULL),
|
| - top_arrow_target_height_(InfoBar::kDefaultArrowTargetHeight) {
|
| + ignore_infobar_state_changed_(false) {
|
| }
|
|
|
| InfoBarContainer::~InfoBarContainer() {
|
| @@ -34,23 +34,26 @@ void InfoBarContainer::ChangeInfoBarManager(InfoBarManager* infobar_manager) {
|
| if (infobar_manager_)
|
| infobar_manager_->RemoveObserver(this);
|
|
|
| - // Hides all infobars in this container without animation.
|
| - while (!infobars_.empty()) {
|
| - InfoBar* infobar = infobars_.front();
|
| - // Inform the infobar that it's hidden. If it was already closing, this
|
| - // deletes it. Otherwise, this ensures the infobar will be deleted if it's
|
| - // closed while it's not in an InfoBarContainer.
|
| - infobar->Hide(false);
|
| - }
|
| + {
|
| + // Ignore intermediate state changes. We'll manually trigger processing
|
| + // once we're finished.
|
| + base::AutoReset<bool> ignore_changes(&ignore_infobar_state_changed_, true);
|
| +
|
| + // Hides all infobars in this container without animation.
|
| + while (!infobars_.empty()) {
|
| + InfoBar* infobar = infobars_.front();
|
| + // Inform the infobar that it's hidden. If it was already closing, this
|
| + // deletes it. Otherwise, this ensures the infobar will be deleted if
|
| + // it's closed while it's not in an InfoBarContainer.
|
| + infobar->Hide(false);
|
| + }
|
|
|
| - infobar_manager_ = infobar_manager;
|
| - if (infobar_manager_) {
|
| - infobar_manager_->AddObserver(this);
|
| + infobar_manager_ = infobar_manager;
|
| + if (infobar_manager_) {
|
| + infobar_manager_->AddObserver(this);
|
|
|
| - for (size_t i = 0; i < infobar_manager_->infobar_count(); ++i) {
|
| - // As when we removed the infobars above, we prevent callbacks to
|
| - // OnInfoBarStateChanged() for each infobar.
|
| - AddInfoBar(infobar_manager_->infobar_at(i), i, false, NO_CALLBACK);
|
| + for (size_t i = 0; i < infobar_manager_->infobar_count(); ++i)
|
| + AddInfoBar(infobar_manager_->infobar_at(i), i, false);
|
| }
|
| }
|
|
|
| @@ -77,16 +80,17 @@ int InfoBarContainer::GetVerticalOverlap(int* total_height) const {
|
| return vertical_overlap;
|
| }
|
|
|
| -void InfoBarContainer::SetMaxTopArrowHeight(int height) {
|
| - // Decrease the height by the arrow stroke thickness, which is the separator
|
| - // line height, because the infobar arrow target heights are without-stroke.
|
| - top_arrow_target_height_ = std::min(
|
| - std::max(height - InfoBar::kSeparatorLineHeight, 0),
|
| - InfoBar::kMaximumArrowTargetHeight);
|
| - UpdateInfoBarArrowTargetHeights();
|
| +void InfoBarContainer::UpdateInfoBarArrowTargetHeights() {
|
| + for (size_t i = 0; i < infobars_.size(); ++i) {
|
| + infobars_[i]->SetArrowTargetHeight(delegate_ ?
|
| + delegate_->ArrowTargetHeightForInfoBar(
|
| + i, const_cast<const InfoBar*>(infobars_[i])->animation()) : 0);
|
| + }
|
| }
|
|
|
| void InfoBarContainer::OnInfoBarStateChanged(bool is_animating) {
|
| + if (ignore_infobar_state_changed_)
|
| + return;
|
| if (delegate_)
|
| delegate_->InfoBarContainerStateChanged(is_animating);
|
| UpdateInfoBarArrowTargetHeights();
|
| @@ -112,7 +116,7 @@ void InfoBarContainer::RemoveAllInfoBarsForDestruction() {
|
| }
|
|
|
| void InfoBarContainer::OnInfoBarAdded(InfoBar* infobar) {
|
| - AddInfoBar(infobar, infobars_.size(), true, WANT_CALLBACK);
|
| + AddInfoBar(infobar, infobars_.size(), true);
|
| }
|
|
|
| void InfoBarContainer::OnInfoBarRemoved(InfoBar* infobar, bool animate) {
|
| @@ -128,7 +132,7 @@ void InfoBarContainer::OnInfoBarReplaced(InfoBar* old_infobar,
|
| DCHECK(i != infobars_.end());
|
| size_t position = i - infobars_.begin();
|
| old_infobar->Hide(false);
|
| - AddInfoBar(new_infobar, position, false, WANT_CALLBACK);
|
| + AddInfoBar(new_infobar, position, false);
|
| }
|
|
|
| void InfoBarContainer::OnManagerShuttingDown(InfoBarManager* manager) {
|
| @@ -139,41 +143,15 @@ void InfoBarContainer::OnManagerShuttingDown(InfoBarManager* manager) {
|
|
|
| void InfoBarContainer::AddInfoBar(InfoBar* infobar,
|
| size_t position,
|
| - bool animate,
|
| - CallbackStatus callback_status) {
|
| + bool animate) {
|
| DCHECK(std::find(infobars_.begin(), infobars_.end(), infobar) ==
|
| infobars_.end());
|
| DCHECK_LE(position, infobars_.size());
|
| infobars_.insert(infobars_.begin() + position, infobar);
|
| UpdateInfoBarArrowTargetHeights();
|
| PlatformSpecificAddInfoBar(infobar, position);
|
| - if (callback_status == WANT_CALLBACK)
|
| - infobar->set_container(this);
|
| + infobar->set_container(this);
|
| infobar->Show(animate);
|
| - if (callback_status == NO_CALLBACK)
|
| - infobar->set_container(this);
|
| -}
|
| -
|
| -void InfoBarContainer::UpdateInfoBarArrowTargetHeights() {
|
| - for (size_t i = 0; i < infobars_.size(); ++i)
|
| - infobars_[i]->SetArrowTargetHeight(ArrowTargetHeightForInfoBar(i));
|
| -}
|
| -
|
| -int InfoBarContainer::ArrowTargetHeightForInfoBar(size_t infobar_index) const {
|
| - if (!delegate_ || !delegate_->DrawInfoBarArrows(NULL))
|
| - return 0;
|
| - if (infobar_index == 0)
|
| - return top_arrow_target_height_;
|
| - const gfx::SlideAnimation& first_infobar_animation =
|
| - const_cast<const InfoBar*>(infobars_.front())->animation();
|
| - if ((infobar_index > 1) || first_infobar_animation.IsShowing())
|
| - return InfoBar::kDefaultArrowTargetHeight;
|
| - // When the first infobar is animating closed, we animate the second infobar's
|
| - // arrow target height from the default to the top target height. Note that
|
| - // the animation values here are going from 1.0 -> 0.0 as the top bar closes.
|
| - return top_arrow_target_height_ + static_cast<int>(
|
| - (InfoBar::kDefaultArrowTargetHeight - top_arrow_target_height_) *
|
| - first_infobar_animation.GetCurrentValue());
|
| }
|
|
|
| } // namespace infobars
|
|
|