Elide origins displayed on web notifications.

ui/message_center/notification.h

Index: ui/message_center/notification.h
diff --git a/ui/message_center/notification.h b/ui/message_center/notification.h
index 122cd6760629823e67d51cc789869a27269cad54..489255e0d95ee59bf88ae4f2b5511559b1ecae7f 100644
--- a/ui/message_center/notification.h
+++ b/ui/message_center/notification.h
@@ -33,6 +33,13 @@ struct MESSAGE_CENTER_EXPORT ButtonInfo {
   ButtonInfo(const base::string16& title);
 };
 
+struct MESSAGE_CENTER_EXPORT ContextMessage {

[Jun Mukai, 2015/08/14 17:17:07]

I prefer to keep string16 for context message. Could you:
- move the 'origin_url' field from chrome/browser/notifications/notification.h
to ui/message_center/notification.h
- use this origin_url if context_message is empty (and origin_url isn't
extension origin), apply the url_formatter to the origin_url

[Miguel Garcia, 2015/08/14 18:28:45]

Will give a shot, now that you mention it I recall Peter saying exactly the same
thing about not switching to std::string but I cannot remeber the reasoning,
could you elaborate why not make the change?

[Jun Mukai, 2015/08/17 18:36:14]

string16 is the text representation on UI side, so it's a good design to convert
into string16 at the earliest point and keep it as far as possible -- also it
will reduce the number of conversions / copies.

I think, however, in your new design, this might not satisfy because you want to
the message into a GURL anyways.
That's why I suggested to hold the GURL instance itself, I think it would reduce
the number of conversions between GURL<->string as well.

+  ContextMessage(std::string msg, bool origin)
+      : message(msg), is_origin(origin) {}
+  std::string message;
+  bool is_origin;
+};
+
 class MESSAGE_CENTER_EXPORT RichNotificationData {
  public:
   RichNotificationData();
@@ -42,7 +49,7 @@ class MESSAGE_CENTER_EXPORT RichNotificationData {
   int priority;
   bool never_timeout;
   base::Time timestamp;
-  base::string16 context_message;
+  ContextMessage context_message;
   gfx::Image image;
   gfx::Image small_image;
   std::vector<NotificationItem> items;
@@ -123,11 +130,13 @@ class MESSAGE_CENTER_EXPORT Notification {
     optional_fields_.timestamp = timestamp;
   }
 
-  const base::string16& context_message() const {
+  const ContextMessage context_message() const {
     return optional_fields_.context_message;
   }
-  void set_context_message(const base::string16& context_message) {
-    optional_fields_.context_message = context_message;
+
+  void set_context_message(const std::string& context_message, bool is_origin) {
+    optional_fields_.context_message.message = context_message;
+    optional_fields_.context_message.is_origin = is_origin;
   }
 
   const std::vector<NotificationItem>& items() const {