Commit b6314282 authored by Jian Li's avatar Jian Li Committed by Commit Bot

Check for null loadUrlParams

Bug: 897670
Change-Id: Ib5f7892332b8f6129fd9e5822aef096574856dd3
Reviewed-on: https://chromium-review.googlesource.com/c/1294834Reviewed-by: default avatarTheresa <twellington@chromium.org>
Commit-Queue: Jian Li <jianli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#601751}
parent 48029885
...@@ -104,6 +104,7 @@ public class SuggestionsNavigationDelegate extends NativePageNavigationDelegateI ...@@ -104,6 +104,7 @@ public class SuggestionsNavigationDelegate extends NativePageNavigationDelegateI
OfflinePageUtils.getLoadUrlParamsForOpeningOfflineVersion(article.mUrl, OfflinePageUtils.getLoadUrlParamsForOpeningOfflineVersion(article.mUrl,
article.getOfflinePageOfflineId(), LaunchLocation.SUGGESTION, article.getOfflinePageOfflineId(), LaunchLocation.SUGGESTION,
(loadUrlParams) -> { (loadUrlParams) -> {
if (loadUrlParams == null) return;
// Extra headers are not read in loadUrl, but verbatim headers are. // Extra headers are not read in loadUrl, but verbatim headers are.
loadUrlParams.setVerbatimHeaders(loadUrlParams.getExtraHeadersString()); loadUrlParams.setVerbatimHeaders(loadUrlParams.getExtraHeadersString());
openDownloadSuggestion(windowOpenDisposition, article, loadUrlParams); openDownloadSuggestion(windowOpenDisposition, article, loadUrlParams);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment