Commit 7caea766 authored by Xi Cheng's avatar Xi Cheng Committed by Commit Bot

Tweak the comment in BuildNotificationTemplate()

Change-Id: I5059c4f7e1351ea62749fd381fd6fea3a812f749
Reviewed-on: https://chromium-review.googlesource.com/c/1319345Reviewed-by: default avatarGreg Thompson <grt@chromium.org>
Commit-Queue: Xi Cheng <chengx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#605751}
parent cf73ee3c
......@@ -334,15 +334,16 @@ void EnsureReminderHasButton(XmlWriter* xml_writer,
const char kNotificationToastElement[] = "toast";
const char kNotificationLaunchAttribute[] = "launch";
// libXml was preferred (over WinXml, which the samples in the link below tend
// to use) for building the XML template because it is used frequently in
// Chrome, is nicer to use and has already been vetted.
// https://docs.microsoft.com/en-us/windows/uwp/controls-and-patterns/tiles-and-notifications-adaptive-interactive-toasts
base::string16 BuildNotificationTemplate(
NotificationImageRetainer* image_retainer,
const NotificationLaunchId& launch_id,
const message_center::Notification& notification) {
DCHECK(image_retainer);
// libXml was preferred (over WinXml, which the samples tend to use) because
// it is used frequently in Chrome, is nicer to use and has already been
// vetted.
XmlWriter xml_writer;
xml_writer.StartWriting();
......
......@@ -23,7 +23,6 @@ extern const char kNotificationLaunchAttribute[];
// Builds XML-based notification template for displaying a given |notification|
// in the Windows Action Center.
// https://docs.microsoft.com/en-us/windows/uwp/controls-and-patterns/tiles-and-notifications-adaptive-interactive-toasts
base::string16 BuildNotificationTemplate(
NotificationImageRetainer* image_retainer,
const NotificationLaunchId& launch_id,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment