Fail when saving page as MHTML provides information about the cause.

content/common/download/mhtml_save_status.cc

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "content/common/download/mhtml_save_status.h"
+
+#include "base/logging.h"
+#include "base/strings/stringprintf.h"

[nasko, 2016/11/23 18:07:34]

Why is this include needed?

[carlosk, 2016/11/23 20:08:40]

My mistake. Removed.

+
+namespace content {
+
+const char* GetMhtmlSaveStatusLabel(MhtmlSaveStatus save_status) {
+  switch (save_status) {
+    case MhtmlSaveStatus::SUCCESS:
+      return "Success";
+    case MhtmlSaveStatus::FILE_CLOSING_ERROR:
+      return "File closing error";
+    case MhtmlSaveStatus::FILE_CREATION_ERROR:
+      return "File creation error";
+    case MhtmlSaveStatus::FILE_WRITTING_ERROR:
+      return "File writing error";
+    case MhtmlSaveStatus::FRAME_NO_LONGER_EXISTS:
+      return "Frame no longer exists";
+    case MhtmlSaveStatus::FRAME_SERIALIZATION_FORBIDDEN:
+      return "Main frame serialization forbidden";
+    case MhtmlSaveStatus::RENDER_PROCESS_EXITED:
+      return "Render process no longer exists";
+    case MhtmlSaveStatus::STATUS_COUNT:
+      break;
+  }
+  NOTREACHED();
+  return "INVALID STATUS";

[nasko, 2016/11/23 18:07:34]

Keep strings consistent - you don't use all caps anywhere else, including
failures, so this should match them.

[carlosk, 2016/11/23 20:08:40]

This seemed like a big enough error case to flag it differently. I reverted to
not use all caps but added extra symbols to make it clear this is not an
expected status.

+}
+
+}  // namespace content