Linux native notifications: Delete notification after sending CloseNotification.

Linux native notifications: Delete notification after sending CloseNotification.

Also check the "dbus_id" when handling signals.

Fix some nits as well.

BUG=676220
Review-Url: https://codereview.chromium.org/2859483003
Cr-Commit-Position: refs/heads/master@{#469148}
Committed: https://chromium.googlesource.com/chromium/src/+/8cf43de8a87be9447615ea8bf8965c9979ff18f1

[Lei Zhang, 2017-05-03 07:54:55]

The CQ bit was checked by thestig@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-03 07:55:04]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-03 08:39:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-03 08:39:02]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2017-05-03 20:56:23]

thestig@chromium.org changed reviewers:
+ thomasanderson@google.com

[Tom (Use chromium acct), 2017-05-03 21:42:56]

lgtm

https://codereview.chromium.org/2859483003/diff/1/chrome/browser/notification...
File chrome/browser/notifications/notification_platform_bridge_linux.cc (right):

https://codereview.chromium.org/2859483003/diff/1/chrome/browser/notification...
chrome/browser/notifications/notification_platform_bridge_linux.cc:12: #include
<vector>
out of curiosity, did you do this by hand?

[Lei Zhang, 2017-05-03 21:45:24]

https://codereview.chromium.org/2859483003/diff/1/chrome/browser/notification...
File chrome/browser/notifications/notification_platform_bridge_linux.cc (right):

https://codereview.chromium.org/2859483003/diff/1/chrome/browser/notification...
chrome/browser/notifications/notification_platform_bridge_linux.cc:12: #include
<vector>
On 2017/05/03 21:42:56, Tom Anderson wrote:
> out of curiosity, did you do this by hand?

"git cl lint" checks for these.

[Lei Zhang, 2017-05-03 21:45:33]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2017-05-03 21:46:05]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-03 21:46:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-03 21:46:06]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[Tom Anderson, 2017-05-03 21:48:06]

thomasanderson@chromium.org changed reviewers:
+ thomasanderson@chromium.org

[Tom Anderson, 2017-05-03 21:48:06]

The CQ bit was checked by thomasanderson@chromium.org

[Tom Anderson, 2017-05-03 21:48:06]

lgtm

[commit-bot: I haz the power, 2017-05-03 21:48:44]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-03 21:57:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493848086423770, "parent_rev":
"462abbbe5b9206198afacbb76f5286da80de1106", "commit_rev":
"8cf43de8a87be9447615ea8bf8965c9979ff18f1"}

[commit-bot: I haz the power, 2017-05-03 21:58:01]

Description was changed from

==========
Linux native notifications: Delete notification after sending CloseNotification.

Also check the "dbus_id" when handling signals.

Fix some nits as well.

BUG=676220
==========

to

==========
Linux native notifications: Delete notification after sending CloseNotification.

Also check the "dbus_id" when handling signals.

Fix some nits as well.

BUG=676220

Review-Url: https://codereview.chromium.org/2859483003
Cr-Commit-Position: refs/heads/master@{#469148}
Committed:
https://chromium.googlesource.com/chromium/src/+/8cf43de8a87be9447615ea8bf896...
==========

[commit-bot: I haz the power, 2017-05-03 21:58:02]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8cf43de8a87be9447615ea8bf896...