Showing previews UI for Offline Previews

chrome/browser/previews/previews_infobar_delegate.cc

Index: chrome/browser/previews/previews_infobar_delegate.cc
diff --git a/chrome/browser/previews/previews_infobar_delegate.cc b/chrome/browser/previews/previews_infobar_delegate.cc
index 275811a4a1cf47db5412207bee0ff20c2beec63c..f1eaa491ab0f7abbae2ee67ddf27e9116eaa0f9b 100644
--- a/chrome/browser/previews/previews_infobar_delegate.cc
+++ b/chrome/browser/previews/previews_infobar_delegate.cc
@@ -113,28 +113,28 @@ base::string16 PreviewsInfoBarDelegate::GetMessageText() const {
 int PreviewsInfoBarDelegate::GetButtons() const {
   return BUTTON_NONE;
 }
 
 base::string16 PreviewsInfoBarDelegate::GetLinkText() const {
   return l10n_util::GetStringUTF16(IDS_PREVIEWS_INFOBAR_LINK);
 }
 
 bool PreviewsInfoBarDelegate::LinkClicked(WindowOpenDisposition disposition) {
   RecordPreviewsInfoBarAction(infobar_type_, INFOBAR_LOAD_ORIGINAL_CLICKED);
-
+  content::WebContents* web_contents =
+      InfoBarService::WebContentsFromInfoBar(infobar());

[megjablon, 2016/10/26 22:01:11]

Any reason why this was changed from auto*?

[RyanSturm, 2016/10/27 00:01:01]

It's unclear from the Style guide if auto is really allowed here, but I prefer
declaring types except for iterators. I think I added it for myself and forgot
to remove it.

   if (infobar_type_ == LITE_PAGE || infobar_type_ == LOFI) {
-    auto* web_contents =
-        InfoBarService::WebContentsFromInfoBar(infobar());
-
     if (infobar_type_ == LITE_PAGE)
       web_contents->GetController().ReloadDisableLoFi(true);
     else if (infobar_type_ == LOFI)
       web_contents->ReloadLoFiImages();
 
     auto* data_reduction_proxy_settings =
         DataReductionProxyChromeSettingsFactory::GetForBrowserContext(
             web_contents->GetBrowserContext());
     data_reduction_proxy_settings->IncrementLoFiUserRequestsForImages();
+  } else if (infobar_type_ == OFFLINE) {
+    web_contents->GetController().Reload(true);
   }
 
   return true;
 }