BrowserActionsContainer breaks for nobody... or maybe someone

chrome/browser/views/browser_actions_container.cc

 // Copyright (c) 2009 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 "chrome/browser/views/browser_actions_container.h"
 
 #include "app/gfx/canvas.h"
 #include "app/resource_bundle.h"
 #include "base/stl_util-inl.h"
 #include "base/string_util.h"
@@ skipping 402 matching lines @@
       RemoveBrowserAction(Details<Extension>(details).ptr());
       OnBrowserActionVisibilityChanged();
       break;
 
     case NotificationType::EXTENSION_HOST_VIEW_SHOULD_CLOSE:
       // If we aren't the host of the popup, then disregard the notification.
       if (!popup_ || Details<ExtensionHost>(popup_->host()) != details)
         return;
 
       HidePopup();
+      break;
 
     default:
       NOTREACHED() << L"Unexpected notification";

[Evan Stade, 2009/11/03 20:11:53]

while you're here, could you remove this unsightly L?

   }
 }
 
 void BrowserActionsContainer::BubbleBrowserWindowMoved(BrowserBubble* bubble) {
 }
 
 void BrowserActionsContainer::BubbleBrowserWindowClosing(
     BrowserBubble* bubble) {
   HidePopup();
 }
@@ skipping 22 matching lines @@
       std::min(static_cast<int>(browser_action_views_.size()),
                kMinimumNumberOfVisibleBrowserActions) * kButtonSize;
 
   // Even if available_width is <= 0, we still return at least the |min_width|.
   if (available_width <= 0)
     return min_width;
 
   return std::max(min_width, available_width - available_width % kButtonSize +
                   kHorizontalPadding * 2);
 }