Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Unified Diff: ios/chrome/browser/content_suggestions/content_suggestions_coordinator.mm

Issue 2766713002: Add "Read Later" action to ContentSuggestions' context menu (Closed)
Patch Set: Removing guards Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/chrome/browser/content_suggestions/BUILD.gn ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/content_suggestions/content_suggestions_coordinator.mm
diff --git a/ios/chrome/browser/content_suggestions/content_suggestions_coordinator.mm b/ios/chrome/browser/content_suggestions/content_suggestions_coordinator.mm
index 69a72e087de15935d07e14083d0fae8b25ff43d6..7f40bd732839e1fa7bc4e8d8935414d6d43365da 100644
--- a/ios/chrome/browser/content_suggestions/content_suggestions_coordinator.mm
+++ b/ios/chrome/browser/content_suggestions/content_suggestions_coordinator.mm
@@ -5,11 +5,15 @@
#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/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/content_suggestions/cells/content_suggestions_article_item.h"
#import "ios/chrome/browser/ui/content_suggestions/content_suggestions_commands.h"
@@ -135,6 +139,7 @@
__weak ContentSuggestionsCoordinator* weakSelf = self;
GURL articleURL = articleItem.articleURL;
+ NSString* articleTitle = articleItem.title;
__weak ContentSuggestionsArticleItem* weakArticle = articleItem;
NSString* openInNewTabTitle =
@@ -157,6 +162,28 @@
}
style:UIAlertActionStyleDefault];
+ NSString* readLaterTitle =
+ l10n_util::GetNSString(IDS_IOS_CONTENT_CONTEXT_ADDTOREADINGLIST);
+ [self.alertCoordinator
+ addItemWithTitle:readLaterTitle
+ action:^{
+ ContentSuggestionsCoordinator* strongSelf = weakSelf;
+ if (!strongSelf)
+ return;
+
+ base::RecordAction(
+ base::UserMetricsAction("MobileReadingListAdd"));
+ // TODO(crbug.com/691979): Add metrics.
+
+ ReadingListModel* readingModel =
+ ReadingListModelFactory::GetForBrowserState(
+ strongSelf.browserState);
+ readingModel->AddEntry(articleURL,
+ base::SysNSStringToUTF8(articleTitle),
+ reading_list::ADDED_VIA_CURRENT_APP);
+ }
+ style:UIAlertActionStyleDefault];
+
NSString* deleteTitle =
l10n_util::GetNSString(IDS_IOS_CONTENT_SUGGESTIONS_DELETE);
[self.alertCoordinator addItemWithTitle:deleteTitle
« no previous file with comments | « ios/chrome/browser/content_suggestions/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698