Issue 1107 - Support notifications

include/AdblockPlus/Notification.h

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 #ifndef ADBLOCK_PLUS_NOTIFICATION_H
 #define ADBLOCK_PLUS_NOTIFICATION_H
 
 #include <string>
 #include <vector>
-#include <memory>
+#include "tr1_memory.h"
 
 namespace AdblockPlus
 {
   class FilterEngine;
+  /**
+   * Possible notification types.
+   */
   enum NotificationType
   {
-    NOTIFICATION_TYPE_INFORMATION, NOTIFICATION_TYPE_QUESTION, NOTIFICATION_TYPE_CRITICAL

[Wladimir Palant, 2015/01/22 15:19:51]

Nit: listing these one per line would be better, this line is too long right
now.

[sergei, 2015/01/22 16:15:11]

fixed

+    NOTIFICATION_TYPE_INFORMATION,
+    NOTIFICATION_TYPE_QUESTION,
+    NOTIFICATION_TYPE_CRITICAL
   };
 
+  /**
+   * Wrapper for an Adblock Plus notification object.
+   */
   class Notification: public JsValue
   {
     friend class FilterEngine;
   protected:
-    struct PrivateCtrArg{};
     static std::tr1::shared_ptr<Notification> JsValueToNotification(const JsValuePtr& jsValue);
+    /**
+     * Constructor.
+     * @param jsValue `JsValuePtr` notification JavaScript object.
+     */
+    explicit Notification(const JsValuePtr& jsValue);
   public:
-    explicit Notification(const JsValuePtr& jsValue, PrivateCtrArg);
+    /**
+     * Retrieves the type of this notification.
+     * @return Type of this notification.
+     */
     NotificationType GetType() const;
+
     /**
-     * Localizes the texts of the supplied notification.
-     * @return the translated texts.
+     * Retrieves the title of this notification.
+     * @return Title of this notification.
      */
     const std::string& GetTitle() const;
+
+    /**
+     * Retrieves the message of this notification.
+     * @return Message of this notification.
+     */
     const std::string& GetMessageString() const;

[sergei, 2015/01/22 14:08:15]

It's not GetMessage because it's not possible to call it when windows.h is
included in the same cpp file due to #define GetMessage GetMessageW in
windows.h.

+
+    /**
+     * Marks this notification as shown. It is only relevant for question
+     * notifications. Other notifications are marked automatically.
+     */
     void MarkAsShown();
   private:
     std::string title;
     std::string message;
     NotificationType type;
   };
   typedef std::tr1::shared_ptr<Notification> NotificationPtr;
 }
 
 #endif