Synced Notification Sync Change Processor

chrome/browser/notifier/chrome_notifier_delegate.cc

Index: chrome/browser/notifier/chrome_notifier_delegate.cc
diff --git a/chrome/browser/notifier/chrome_notifier_delegate.cc b/chrome/browser/notifier/chrome_notifier_delegate.cc
new file mode 100644
index 0000000000000000000000000000000000000000..406fe7267e3a2d2f65bc202f0a37c4a9c4658aed
--- /dev/null
+++ b/chrome/browser/notifier/chrome_notifier_delegate.cc
@@ -0,0 +1,15 @@
+// Copyright (c) 2012 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.
+
+// ChromeNotifierDelegate is a NotificationDelegate which catches
+// responses from the NotificationUIManager when a notification
+// has been closed.

[dcheng, 2013/01/18 21:56:13]

Duplicate comment? This would be a good place to talk about tricky
implementation details if needed.

[Pete Williamson, 2013/01/23 01:45:55]

Yep, this is a dup, removed.
(my theory is that class comment duplication is good, I've removed it since you
seem to disagree.)

+
+#include "chrome/browser/notifier/chrome_notifier_delegate.h"
+
+#include <string>

[dcheng, 2013/01/18 21:56:13]

You shouldn't need this #include here or the .h. If you do, that's a sign that a
header file elsewhere isn't including it when it should be.

[Pete Williamson, 2013/01/23 01:45:55]

Done.

+
+void ChromeNotifierDelegate::Close(bool by_user) {
+  // TODO(petewil): implement
+}