Issue 4127 - [emscripten] Convert subscription classes to C++ - Part 2

compiled/subscription/Subscription.cpp

 #include <cstdio>
 #include <cstdlib>
 
 #include "Subscription.h"
 #include "DownloadableSubscription.h"
 #include "UserDefinedSubscription.h"
 #include "../StringMap.h"
 
 namespace
 {
   StringMap<Subscription*> knownSubscriptions(16);
 }
 
 Subscription::Subscription(Type type, const String& id)
     : mType(type), mID(id), mDisabled(false)
 {
   annotate_address(this, "Subscription");
 }
 
 Subscription::~Subscription()
 {
   knownSubscriptions.erase(mID);
 }
 
+Filter* Subscription::FilterAt(unsigned index)
+{
+  if (index >= mFilters.size())
+    return nullptr;
+
+  FilterPtr result(mFilters[index]);
+  return result.release();
+}
+
+int Subscription::IndexOfFilter(Filter* filter)
+{
+  for (unsigned i = 0; i < mFilters.size(); i++)
+    if (mFilters[i] == filter)
+      return i;
+  return -1;
+}
+
 OwnedString Subscription::Serialize() const
 {
   OwnedString result(u"[Subscription]\nurl="_str);
   result.append(mID);
   result.append(u'\n');
   if (!mTitle.empty())
   {
     result.append(u"title="_str);
     result.append(mTitle);
     result.append(u'\n');
   }
   if (mDisabled)
     result.append(u"disabled=true\n"_str);
 
   return result;
 }
 
 OwnedString Subscription::SerializeFilters() const
 {
-  // TODO
-  return OwnedString();
+  if (!mFilters.size())
+    return OwnedString();
+
+  OwnedString result(u"[Subscription filters]\n"_str);
+  for (unsigned i = 0; i < mFilters.size(); i++)
+  {
+    // TODO: Escape [ characters

[Wladimir Palant, 2017/03/16 18:31:23]

Not fixing this at this point because the serialization code will have to be
replaced anyway as part of https://issues.adblockplus.org/ticket/4128.

+    result.append(mFilters[i]->GetText());
+    result.append(u'\n');
+  }
+  return result;
 }
 
 Subscription* Subscription::FromID(const String& id)
 {
   if (id.empty())
   {
     // Generate a new random ID
     unsigned seed = knownSubscriptions.size();
     OwnedString randomID(u"~user~000000"_str);
     do
@@ skipping 10 matching lines @@
 
   auto knownSubscription = knownSubscriptions.find(id);
   if (knownSubscription)
   {
     knownSubscription->second->AddRef();
     return knownSubscription->second;
   }
 
   SubscriptionPtr subscription;
   if (!id.empty() && id[0] == '~')
-    subscription = new UserDefinedSubscription(id);
+    subscription = SubscriptionPtr(new UserDefinedSubscription(id), false);
   else
-    subscription = new DownloadableSubscription(id);
+    subscription = SubscriptionPtr(new DownloadableSubscription(id), false);
 
   // This is a hack: we looked up the entry using id but create it using
   // subscription->mID. This works because both are equal at this point.
   // However, id refers to a temporary buffer which will go away.
   enter_context("Adding to known subscriptions");
   knownSubscription.assign(subscription->mID, subscription.get());
   exit_context();
 
   return subscription.release();
 }