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

Side by Side Diff: content/common/download/mhtml_save_status.h

Issue 2519273002: Fail when saving page as MHTML provides information about the cause. (Closed)
Patch Set: Minor changes. Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CONTENT_COMMON_DOWNLOAD_MHTML_SAVE_STATUS_H_
6 #define CONTENT_COMMON_DOWNLOAD_MHTML_SAVE_STATUS_H_
7
8 namespace content {
9
10 // Status result enum for MHTML generation.
11 enum class MhtmlSaveStatus {
12 SUCCESS = 0,
13
14 // Could not properly close the file where data was written to. Determined by
15 // the browser.
16 FILE_CLOSING_ERROR,
17
18 // Could not create the file that would be written to. Determined by the
19 // browser.
20 FILE_CREATION_ERROR,
21
22 // Could not write serialized data to the file. Determined by the renderer.
23 FILE_WRITTING_ERROR,
24
25 // The DOM changed and a previously existing frame is no more. Determined by
26 // the browser.
27 FRAME_NO_LONGER_EXIST,
Łukasz Anforowicz 2016/11/22 19:22:13 nit: FRAME_NO_LONGER_EXISTS
carlosk 2016/11/22 23:26:24 Done.
28
29 // A frame was not allowed to be serialized by either policy or cache control.
Łukasz Anforowicz 2016/11/22 19:22:13 nit: s/A frame/The main frame/ ?
carlosk 2016/11/22 23:26:23 Done.
30 // Determined by the renderer.
31 FRAME_SERIALIZATION_FORBIDDEN,
32
33 // A render process needed for the serialization of one of the page's frame is
34 // no more. Determined by the browser.
35 RENDER_PROCESS_EXITED,
36
37 // NOTE: always keep this entry at the end. Add new result types only
38 // immediately above this line. Make sure to update the corresponding
39 // histogram enum accordingly.
40 STATUS_COUNT,
41 };
42
43 } // namespace content
44
45 #endif // CONTENT_COMMON_DOWNLOAD_MHTML_SAVE_STATUS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698