[Android] Implement GetRuleIterator for channels provider

chrome/browser/notifications/notification_channels_provider_android.h

Index: chrome/browser/notifications/notification_channels_provider_android.h
diff --git a/chrome/browser/notifications/notification_channels_provider_android.h b/chrome/browser/notifications/notification_channels_provider_android.h
index 8c629070276f818c4a5752a3701cf2bb903faad4..f1e399e7965076df3b30d6122a0acf76c14c6134 100644
--- a/chrome/browser/notifications/notification_channels_provider_android.h
+++ b/chrome/browser/notifications/notification_channels_provider_android.h
@@ -21,6 +21,11 @@ namespace {
 // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.chrome.browser.notifications
 enum NotificationChannelStatus { ENABLED, BLOCKED, UNAVAILABLE };
 
+struct Channel {
+  std::string origin;
+  NotificationChannelStatus status;

[Peter Beverloo, 2017/06/02 15:14:26]

nit: give a default value

[awdf, 2017/06/02 16:59:48]

Done.

+};
+
 }  // anonymous namespace

[Peter Beverloo, 2017/06/02 15:14:26]

Sorry that I missed this earlier - we shouldn't be using an anonymous namespace
in a header file, is doesn't mean anything. If you qualify Channel as
NotificationChannel we can just have them on the global scope. Alternatively you
can add them as members of NotificationChannelsProviderAndroid.

[awdf, 2017/06/02 16:59:48]

Done.

 
 // This class provides notification content settings from system notification
@@ -39,6 +44,7 @@ class NotificationChannelsProviderAndroid
     virtual NotificationChannelStatus GetChannelStatus(
         const std::string& origin) = 0;
     virtual void DeleteChannel(const std::string& origin) = 0;
+    virtual std::vector<Channel> GetChannels() = 0;
   };
 
   NotificationChannelsProviderAndroid();