Compact Navigation prototype

chrome/browser/ui/views/dropdown_bar_host_delegate.h

+// Copyright (c) 2011 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.
+
+#ifndef CHROME_BROWSER_UI_VIEWS_DROPDOWN_BAR_HOST_DELEGATE_H_
+#define CHROME_BROWSER_UI_VIEWS_DROPDOWN_BAR_HOST_DELEGATE_H_
+
+class DropdownBarHostDelegate {
+ public:
+  virtual ~DropdownBarHostDelegate() {}

[sky, 2011/05/09 14:27:22]

Move this to the protected section.

[SteveT, 2011/05/09 19:40:04]

Done. For my understanding: My guess as to why you want this is to prevent the
delegate owners from destroying the delegate. Is that accurate? Should I add a
comment to that effect or do you consider that well understood? I don't see it
being done in some similar cases.

[sky, 2011/05/09 21:15:03]

It makes it clear that DropdownBarHost doesn't own the delegate. Well, I suppose
it could be away of a friend, but we typically don't do that.

+
+  // Claims focus for the text field and selects its contents.
+  virtual void SetFocusAndSelection(bool select_all) = 0;
+
+  // Updates the view to let it know where the host is clipping the
+  // dropdown widget (while animating the opening or closing of the widget).
+  virtual void SetAnimationOffset(int offset) = 0;
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_DROPDOWN_BAR_HOST_DELEGATE_H_