Add BubbleManager to manage bubbles and ChromeBubbleManager for events.

components/bubble/bubble_ui.h

+// Copyright 2015 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 COMPONENTS_BUBBLE_BUBBLE_UI_H_
+#define COMPONENTS_BUBBLE_BUBBLE_UI_H_
+
+class BubbleUI {
+ public:
+  virtual ~BubbleUI() {}

[msw, 2015/08/13 03:37:23]

nit: Should this be defined in a separate .cc file? (or perhaps inline the
BubbleDelegate ctor/dtor if this is really okay)

[hcarmona, 2015/08/15 02:03:21]

Done.

+
+  // Should display the bubble UI

[msw, 2015/08/13 03:37:23]

nit: trailing period here and on the other function comments.

[hcarmona, 2015/08/15 02:03:21]

Done.

+  virtual void Show() = 0;
+
+  // Should hide the bubble UI
+  virtual void Hide() = 0;
+
+  // Should update the bubble UI's position
+  virtual void UpdateAnchorPosition() = 0;
+
+  // Should return true if the bubble's contents can be updated.

[msw, 2015/08/13 03:37:23]

As I mentioned before, I'm not entirely clear on what this means. Does it mean
updating the anchor position, regenerating the bubble ui instance, or something
else. Please clarify the comments for ignorant readers like myself.

[hcarmona, 2015/08/15 02:03:21]

Acknowledged.

+  virtual bool CanAcceptUpdate() const = 0;

[msw, 2015/08/13 03:37:24]

Consider finding common terminology between this and
BubbleController::ShouldUpdateBubble

[hcarmona, 2015/08/15 02:03:21]

Acknowledged.

+};
+
+#endif  // COMPONENTS_BUBBLE_BUBBLE_UI_H_