diff options
author | Kirill Burtsev <kirill.burtsev@qt.io> | 2022-03-24 13:46:51 +0100 |
---|---|---|
committer | Kirill Burtsev <kirill.burtsev@qt.io> | 2022-04-18 08:42:38 +0200 |
commit | 4c4ac0df9cea39b8d2a78c3563dda5fee199c9e8 (patch) | |
tree | 1cebfd425ff88d4828c2d169280a48c9cf9f3b9a /src/core/web_engine_error.cpp | |
parent | 20c4e8bc408770c714ea3f7c6465bc4ca65d3c09 (diff) |
Introduce http status code domain for loading info
Fix inconsistency in reporting load info error category for http status
codes, which across versions was reported either as internal error
category or just no error domain. Add new separate domain for http
status codes.
[ChangeLog][QWebEngineLoadingInfo] Added HttpStatusCodeDomain for http status
code range of errors
Fixes: QTBUG-94963
Change-Id: I9fd496248c6fa33c424d758e9a0be99758aaf061
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Benjamin Terrier <b.terrier@gmail.com>
Reviewed-by: Michal Klocek <michal.klocek@qt.io>
Diffstat (limited to 'src/core/web_engine_error.cpp')
-rw-r--r-- | src/core/web_engine_error.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/core/web_engine_error.cpp b/src/core/web_engine_error.cpp index a6580d76b..ff46be5c8 100644 --- a/src/core/web_engine_error.cpp +++ b/src/core/web_engine_error.cpp @@ -65,6 +65,10 @@ const int endErrors = -900; WebEngineError::ErrorDomain WebEngineError::toQtErrorDomain(int error_code) { + // net errors are always negative values, and https response codes are positive + if (error_code > 0) + return HttpStatusCodeDomain; + // Chromium's ranges from net/base/net_error_list.h: // 0 No error // 1- 99 System related errors |