Suggestions UI - favicon item

ios/showcase/suggestions/sc_suggestions_coordinator.mm

Index: ios/showcase/suggestions/sc_suggestions_coordinator.mm
diff --git a/ios/showcase/suggestions/sc_suggestions_coordinator.mm b/ios/showcase/suggestions/sc_suggestions_coordinator.mm
index 06c6feb1c4692016878f56078b222c4355f6d4d9..79eaf32a8f704888b12e830611b13ca5ff192498 100644
--- a/ios/showcase/suggestions/sc_suggestions_coordinator.mm
+++ b/ios/showcase/suggestions/sc_suggestions_coordinator.mm
@@ -59,4 +59,8 @@
       openFirstPageOfReadingList];
 }
 
+- (void)openFaviconAtIndex:(NSInteger)index {
+  [static_cast<id<SuggestionsCommands>>(self.alerter) openFaviconAtIndex:index];

[marq (ping after 24h), 2017/01/19 16:20:26]

I'm going to again suggest you just directly assign the alerter as
_suggestionViewController.suggestionCommandHandler instead of manually
forwarding each method in this way. That's the intended use, specifically so
that when commands are added to a protocol they are supported in the showcase
without any additional code.

[marq (ping after 24h), 2017/01/19 16:20:26]

Oh, and a minor nit: my current understanding is that reinterpret_cast<> is the
right choice here.

[gambard, 2017/01/19 17:37:32]

Done.

[gambard, 2017/01/19 17:37:32]

Done.

+}
+
 @end