Adding new general bubble error message consisting of icon, message and caption. Using this message…

ash/popup_message.h

+// Copyright (c) 2013 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 ASH_POPUP_MESSAGE_H_
+#define ASH_POPUP_MESSAGE_H_
+
+#include "ash/ash_export.h"
+#include "base/basictypes.h"
+#include "base/string16.h"
+#include "ui/gfx/rect.h"
+#include "ui/views/bubble/bubble_border.h"
+#include "ui/views/bubble/bubble_delegate.h"
+
+namespace views {
+class BubbleDelegateView;
+}
+
+namespace ash {
+
+// PopupMessage shows a message to the user. Since the user is not able to
+// dismiss it, the calling code needs to explictly close and destroy it.
+class ASH_EXPORT PopupMessage {
+ public:
+  enum IconType {
+    ICON_WARNING,
+    ICON_NONE
+  };
+
+  // Creates a message pointing towards |anchor| with the requested
+  // |arrow_orientation|. The message contains an optional |caption| which is
+  // drawn in bold and an optional |message| together with an optional icon of
+  // shape |message_type|. If a component in |size_override| is not 0 the value
+  // is the used as output size. If |arrow_offset| is not 0, the number is the
+  // arrow offset in pixels from the border.
+  PopupMessage(const base::string16& caption,

[msw, 2013/05/21 01:41:27]

Could the GlobalErrorBubble be extended for this use case? Or could you expose
and re-use the error message appearance (and/or other components) of its
internal Views implementation? That might be better than re-implementing
something similar but separate.

[Mr4D (OOO till 08-26), 2013/05/21 17:26:40]

I have never seen that thing being used - even more so I didn't even know that
existed.

Looking at the views version of this bubble I can see that there are about 20
lines of "real code" and 160 lines of construction of the elements (which would
possibly be replaced for that). Furthermore it seems not remotely to be what is
being asked for. As such I would say that the idea to use a general class is
good (in fact I asked around before implementing and no one knew about this) but
that thing is too far away from what we want. (It is meant to be a targeted
notification).

So the answer would be no.

+               const base::string16& message,
+               IconType message_type,
+               views::View* anchor,
+               views::BubbleBorder::Arrow arrow,
+               const gfx::Size& size_override,
+               int arrow_offset);
+  // If the message was not explicitly closed before, it closes the message
+  // without animation.
+  virtual ~PopupMessage();
+
+  // Closes the message with a fade out animation.
+  void Close();
+
+ private:
+  class MessageBubble;
+
+  void CancelHidingAnimation();
+
+  MessageBubble* view_;
+  views::Widget* widget_;
+
+  // Variables of the construction time.
+  views::View* anchor_;
+  base::string16 caption_;
+  base::string16 message_;
+  IconType message_type_;
+  views::BubbleBorder::Arrow arrow_orientation_;
+
+
+  DISALLOW_COPY_AND_ASSIGN(PopupMessage);
+};
+
+}  // namespace ash
+
+#endif  // ASH_POPUP_MESSAGE_H_