Open Reading List on tap on More

ios/chrome/browser/content_suggestions/content_suggestions_coordinator.mm

 // Copyright 2017 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.
 
 #import "ios/chrome/browser/content_suggestions/content_suggestions_coordinator.h"
 
 #include "base/mac/scoped_nsobject.h"
 #include "base/metrics/user_metrics.h"
 #include "base/metrics/user_metrics_action.h"
 #include "base/strings/sys_string_conversions.h"
 #include "components/ntp_snippets/content_suggestions_service.h"
 #include "components/ntp_snippets/remote/remote_suggestions_scheduler.h"
 #include "components/reading_list/core/reading_list_model.h"
 #import "ios/chrome/browser/content_suggestions/content_suggestions_mediator.h"
 #include "ios/chrome/browser/favicon/ios_chrome_large_icon_service_factory.h"
 #include "ios/chrome/browser/ntp_snippets/ios_chrome_content_suggestions_service_factory.h"
 #include "ios/chrome/browser/reading_list/reading_list_model_factory.h"
 #import "ios/chrome/browser/ui/alert_coordinator/action_sheet_coordinator.h"
+#import "ios/chrome/browser/ui/commands/UIKit+ChromeExecuteCommand.h"
+#import "ios/chrome/browser/ui/commands/generic_chrome_command.h"
+#include "ios/chrome/browser/ui/commands/ios_command_ids.h"
 #import "ios/chrome/browser/ui/content_suggestions/cells/content_suggestions_article_item.h"
 #import "ios/chrome/browser/ui/content_suggestions/content_suggestions_commands.h"
 #import "ios/chrome/browser/ui/content_suggestions/content_suggestions_view_controller.h"
 #import "ios/chrome/browser/ui/content_suggestions/identifier/content_suggestion_identifier.h"
 #import "ios/chrome/browser/ui/url_loader.h"
 #include "ios/chrome/grit/ios_strings.h"
 #include "ios/web/public/referrer.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/strings/grit/ui_strings.h"
 
@@ skipping 40 matching lines @@
 
   ntp_snippets::ContentSuggestionsService* contentSuggestionsService =
       IOSChromeContentSuggestionsServiceFactory::GetForBrowserState(
           self.browserState);
   contentSuggestionsService->remote_suggestions_scheduler()->OnNTPOpened();
 
   self.contentSuggestionsMediator = [[ContentSuggestionsMediator alloc]
       initWithContentService:contentSuggestionsService
             largeIconService:IOSChromeLargeIconServiceFactory::
                                  GetForBrowserState(self.browserState)];
+  self.contentSuggestionsMediator.commandHandler = self;
 
   self.suggestionsViewController = [[ContentSuggestionsViewController alloc]
       initWithStyle:CollectionViewControllerStyleDefault
          dataSource:self.contentSuggestionsMediator];
 
   self.suggestionsViewController.suggestionCommandHandler = self;
   _navigationController = [[UINavigationController alloc]
       initWithRootViewController:self.suggestionsViewController];
 
   self.suggestionsViewController.navigationItem.leftBarButtonItem =
@@ skipping 14 matching lines @@
                          completion:nil];
   self.navigationController = nil;
   self.contentSuggestionsMediator = nil;
   self.alertCoordinator = nil;
   _visible = NO;
 }
 
 #pragma mark - ContentSuggestionsCommands
 
 - (void)openReadingList {
+  [self.baseViewController
+      chromeExecuteCommand:[GenericChromeCommand
+                               commandWithTag:IDC_SHOW_READING_LIST]];

[stkhapugin, 2017/04/05 13:06:02]

This is ugly. Isn't it supposed to be done through coordinators somehow?

[gambard, 2017/04/05 15:30:17]

Yes in bling*... For now this is the way (from marq).

 }
 
 - (void)openFirstPageOfReadingList {
 }
 
 - (void)openFaviconAtIndex:(NSInteger)index {
 }
 
 - (void)openURL:(const GURL&)URL {
   // TODO(crbug.com/691979): Add metrics.
@@ skipping 107 matching lines @@
   if (!article)
     return;
 
   // TODO(crbug.com/691979): Add metrics.
   [self.contentSuggestionsMediator
       dismissSuggestion:article.suggestionIdentifier];
   [self.suggestionsViewController dismissEntryAtIndexPath:indexPath];
 }
 
 @end