[NTP] Propagate OfflineId from C++ for Downloads.

chrome/browser/ntp_snippets/download_suggestions_provider.cc

Index: chrome/browser/ntp_snippets/download_suggestions_provider.cc
diff --git a/chrome/browser/ntp_snippets/download_suggestions_provider.cc b/chrome/browser/ntp_snippets/download_suggestions_provider.cc
index c5ef7dda97185091978abecbf1b093c809bb452e..332dfeb93bc242d2da83b31f3630949f365b95b0 100644
--- a/chrome/browser/ntp_snippets/download_suggestions_provider.cc
+++ b/chrome/browser/ntp_snippets/download_suggestions_provider.cc
@@ -488,6 +488,7 @@ ContentSuggestion DownloadSuggestionsProvider::ConvertOfflinePage(
   suggestion.set_publisher_name(base::UTF8ToUTF16(offline_page.url.host()));
   auto extra = base::MakeUnique<ntp_snippets::DownloadSuggestionExtra>();
   extra->is_download_asset = false;
+  extra->offline_page_id = base::Int64ToString(offline_page.offline_id);

[Marc Treib, 2016/11/16 15:22:58]

Er, any reason why we pass this as a string? Seems like the Java side just
converts it back to an int?

[vitaliii, 2016/11/16 15:45:08]

As we discussed it is not clear how to match int64 on Java side, so TODO for
now.

[Marc Treib, 2016/11/16 15:55:56]

There's a jlong analogous to jint, I'm pretty sure that does the right thing :)
Anyway, this is fine in a followup.

[vitaliii, 2016/11/16 15:59:23]

Acknowledged.

   suggestion.set_download_suggestion_extra(std::move(extra));
   return suggestion;
 }