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

Unified Diff: components/offline_pages/core/offline_page_model_impl.cc

Issue 2810913002: [Offline pages] Add new Error page failure status inside the MHTML archiver and update correspondin… (Closed)
Patch Set: Created 3 years, 8 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
Index: components/offline_pages/core/offline_page_model_impl.cc
diff --git a/components/offline_pages/core/offline_page_model_impl.cc b/components/offline_pages/core/offline_page_model_impl.cc
index 2338be5a990958fb3060ab4193bb540b625f2c3f..96587fe36352a0a45d542f9268dedfca9467b7cd 100644
--- a/components/offline_pages/core/offline_page_model_impl.cc
+++ b/components/offline_pages/core/offline_page_model_impl.cc
@@ -72,6 +72,9 @@ SavePageResult ToSavePageResult(ArchiverResult archiver_result) {
case ArchiverResult::ERROR_SECURITY_CERTIFICATE:
result = SavePageResult::SECURITY_CERTIFICATE_ERROR;
break;
+ case ArchiverResult::ERROR_PAGE:
+ result = SavePageResult::ERROR_PAGE_DETECTED;
fgorski 2017/04/10 21:19:17 fix this somehow as well, please.
chili 2017/04/11 20:40:16 Done.
+ break;
default:
NOTREACHED();
result = SavePageResult::CONTENT_UNAVAILABLE;

Powered by Google App Engine
This is Rietveld 408576698