[Extensions Toolbar] Protect against crazy bounds

chrome/browser/ui/toolbar/toolbar_actions_model.h

Index: chrome/browser/ui/toolbar/toolbar_actions_model.h
diff --git a/chrome/browser/ui/toolbar/toolbar_actions_model.h b/chrome/browser/ui/toolbar/toolbar_actions_model.h
index af7b0bd16cdbb336a5db199a9c59e5c0cadf6d93..684f485928472c5654dd23af124756bfa2a88ebb 100644
--- a/chrome/browser/ui/toolbar/toolbar_actions_model.h
+++ b/chrome/browser/ui/toolbar/toolbar_actions_model.h
@@ -28,7 +28,10 @@ class ExtensionRegistry;
 class ExtensionSet;
 }
 
-// Model for the browser actions toolbar.
+// Model for the browser actions toolbar. This is a per-profile instance, and
+// manages the user's global preferences. These will typically be reflected in
+// each window, but can be overridden in special circumstances by the window's
+// ToolbarActionsBar (see also that class for details).

[Peter Kasting, 2015/09/17 19:59:53]

Nit: Maybe instead of the sentence about "special circumstances" you could more
specifically say "when the window is narrow" or the like?  Something like:

Each browser window will attempt to show browser actions as specified by this
model, but if the window is too narrow, actions may end up pushed into the
overflow menu on a per-window basis.  Callers interested in the arrangement of
actions in a particular window should check that window's instance of
ToolbarActionsBar, which is responsible for this per-window overflow.

[Devlin, 2015/09/17 21:38:38]

Done.

 class ToolbarActionsModel : public extensions::ExtensionActionAPI::Observer,
                             public extensions::ExtensionRegistryObserver,
                             public KeyedService {