Index: chrome/browser/notifications/notification_platform_bridge_linux.cc |
diff --git a/chrome/browser/notifications/notification_platform_bridge_linux.cc b/chrome/browser/notifications/notification_platform_bridge_linux.cc |
index e6d39f84bbb9be230e605fa2279194d99b0a518e..f95332847fb55b674718c3aeedb718d952b1e6cc 100644 |
--- a/chrome/browser/notifications/notification_platform_bridge_linux.cc |
+++ b/chrome/browser/notifications/notification_platform_bridge_linux.cc |
@@ -7,34 +7,28 @@ |
#include <algorithm> |
#include "base/files/file_util.h" |
-#include "base/memory/ptr_util.h" |
-#include "base/stl_util.h" |
#include "base/strings/nullable_string16.h" |
#include "base/strings/string_number_conversions.h" |
#include "base/strings/string_util.h" |
#include "base/strings/utf_string_conversions.h" |
+#include "base/synchronization/waitable_event.h" |
#include "base/task_scheduler/post_task.h" |
#include "chrome/browser/browser_process.h" |
-#include "chrome/browser/chrome_notification_types.h" |
#include "chrome/browser/notifications/native_notification_display_service.h" |
#include "chrome/browser/notifications/notification.h" |
#include "chrome/browser/notifications/notification_display_service_factory.h" |
#include "chrome/browser/profiles/profile_manager.h" |
#include "chrome/browser/shell_integration_linux.h" |
-#include "content/public/browser/notification_service.h" |
+#include "content/public/browser/browser_thread.h" |
+#include "dbus/bus.h" |
+#include "dbus/message.h" |
+#include "dbus/object_proxy.h" |
namespace { |
const char kFreedesktopNotificationsName[] = "org.freedesktop.Notifications"; |
const char kFreedesktopNotificationsPath[] = "/org/freedesktop/Notifications"; |
-void AddActionToNotification(GVariantBuilder* actions_builder, |
- const char* action_id, |
- const char* button_label) { |
- g_variant_builder_add(actions_builder, "s", action_id); |
- g_variant_builder_add(actions_builder, "s", button_label); |
-} |
- |
int NotificationPriorityToFdoUrgency(int priority) { |
enum FdoUrgency { |
LOW = 0, |
@@ -50,29 +44,11 @@ int NotificationPriorityToFdoUrgency(int priority) { |
return CRITICAL; |
default: |
NOTREACHED(); |
- // fallthrough |
case message_center::DEFAULT_PRIORITY: |
return NORMAL; |
} |
} |
-// Callback used by GLib when the "Notify" message completes for the |
-// first time. |
-void NotifyCompleteReceiver(GObject* source_object, |
- GAsyncResult* result, |
- gpointer user_data) { |
- GDBusProxy* proxy = G_DBUS_PROXY(source_object); |
- GVariant* value = g_dbus_proxy_call_finish(proxy, result, nullptr); |
- if (!value) { |
- // The message might have been cancelled, in which case |
- // |user_data| points to a destroyed NotificationData. |
- return; |
- } |
- auto* platform_bridge_linux = static_cast<NotificationPlatformBridgeLinux*>( |
- g_browser_process->notification_platform_bridge()); |
- platform_bridge_linux->NotifyCompleteInternal(user_data, value); |
-} |
- |
// Runs once the profile has been loaded in order to perform a given |
// |operation| on a notification. |
void ProfileLoadedCallback(NotificationCommon::Operation operation, |
@@ -111,384 +87,334 @@ base::FilePath WriteDataToTmpFile( |
return file_path; |
} |
-void DeleteNotificationResourceFile(const base::FilePath& file_path) { |
- if (file_path.empty()) |
- return; |
- base::PostTaskWithTraits( |
- FROM_HERE, |
- base::TaskTraits() |
- .MayBlock() |
- .WithPriority(base::TaskPriority::BACKGROUND) |
- .WithShutdownBehavior(base::TaskShutdownBehavior::BLOCK_SHUTDOWN), |
- base::Bind(base::IgnoreResult(base::DeleteFile), file_path, false)); |
-} |
- |
} // namespace |
// static |
NotificationPlatformBridge* NotificationPlatformBridge::Create() { |
- GDBusProxy* notification_proxy = g_dbus_proxy_new_for_bus_sync( |
- G_BUS_TYPE_SESSION, |
- static_cast<GDBusProxyFlags>(G_DBUS_PROXY_FLAGS_DO_NOT_LOAD_PROPERTIES | |
- G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START), |
- nullptr, kFreedesktopNotificationsName, kFreedesktopNotificationsPath, |
- kFreedesktopNotificationsName, nullptr, nullptr); |
- if (!notification_proxy) |
- return nullptr; |
- return new NotificationPlatformBridgeLinux(notification_proxy); |
+ NotificationPlatformBridgeLinux* npbl = new NotificationPlatformBridgeLinux(); |
+ if (npbl->BlockUntilReady()) |
+ return npbl; |
+ delete npbl; |
+ return nullptr; |
} |
-struct NotificationPlatformBridgeLinux::NotificationData { |
- NotificationData(NotificationCommon::Type notification_type, |
- const std::string& notification_id, |
- const std::string& profile_id, |
- bool is_incognito, |
- const GURL& origin_url) |
- : notification_type(notification_type), |
- notification_id(notification_id), |
- profile_id(profile_id), |
- is_incognito(is_incognito), |
- origin_url(origin_url), |
- weak_factory(this) {} |
- |
- ~NotificationData() { |
- if (cancellable) |
- g_cancellable_cancel(cancellable); |
- ResetResourceFiles(); |
- } |
- |
- void ResetResourceFiles() { |
- for (const base::FilePath& file : resource_files) |
- DeleteNotificationResourceFile(file); |
- resource_files.clear(); |
+class NotificationPlatformBridgeLinux::NativeNotificationThread |
+ : public base::Thread { |
+ public: |
+ NativeNotificationThread() |
+ : base::Thread("FDO Notifications D-Bus thread"), |
+ event_(base::WaitableEvent::ResetPolicy::MANUAL, |
+ base::WaitableEvent::InitialState::NOT_SIGNALED) { |
+ base::Thread::Options thread_options(base::MessageLoop::TYPE_IO, 0); |
+ if (!StartWithOptions(thread_options)) |
+ event_.Signal(); |
} |
- // The ID used by the notification server. Will be 0 until the |
- // first "Notify" message completes. |
- uint32_t dbus_id = 0; |
- |
- // Same parameters used by NotificationPlatformBridge::Display(). |
- NotificationCommon::Type notification_type; |
- const std::string notification_id; |
- const std::string profile_id; |
- const bool is_incognito; |
- |
- // A copy of the origin_url from the underlying |
- // message_center::Notification. Used to pass back to |
- // NativeNotificationDisplayService. |
- const GURL origin_url; |
- |
- // Used to keep track of the IDs of the buttons currently displayed |
- // on this notification. The valid range of action IDs is |
- // [action_start, action_end). |
- size_t action_start = 0; |
- size_t action_end = 0; |
- |
- // Temporary resource files associated with the notification that |
- // should be cleaned up when the notification is closed or on |
- // shutdown. |
- std::vector<base::FilePath> resource_files; |
- |
- // Used to cancel the initial "Notify" message so we don't call |
- // NotificationPlatformBridgeLinux::NotifyCompleteInternal() with a |
- // destroyed Notification. |
- ScopedGObject<GCancellable> cancellable; |
- |
- // If not null, the data to update the notification with once |
- // |dbus_id| becomes available. |
- std::unique_ptr<Notification> update_data; |
- NotificationCommon::Type update_notification_type = |
- NotificationCommon::TYPE_MAX; |
- |
- // If true, indicates the notification should be closed once |
- // |dbus_id| becomes available. |
- bool should_close = false; |
- |
- base::WeakPtrFactory<NotificationData> weak_factory; |
-}; |
- |
-struct NotificationPlatformBridgeLinux::ResourceFiles { |
- explicit ResourceFiles(const base::FilePath& icon_file) |
- : icon_file(icon_file) {} |
- ~ResourceFiles() { DeleteNotificationResourceFile(icon_file); } |
- base::FilePath icon_file; |
-}; |
- |
-NotificationPlatformBridgeLinux::NotificationPlatformBridgeLinux( |
- GDBusProxy* notification_proxy) |
- : notification_proxy_(notification_proxy), weak_factory_(this) { |
- proxy_signal_handler_ = g_signal_connect( |
- notification_proxy_, "g-signal", G_CALLBACK(GSignalReceiverThunk), this); |
- registrar_.Add(this, chrome::NOTIFICATION_APP_TERMINATING, |
- content::NotificationService::AllSources()); |
-} |
- |
-NotificationPlatformBridgeLinux::~NotificationPlatformBridgeLinux() { |
- if (proxy_signal_handler_) |
- g_signal_handler_disconnect(notification_proxy_, proxy_signal_handler_); |
-} |
- |
-void NotificationPlatformBridgeLinux::Display( |
- NotificationCommon::Type notification_type, |
- const std::string& notification_id, |
- const std::string& profile_id, |
- bool is_incognito, |
- const Notification& notification) { |
- NotificationData* data = |
- FindNotificationData(notification_id, profile_id, is_incognito); |
- if (data) { |
- // Update an existing notification. |
- if (data->dbus_id) { |
+ ~NativeNotificationThread() override { Stop(); } |
+ |
+ void Display(NotificationCommon::Type notification_type, |
+ const std::string& notification_id, |
+ const std::string& profile_id, |
+ bool is_incognito, |
+ const Notification& notification) { |
+ NotificationData* data = |
+ FindNotificationData(notification_id, profile_id, is_incognito); |
+ if (data) { |
+ // Update an existing notification. |
data->notification_type = notification_type; |
- Notify(notification, data, nullptr, nullptr); |
+ data->resource_files.clear(); |
} else { |
- data->update_notification_type = notification_type; |
- data->update_data = base::MakeUnique<Notification>(notification); |
+ // Send the notification for the first time. |
+ data = |
+ new NotificationData(notification_type, notification_id, profile_id, |
+ is_incognito, notification.origin_url()); |
+ notifications_.emplace(data, base::WrapUnique(data)); |
+ } |
+ |
+ dbus::MethodCall method_call(kFreedesktopNotificationsName, "Notify"); |
+ 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(""); |
+ |
+ const std::string title = base::UTF16ToUTF8(notification.title()); |
+ writer.AppendString(title); |
+ |
+ const std::string message = base::UTF16ToUTF8(notification.message()); |
+ writer.AppendString(message); |
+ |
+ // Even-indexed elements in this vector are action IDs passed back to |
+ // us in GSignalReceiver. Odd-indexed ones contain the button text. |
hashimoto
2017/04/17 03:17:58
GSignalReceiver is no longer used.
Tom (Use chromium acct)
2017/04/17 18:07:13
Changed to OnActionInvoked()
|
+ std::vector<std::string> actions; |
+ data->action_start = data->action_end; |
+ for (const auto& button_info : notification.buttons()) { |
+ // FDO notification buttons can contain either an icon or a label, |
+ // but not both, and the type of all buttons must be the same (all |
+ // labels or all icons), so always use labels. |
+ std::string id = base::SizeTToString(data->action_end++); |
+ const std::string label = base::UTF16ToUTF8(button_info.title); |
+ actions.push_back(id); |
+ actions.push_back(label); |
+ } |
+ if (notification.clickable()) { |
+ // Special case: the pair ("default", "") will not add a button, |
+ // but instead makes the entire notification clickable. |
+ actions.push_back("default"); |
+ actions.push_back(""); |
+ } |
+ // Always add a settings button. |
+ actions.push_back("settings"); |
+ actions.push_back("Settings"); |
+ writer.AppendArrayOfStrings(actions); |
+ |
+ dbus::MessageWriter hints_writer(nullptr); |
+ writer.OpenArray("{sv}", &hints_writer); |
+ dbus::MessageWriter urgency_writer(nullptr); |
+ hints_writer.OpenDictEntry(&urgency_writer); |
+ urgency_writer.AppendString("urgency"); |
+ urgency_writer.AppendVariantOfUint32( |
+ NotificationPriorityToFdoUrgency(notification.priority())); |
+ hints_writer.CloseContainer(&urgency_writer); |
+ |
+ std::unique_ptr<base::Environment> env = base::Environment::Create(); |
+ std::string desktop_file = |
+ shell_integration_linux::GetDesktopName(env.get()); |
+ const char kDesktopFileSuffix[] = ".desktop"; |
+ DCHECK(base::EndsWith(desktop_file, kDesktopFileSuffix, |
+ base::CompareCase::SENSITIVE)); |
+ desktop_file = desktop_file.substr( |
+ 0, desktop_file.size() - strlen(kDesktopFileSuffix)); |
+ dbus::MessageWriter desktop_entry_writer(nullptr); |
+ hints_writer.OpenDictEntry(&desktop_entry_writer); |
+ desktop_entry_writer.AppendString("desktop-entry"); |
+ desktop_entry_writer.AppendVariantOfString(desktop_file); |
+ hints_writer.CloseContainer(&desktop_entry_writer); |
+ |
+ base::FilePath icon_file = |
+ WriteDataToTmpFile(notification.icon().As1xPNGBytes()); |
+ if (!icon_file.empty()) { |
+ dbus::MessageWriter image_path_writer(nullptr); |
+ hints_writer.OpenDictEntry(&image_path_writer); |
+ image_path_writer.AppendString("image-path"); |
+ image_path_writer.AppendVariantOfString(icon_file.value()); |
+ hints_writer.CloseContainer(&image_path_writer); |
+ data->resource_files.push_back(ResourceFile(icon_file)); |
+ } |
+ |
+ writer.CloseContainer(&hints_writer); |
+ |
+ int32_t expire_timeout = -1; |
+ writer.AppendInt32(expire_timeout); |
+ |
+ if (data->dbus_id) { |
+ notification_proxy_->CallMethod(&method_call, -1, |
hashimoto
2017/04/17 03:17:58
Please use dbus::ObjectProxy::TIMEOUT_USE_DEFAULT
Tom (Use chromium acct)
2017/04/17 18:07:13
Done.
|
+ base::Bind([](dbus::Response*) {})); |
+ } else { |
+ std::unique_ptr<dbus::Response> response = |
+ notification_proxy_->CallMethodAndBlock(&method_call, -1); |
+ if (response) { |
+ dbus::MessageReader reader(response.get()); |
+ reader.PopUint32(&data->dbus_id); |
+ } |
+ if (!data->dbus_id) { |
+ // There was some sort of error with creating the notification. |
+ notifications_.erase(data); |
+ } |
} |
- } else { |
- // Send the notification for the first time. |
- data = new NotificationData(notification_type, notification_id, profile_id, |
- is_incognito, notification.origin_url()); |
- data->cancellable.reset(g_cancellable_new()); |
- notifications_.emplace(data, base::WrapUnique(data)); |
- Notify(notification, data, NotifyCompleteReceiver, data); |
} |
-} |
- |
-void NotificationPlatformBridgeLinux::Close( |
- const std::string& profile_id, |
- const std::string& notification_id) { |
- std::vector<NotificationData*> to_erase; |
- for (const auto& pair : notifications_) { |
- NotificationData* data = pair.first; |
- if (data->notification_id == notification_id && |
- data->profile_id == profile_id) { |
- if (data->dbus_id) { |
- CloseNow(data->dbus_id); |
- to_erase.push_back(data); |
- } else { |
- data->should_close = true; |
+ |
+ void Close(const std::string& profile_id, |
+ const std::string& notification_id) { |
+ std::vector<NotificationData*> to_erase; |
+ for (const auto& pair : notifications_) { |
+ NotificationData* data = pair.first; |
+ if (data->notification_id == notification_id && |
+ data->profile_id == profile_id) { |
+ dbus::MethodCall method_call(kFreedesktopNotificationsName, |
+ "CloseNotification"); |
+ dbus::MessageWriter writer(&method_call); |
+ writer.AppendUint32(data->dbus_id); |
+ notification_proxy_->CallMethod(&method_call, -1, |
+ base::Bind([](dbus::Response*) {})); |
} |
} |
+ for (NotificationData* data : to_erase) |
+ notifications_.erase(data); |
+ } |
+ |
+ void GetDisplayed(const std::string& profile_id, |
+ bool incognito, |
+ const DisplayedNotificationsCallback& callback) const { |
+ content::BrowserThread::PostTask( |
+ content::BrowserThread::UI, FROM_HERE, |
+ base::Bind(callback, |
+ base::Passed(base::MakeUnique<std::set<std::string>>()), |
+ false)); |
} |
- for (NotificationData* data : to_erase) |
- notifications_.erase(data); |
-} |
- |
-void NotificationPlatformBridgeLinux::GetDisplayed( |
- const std::string& profile_id, |
- bool incognito, |
- const DisplayedNotificationsCallback& callback) const { |
- callback.Run(base::MakeUnique<std::set<std::string>>(), false); |
-} |
- |
-void NotificationPlatformBridgeLinux::NotifyCompleteInternal(gpointer user_data, |
- GVariant* value) { |
- NotificationData* data = reinterpret_cast<NotificationData*>(user_data); |
- if (!base::ContainsKey(notifications_, data)) |
- return; |
- data->cancellable.reset(); |
- if (value && g_variant_is_of_type(value, G_VARIANT_TYPE("(u)"))) |
- g_variant_get(value, "(u)", &data->dbus_id); |
- |
- if (!data->dbus_id) { |
- // There was some sort of error with creating the notification. |
- notifications_.erase(data); |
- } else if (data->should_close) { |
- CloseNow(data->dbus_id); |
- notifications_.erase(data); |
- } else if (data->update_data) { |
- data->notification_type = data->update_notification_type; |
- Notify(*data->update_data, data, nullptr, nullptr); |
- data->update_data.reset(); |
+ |
+ bool BlockUntilReady() { |
+ event_.Wait(); |
+ return notification_proxy_initialized_.IsSet(); |
} |
-} |
- |
-void NotificationPlatformBridgeLinux::Observe( |
- int type, |
- const content::NotificationSource& source, |
- const content::NotificationDetails& details) { |
- DCHECK_EQ(chrome::NOTIFICATION_APP_TERMINATING, type); |
- // The browser process is about to exit. Clean up all notification |
- // resource files. |
- notifications_.clear(); |
-} |
- |
-void NotificationPlatformBridgeLinux::Notify(const Notification& notification, |
- NotificationData* data, |
- GAsyncReadyCallback callback, |
- gpointer user_data) { |
- const scoped_refptr<base::RefCountedMemory> icon_data = |
- notification.icon().As1xPNGBytes(); |
- if (!icon_data->size()) { |
- NotifyNow(notification, data->weak_factory.GetWeakPtr(), callback, |
- user_data, base::MakeUnique<ResourceFiles>(base::FilePath())); |
- } else { |
- base::PostTaskWithTraitsAndReplyWithResult( |
- FROM_HERE, |
- base::TaskTraits() |
- .MayBlock() |
- .WithPriority(base::TaskPriority::USER_BLOCKING) |
- .WithShutdownBehavior(base::TaskShutdownBehavior::SKIP_ON_SHUTDOWN), |
+ |
+ private: |
+ struct ResourceFile { |
+ explicit ResourceFile(const base::FilePath& file_path) |
+ : file_path(file_path) {} |
+ ~ResourceFile() { base::DeleteFile(file_path, false); } |
+ base::FilePath file_path; |
+ }; |
+ |
+ struct NotificationData { |
+ NotificationData(NotificationCommon::Type notification_type, |
+ const std::string& notification_id, |
+ const std::string& profile_id, |
+ bool is_incognito, |
+ const GURL& origin_url) |
+ : notification_type(notification_type), |
+ notification_id(notification_id), |
+ profile_id(profile_id), |
+ is_incognito(is_incognito), |
+ origin_url(origin_url) {} |
+ |
+ // The ID used by the notification server. Will be 0 until the |
+ // first "Notify" message completes. |
+ uint32_t dbus_id = 0; |
+ |
+ // Same parameters used by NotificationPlatformBridge::Display(). |
+ NotificationCommon::Type notification_type; |
+ const std::string notification_id; |
+ const std::string profile_id; |
+ const bool is_incognito; |
+ |
+ // A copy of the origin_url from the underlying |
+ // message_center::Notification. Used to pass back to |
+ // NativeNotificationDisplayService. |
+ const GURL origin_url; |
+ |
+ // Used to keep track of the IDs of the buttons currently displayed |
+ // on this notification. The valid range of action IDs is |
+ // [action_start, action_end). |
+ size_t action_start = 0; |
+ size_t action_end = 0; |
+ |
+ // Temporary resource files associated with the notification that |
+ // should be cleaned up when the notification is closed or on |
+ // shutdown. |
+ std::vector<ResourceFile> resource_files; |
+ }; |
+ |
+ void Init() override { |
+ dbus::Bus::Options bus_options; |
+ bus_options.bus_type = dbus::Bus::SESSION; |
+ bus_options.connection_type = dbus::Bus::PRIVATE; |
+ bus_options.dbus_task_runner = task_runner(); |
hashimoto
2017/04/17 03:17:58
You don't need to fill dbus_task_runner here.
dbus
Tom (Use chromium acct)
2017/04/17 18:07:13
Done.
|
+ bus_ = new dbus::Bus(bus_options); |
+ |
+ notification_proxy_ = |
+ bus_->GetObjectProxy(kFreedesktopNotificationsName, |
+ dbus::ObjectPath(kFreedesktopNotificationsPath)); |
+ if (notification_proxy_) |
+ notification_proxy_initialized_.Set(); |
+ event_.Signal(); |
+ if (!notification_proxy_) |
+ return; |
+ |
+ auto on_signal_connected = [](const std::string&, const std::string&, |
+ bool success) { DCHECK(success); }; |
+ notification_proxy_->ConnectToSignal( |
+ kFreedesktopNotificationsName, "ActionInvoked", |
+ base::Bind(&NativeNotificationThread::OnActionInvoked, |
+ base::Unretained(this)), |
+ base::Bind(on_signal_connected)); |
+ notification_proxy_->ConnectToSignal( |
+ kFreedesktopNotificationsName, "NotificationClosed", |
+ base::Bind(&NativeNotificationThread::OnNotificationClosed, |
+ base::Unretained(this)), |
+ base::Bind(on_signal_connected)); |
+ } |
+ |
+ void CleanUp() override { |
+ bus_->ShutdownAndBlock(); |
+ notifications_.clear(); |
+ } |
+ |
+ NotificationData* FindNotificationData(const std::string& notification_id, |
+ const std::string& profile_id, |
+ bool is_incognito) { |
+ for (const auto& pair : notifications_) { |
+ NotificationData* data = pair.first; |
+ if (data->notification_id == notification_id && |
+ data->profile_id == profile_id && |
+ data->is_incognito == is_incognito) { |
+ return data; |
+ } |
+ } |
+ |
+ return nullptr; |
+ } |
+ |
+ NotificationData* FindNotificationData(uint32_t dbus_id) { |
+ for (const auto& pair : notifications_) { |
+ NotificationData* data = pair.first; |
+ if (data->dbus_id == dbus_id) |
+ return data; |
+ } |
+ |
+ return nullptr; |
+ } |
+ |
+ void ForwardNotificationOperation(uint32_t dbus_id, |
+ NotificationCommon::Operation operation, |
+ int action_index) { |
+ NotificationData* data = FindNotificationData(dbus_id); |
+ if (!data) { |
+ // This notification either belongs to a different app or we |
+ // already removed the NotificationData after sending a |
+ // "CloseNotification" message. |
+ return; |
+ } |
+ |
+ content::BrowserThread::PostTask( |
+ content::BrowserThread::UI, FROM_HERE, |
base::Bind( |
- [](scoped_refptr<base::RefCountedMemory> icon) { |
- return base::MakeUnique<ResourceFiles>(WriteDataToTmpFile(icon)); |
+ [](NotificationCommon::Operation operation, |
+ NotificationCommon::Type notification_type, |
+ const std::string& origin, const std::string& notification_id, |
+ int action_index, const std::string& profile_id, |
+ bool is_incognito) { |
+ ProfileManager* profile_manager = |
+ g_browser_process->profile_manager(); |
+ DCHECK(profile_manager); |
+ |
+ profile_manager->LoadProfile( |
+ profile_id, is_incognito, |
+ base::Bind(&ProfileLoadedCallback, operation, |
+ notification_type, origin, notification_id, |
+ action_index, base::NullableString16())); |
}, |
- icon_data), |
- base::Bind(&NotificationPlatformBridgeLinux::NotifyNow, |
- weak_factory_.GetWeakPtr(), notification, |
- data->weak_factory.GetWeakPtr(), callback, user_data)); |
+ operation, data->notification_type, data->origin_url.spec(), |
+ data->notification_id, action_index, data->profile_id, |
+ data->is_incognito)); |
} |
-} |
-void NotificationPlatformBridgeLinux::NotifyNow( |
- const Notification& notification, |
- base::WeakPtr<NotificationData> data, |
- GAsyncReadyCallback callback, |
- gpointer user_data, |
- std::unique_ptr<ResourceFiles> resource_files) { |
- if (!data) |
- return; |
+ void OnActionInvoked(dbus::Signal* signal) { |
+ dbus::MessageReader reader(signal); |
+ uint32_t dbus_id; |
+ if (!reader.PopUint32(&dbus_id)) |
+ return; |
+ std::string action; |
+ if (!reader.PopString(&action)) |
+ return; |
- if (data->dbus_id) |
- DCHECK(!data->cancellable); |
- |
- data->ResetResourceFiles(); |
- |
- GVariantBuilder actions_builder; |
- // Even-indexed elements in this array are action IDs passed back to |
- // us in GSignalReceiver. Odd-indexed ones contain the button text. |
- g_variant_builder_init(&actions_builder, G_VARIANT_TYPE("as")); |
- data->action_start = data->action_end; |
- for (const auto& button_info : notification.buttons()) { |
- // FDO notification buttons can contain either an icon or a label, |
- // but not both, and the type of all buttons must be the same (all |
- // labels or all icons), so always use labels. |
- std::string id = base::SizeTToString(data->action_end++); |
- const std::string label = base::UTF16ToUTF8(button_info.title); |
- AddActionToNotification(&actions_builder, id.c_str(), label.c_str()); |
- } |
- if (notification.clickable()) { |
- // Special case: the pair ("default", "") will not add a button, |
- // but instead makes the entire notification clickable. |
- AddActionToNotification(&actions_builder, "default", ""); |
- } |
- // Always add a settings button. |
- AddActionToNotification(&actions_builder, "settings", "Settings"); |
- |
- GVariantBuilder hints_builder; |
- g_variant_builder_init(&hints_builder, G_VARIANT_TYPE("a{sv}")); |
- |
- g_variant_builder_add(&hints_builder, "{sv}", "urgency", |
- g_variant_new_byte(NotificationPriorityToFdoUrgency( |
- notification.priority()))); |
- |
- std::unique_ptr<base::Environment> env = base::Environment::Create(); |
- std::string desktop_file = shell_integration_linux::GetDesktopName(env.get()); |
- const char kDesktopFileSuffix[] = ".desktop"; |
- DCHECK(base::EndsWith(desktop_file, kDesktopFileSuffix, |
- base::CompareCase::SENSITIVE)); |
- desktop_file = |
- desktop_file.substr(0, desktop_file.size() - strlen(kDesktopFileSuffix)); |
- g_variant_builder_add(&hints_builder, "{sv}", "desktop-entry", |
- g_variant_new_string(desktop_file.c_str())); |
- |
- if (!resource_files->icon_file.empty()) { |
- g_variant_builder_add( |
- &hints_builder, "{sv}", "image-path", |
- g_variant_new_string(resource_files->icon_file.value().c_str())); |
- data->resource_files.push_back(resource_files->icon_file); |
- resource_files->icon_file.clear(); |
- } |
- |
- const std::string title = base::UTF16ToUTF8(notification.title()); |
- const std::string message = base::UTF16ToUTF8(notification.message()); |
- |
- GVariant* parameters = g_variant_new( |
- "(susssasa{sv}i)", "" /* app_name passed implicitly via desktop-entry */, |
- data->dbus_id, "" /* app_icon passed implicitly via desktop-entry */, |
- title.c_str(), message.c_str(), &actions_builder, &hints_builder, -1); |
- g_dbus_proxy_call(notification_proxy_, "Notify", parameters, |
- G_DBUS_CALL_FLAGS_NONE, -1, data->cancellable, callback, |
- user_data); |
-} |
- |
-void NotificationPlatformBridgeLinux::CloseNow(uint32_t dbus_id) { |
- g_dbus_proxy_call(notification_proxy_, "CloseNotification", |
- g_variant_new("(u)", dbus_id), G_DBUS_CALL_FLAGS_NONE, -1, |
- nullptr, nullptr, nullptr); |
-} |
- |
-NotificationPlatformBridgeLinux::NotificationData* |
-NotificationPlatformBridgeLinux::FindNotificationData( |
- const std::string& notification_id, |
- const std::string& profile_id, |
- bool is_incognito) { |
- for (const auto& pair : notifications_) { |
- NotificationData* data = pair.first; |
- if (data->notification_id == notification_id && |
- data->profile_id == profile_id && data->is_incognito == is_incognito) { |
- return data; |
- } |
- } |
- |
- return nullptr; |
-} |
- |
-NotificationPlatformBridgeLinux::NotificationData* |
-NotificationPlatformBridgeLinux::FindNotificationData(uint32_t dbus_id) { |
- for (const auto& pair : notifications_) { |
- NotificationData* data = pair.first; |
- if (data->dbus_id == dbus_id) |
- return data; |
- } |
- |
- return nullptr; |
-} |
- |
-void NotificationPlatformBridgeLinux::ForwardNotificationOperation( |
- uint32_t dbus_id, |
- NotificationCommon::Operation operation, |
- int action_index) { |
- NotificationData* data = FindNotificationData(dbus_id); |
- if (!data) { |
- // This notification either belongs to a different app or we |
- // already removed the NotificationData after sending a |
- // "CloseNotification" message. |
- return; |
- } |
- |
- ProfileManager* profile_manager = g_browser_process->profile_manager(); |
- DCHECK(profile_manager); |
- |
- profile_manager->LoadProfile( |
- data->profile_id, data->is_incognito, |
- base::Bind(&ProfileLoadedCallback, operation, data->notification_type, |
- data->origin_url.spec(), data->notification_id, action_index, |
- base::NullableString16())); |
-} |
- |
-void NotificationPlatformBridgeLinux::GSignalReceiver(GDBusProxy* proxy, |
- const char* sender_name, |
- const char* sender_signal, |
- GVariant* parameters) { |
- uint32_t dbus_id = 0; |
- if (strcmp("NotificationClosed", sender_signal) == 0 && |
- g_variant_is_of_type(parameters, G_VARIANT_TYPE("(uu)"))) { |
- uint32_t reason; |
- g_variant_get(parameters, "(uu)", &dbus_id, &reason); |
- ForwardNotificationOperation(dbus_id, NotificationCommon::CLOSE, -1); |
- // std::unordered_map::erase(nullptr) is safe here. |
- notifications_.erase(FindNotificationData(dbus_id)); |
- } else if (strcmp("ActionInvoked", sender_signal) == 0 && |
- g_variant_is_of_type(parameters, G_VARIANT_TYPE("(us)"))) { |
- const gchar* action = nullptr; |
- g_variant_get(parameters, "(u&s)", &dbus_id, &action); |
- DCHECK(action); |
- |
- if (strcmp(action, "default") == 0) { |
+ if (action == "default") { |
ForwardNotificationOperation(dbus_id, NotificationCommon::CLICK, -1); |
- } else if (strcmp(action, "settings") == 0) { |
+ } else if (action == "settings") { |
ForwardNotificationOperation(dbus_id, NotificationCommon::SETTINGS, -1); |
} else { |
size_t id; |
@@ -505,4 +431,74 @@ void NotificationPlatformBridgeLinux::GSignalReceiver(GDBusProxy* proxy, |
id_zero_based); |
} |
} |
+ |
+ void OnNotificationClosed(dbus::Signal* signal) { |
+ dbus::MessageReader reader(signal); |
+ uint32_t dbus_id; |
+ if (!reader.PopUint32(&dbus_id)) |
+ return; |
+ ForwardNotificationOperation(dbus_id, NotificationCommon::CLOSE, -1); |
+ // std::unordered_map::erase(nullptr) is safe here. |
+ notifications_.erase(FindNotificationData(dbus_id)); |
+ } |
+ |
+ scoped_refptr<dbus::Bus> bus_; |
+ |
+ dbus::ObjectProxy* notification_proxy_ = nullptr; |
+ // Event that will be signaled once |notification_proxy_| is done |
+ // connecting, or has failed to connect. |
+ base::WaitableEvent event_; |
+ |
+ base::AtomicFlag notification_proxy_initialized_; |
+ |
+ // A std::set<std::unique_ptr<T>> doesn't work well because |
+ // eg. std::set::erase(T) would require a std::unique_ptr<T> |
+ // argument, so the data would get double-destructed. |
+ template <typename T> |
+ using UnorderedUniqueSet = std::unordered_map<T*, std::unique_ptr<T>>; |
+ |
+ UnorderedUniqueSet<NotificationData> notifications_; |
+}; |
+ |
+NotificationPlatformBridgeLinux::NotificationPlatformBridgeLinux() |
+ : thread_(new NativeNotificationThread()) {} |
+ |
+NotificationPlatformBridgeLinux::~NotificationPlatformBridgeLinux() {} |
+ |
+void NotificationPlatformBridgeLinux::Display( |
+ NotificationCommon::Type notification_type, |
+ const std::string& notification_id, |
+ const std::string& profile_id, |
+ bool is_incognito, |
+ const Notification& notification) { |
+ // TODO(thomasanderson): Subclass Notification and add pass it |
+ // thread-safe style to the other thread. |
+ thread_->task_runner()->PostTask( |
+ FROM_HERE, |
+ base::Bind(&NativeNotificationThread::Display, |
+ base::Unretained(thread_.get()), notification_type, |
+ notification_id, profile_id, is_incognito, notification)); |
+} |
+ |
+void NotificationPlatformBridgeLinux::Close( |
+ const std::string& profile_id, |
+ const std::string& notification_id) { |
+ thread_->task_runner()->PostTask( |
+ FROM_HERE, |
+ base::Bind(&NativeNotificationThread::Close, |
+ base::Unretained(thread_.get()), profile_id, notification_id)); |
+} |
+ |
+void NotificationPlatformBridgeLinux::GetDisplayed( |
+ const std::string& profile_id, |
+ bool incognito, |
+ const DisplayedNotificationsCallback& callback) const { |
+ thread_->task_runner()->PostTask( |
+ FROM_HERE, base::Bind(&NativeNotificationThread::GetDisplayed, |
+ base::Unretained(thread_.get()), profile_id, |
+ incognito, callback)); |
+} |
+ |
+bool NotificationPlatformBridgeLinux::BlockUntilReady() { |
+ return thread_->BlockUntilReady(); |
} |