Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3824)

Unified Diff: chrome/browser/ui/views/toolbar/browser_actions_container.cc

Issue 312093006: Remove log statements that are no longer needed. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/extensions/extension_toolbar_model_browsertest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/toolbar/browser_actions_container.cc
diff --git a/chrome/browser/ui/views/toolbar/browser_actions_container.cc b/chrome/browser/ui/views/toolbar/browser_actions_container.cc
index 51b73105131cc652a7ae090a051b251fd62f99aa..06885cdb7e5eeabc6b2d72b3e1c4c721cdc6816f 100644
--- a/chrome/browser/ui/views/toolbar/browser_actions_container.cc
+++ b/chrome/browser/ui/views/toolbar/browser_actions_container.cc
@@ -184,8 +184,6 @@ size_t BrowserActionsContainer::VisibleBrowserActions() const {
if (browser_action_views_[i]->visible())
++visible_actions;
}
- VLOG(4) << "BAC::VisibleBrowserActions() returns " << visible_actions
- << " with size=" << browser_action_views_.size();
return visible_actions;
}
@@ -193,11 +191,7 @@ size_t BrowserActionsContainer::VisibleBrowserActionsAfterAnimation() const {
if (!animating())
return VisibleBrowserActions();
- size_t visible_actions = WidthToIconCount(animation_target_size_);
- VLOG(4) << "BAC::VisibleBrowserActionsAfterAnimation() returns "
- << visible_actions
- << " with size=" << browser_action_views_.size();
- return visible_actions;
+ return WidthToIconCount(animation_target_size_);
}
void BrowserActionsContainer::ExecuteExtensionCommand(
@@ -685,13 +679,10 @@ void BrowserActionsContainer::BrowserActionAdded(const Extension* extension,
#endif
CloseOverflowMenu();
- if (!ShouldDisplayBrowserAction(extension)) {
- VLOG(4) << "Should not display: " << extension->name().c_str();
+ if (!ShouldDisplayBrowserAction(extension))
return;
- }
size_t visible_actions = VisibleBrowserActionsAfterAnimation();
- VLOG(4) << "Got back " << visible_actions << " visible.";
// Add the new browser action to the vector and the view hierarchy.
if (profile_->IsOffTheRecord())
@@ -701,21 +692,17 @@ void BrowserActionsContainer::BrowserActionAdded(const Extension* extension,
AddChildViewAt(view, index);
// If we are still initializing the container, don't bother animating.
- if (!model_->extensions_initialized()) {
- VLOG(4) << "Still initializing";
+ if (!model_->extensions_initialized())
return;
- }
// Enlarge the container if it was already at maximum size and we're not in
// the middle of upgrading.
if ((model_->GetVisibleIconCount() < 0) &&
!extensions::ExtensionSystem::Get(profile_)->runtime_data()->
IsBeingUpgraded(extension)) {
- VLOG(4) << "At max, Save and animate";
suppress_chevron_ = true;
SaveDesiredSizeAndAnimate(gfx::Tween::LINEAR, visible_actions + 1);
} else {
- VLOG(4) << "Not at max";
// Just redraw the (possibly modified) visible icon set.
OnBrowserActionVisibilityChanged();
}
@@ -891,12 +878,8 @@ void BrowserActionsContainer::SaveDesiredSizeAndAnimate(
// NOTE: Don't save the icon count in incognito because there may be fewer
// icons in that mode. The result is that the container in a normal window is
// always at least as wide as in an incognito window.
- if (!profile_->IsOffTheRecord()) {
+ if (!profile_->IsOffTheRecord())
model_->SetVisibleIconCount(num_visible_icons);
- VLOG(4) << "Setting visible count: " << num_visible_icons;
- } else {
- VLOG(4) << "|Skipping| setting visible count: " << num_visible_icons;
- }
int target_size = IconCountToWidth(num_visible_icons,
num_visible_icons < browser_action_views_.size());
if (!disable_animations_during_testing_) {
« no previous file with comments | « chrome/browser/extensions/extension_toolbar_model_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698