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

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: add comments and fix tests 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..fd8c266a90582b3c39f94c5704b1eccf8fb03558 100644
--- a/components/offline_pages/core/offline_page_model_impl.cc
+++ b/components/offline_pages/core/offline_page_model_impl.cc
@@ -72,6 +72,11 @@ SavePageResult ToSavePageResult(ArchiverResult archiver_result) {
case ArchiverResult::ERROR_SECURITY_CERTIFICATE:
result = SavePageResult::SECURITY_CERTIFICATE_ERROR;
break;
+ case ArchiverResult::ERROR_ERROR_PAGE:
+ result = SavePageResult::ERROR_PAGE;
+ break;
+ case ArchiverResult::ERROR_INTERSTITIAL_PAGE:
+ result = SavePageResult::INTERSTITIAL_PAGE;
default:
NOTREACHED();
result = SavePageResult::CONTENT_UNAVAILABLE;
« no previous file with comments | « components/offline_pages/core/offline_page_archiver.h ('k') | components/offline_pages/core/offline_page_types.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698