Linux native notifications: Add percentage to title for progress notifications

chrome/browser/notifications/notification_platform_bridge_linux.cc

 // Copyright 2017 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.
 
 #include "chrome/browser/notifications/notification_platform_bridge_linux.h"
 
 #include <algorithm>
 #include <memory>
 #include <set>
 #include <unordered_map>
@@ skipping 414 matching lines @@
     dbus::MessageWriter writer(&method_call);
 
     // app_name passed implicitly via desktop-entry.
     writer.AppendString("");
 
     writer.AppendUint32(data->dbus_id);
 
     // app_icon passed implicitly via desktop-entry.
     writer.AppendString("");
 
-    writer.AppendString(base::UTF16ToUTF8(notification->title()));
+    std::string summary = base::UTF16ToUTF8(notification->title());
+    if (notification->type() == message_center::NOTIFICATION_TYPE_PROGRESS)
+      summary = base::IntToString(notification->progress()) + "% - " + summary;

[Peter Beverloo, 2017/05/09 22:51:54]

nit: prefer base::FormatPercent()

You should probably be able to copy/paste the implementation here:

https://cs.chromium.org/chromium/src/chrome/browser/notifications/notificatio...

(We'll generalize on the third user, which I'm fairly sure is coming up soon.)

[Tom (Use chromium acct), 2017/05/10 20:10:20]

Done.

+    writer.AppendString(summary);
 
     std::string body;
     if (base::ContainsKey(capabilities_, "body")) {
       body = base::UTF16ToUTF8(notification->message());
       if (base::ContainsKey(capabilities_, "body-markup")) {
         base::ReplaceSubstringsAfterOffset(&body, 0, "&", "&");
         base::ReplaceSubstringsAfterOffset(&body, 0, "<", "&lt;");
         base::ReplaceSubstringsAfterOffset(&body, 0, ">", "&gt;");
       }
     }
@@ skipping 342 matching lines @@
 }
 
 void NotificationPlatformBridgeLinux::SetReadyCallback(
     NotificationBridgeReadyCallback callback) {
   impl_->SetReadyCallback(std::move(callback));
 }
 
 void NotificationPlatformBridgeLinux::CleanUp() {
   impl_->CleanUp();
 }