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

Issue 2705373005: Revert of Update Crashpad to 6da9708e7cc93e2e1772439d51646e47583cb225 (Closed)

Created:
3 years, 10 months ago by tzik
Modified:
3 years, 10 months ago
CC:
chromium-reviews, cbentzel+watch_chromium.org, asvitkine+watch_chromium.org, net-reviews_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Update Crashpad to 6da9708e7cc93e2e1772439d51646e47583cb225 Reason for revert: This CL causes a mac compile failure: https://build.chromium.org/p/chromium.mac/buildstatus?builder=Mac%20Builder&number=61782 ../../third_party/crashpad/crashpad/handler/handler_main.cc:166:7: error: unused variable 'rv' [-Werror,-Wunused-variable] int rv = sigaction(sig, &sa, nullptr); ^ ../../third_party/crashpad/crashpad/handler/handler_main.cc:241:9: error: unused variable 'rv' [-Werror,-Wunused-variable] int rv = raise(sig); ^ ../../third_party/crashpad/crashpad/handler/handler_main.cc:254:7: error: unused variable 'rv' [-Werror,-Wunused-variable] int rv = raise(sig); ^ Original issue's description: > Update Crashpad to 6da9708e7cc93e2e1772439d51646e47583cb225 > > 1f82c6cc8a35 Ensure Content-Length does not appear with Transfer- > Encoding > c1b305244aba Update mig.py to take an explicit sdk argument > d98a4de718d9 win: support native x86 CONTEXT structures with x87 but no > SSE context > 0c322ecc3f71 Use zlib to gzip-compress uploads > 546e64cd0b1b Centrally define CPUContextX86::Fsave and fsave↔︎fxsave > conversions > 4c6f6e52e2d7 Remove vestigial support for in-Chromium GYP build > f34ed66b93a0 metrics: Record handler lifetime milestone events > c1af20f1aa8a metrics: Consistently comment about enums used for metrics > 9c8407123796 Fix warning when building with clang on Windows > 6da9708e7cc9 doc: Fix Doxygen errors > > BUG=crashpad:100, crashpad:157, crashpad:159, crashpad:161, crashpad:162 > > Review-Url: https://codereview.chromium.org/2710663006 > Cr-Commit-Position: refs/heads/master@{#452387} TBR=mark@chromium.org, isherman@chromium.org, scottmg@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=crashpad:100, crashpad:157, crashpad:159, crashpad:161, crashpad:162 Review-Url: https://codereview.chromium.org/2705373005 Cr-Commit-Position: refs/heads/master@{#452409} Committed: https://chromium.googlesource.com/chromium/src/+/cc032bd4e8986d7c14c09d9a04c856395b90fe0b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+778 lines, -2533 lines) Patch
D build/secondary/third_party/crashpad/crashpad/third_party/zlib/BUILD.gn View 1 chunk +0 lines, -14 lines 0 comments Download
M build/secondary/third_party/crashpad/crashpad/util/BUILD.gn View 3 chunks +0 lines, -7 lines 0 comments Download
M third_party/crashpad/README.chromium View 2 chunks +2 lines, -2 lines 0 comments Download
M third_party/crashpad/crashpad/.gitignore View 1 chunk +0 lines, -1 line 0 comments Download
M third_party/crashpad/crashpad/DEPS View 1 chunk +0 lines, -3 lines 0 comments Download
M third_party/crashpad/crashpad/build/crashpad_dependencies.gypi View 2 chunks +12 lines, -10 lines 0 comments Download
M third_party/crashpad/crashpad/build/gyp_crashpad.py View 2 chunks +5 lines, -6 lines 0 comments Download
M third_party/crashpad/crashpad/handler/crash_report_upload_thread.h View 2 chunks +1 line, -4 lines 0 comments Download
M third_party/crashpad/crashpad/handler/crash_report_upload_thread.cc View 4 chunks +5 lines, -10 lines 0 comments Download
M third_party/crashpad/crashpad/handler/crashpad_handler.md View 1 chunk +0 lines, -8 lines 0 comments Download
M third_party/crashpad/crashpad/handler/handler.gyp View 2 chunks +24 lines, -0 lines 0 comments Download
M third_party/crashpad/crashpad/handler/handler_main.cc View 23 chunks +64 lines, -179 lines 0 comments Download
M third_party/crashpad/crashpad/minidump/minidump_context.h View 3 chunks +13 lines, -11 lines 0 comments Download
M third_party/crashpad/crashpad/minidump/minidump_context_writer.cc View 3 chunks +21 lines, -28 lines 0 comments Download
M third_party/crashpad/crashpad/minidump/test/minidump_context_test_util.cc View 5 chunks +66 lines, -35 lines 0 comments Download
M third_party/crashpad/crashpad/snapshot/cpu_context.h View 4 chunks +3 lines, -66 lines 0 comments Download
M third_party/crashpad/crashpad/snapshot/cpu_context.cc View 2 chunks +7 lines, -79 lines 0 comments Download
M third_party/crashpad/crashpad/snapshot/cpu_context_test.cc View 6 chunks +43 lines, -223 lines 0 comments Download
M third_party/crashpad/crashpad/snapshot/win/cpu_context_win.cc View 6 chunks +19 lines, -64 lines 0 comments Download
M third_party/crashpad/crashpad/snapshot/win/cpu_context_win_test.cc View 2 chunks +17 lines, -92 lines 0 comments Download
D third_party/crashpad/crashpad/test/hex_string.h View 1 chunk +0 lines, -40 lines 0 comments Download
D third_party/crashpad/crashpad/test/hex_string.cc View 1 chunk +0 lines, -35 lines 0 comments Download
D third_party/crashpad/crashpad/test/hex_string_test.cc View 1 chunk +0 lines, -34 lines 0 comments Download
M third_party/crashpad/crashpad/test/test.gyp View 1 chunk +0 lines, -2 lines 0 comments Download
M third_party/crashpad/crashpad/test/test_test.gyp View 1 chunk +0 lines, -1 line 0 comments Download
M third_party/crashpad/crashpad/third_party/gtest/gmock.gyp View 1 chunk +192 lines, -170 lines 0 comments Download
M third_party/crashpad/crashpad/third_party/gtest/gtest.gyp View 1 chunk +249 lines, -227 lines 0 comments Download
M third_party/crashpad/crashpad/third_party/mini_chromium/mini_chromium.gyp View 2 chunks +13 lines, -4 lines 0 comments Download
D third_party/crashpad/crashpad/third_party/zlib/README.crashpad View 1 chunk +0 lines, -18 lines 0 comments Download
D third_party/crashpad/crashpad/third_party/zlib/zlib.gyp View 1 chunk +0 lines, -137 lines 0 comments Download
D third_party/crashpad/crashpad/third_party/zlib/zlib_crashpad.h View 1 chunk +0 lines, -32 lines 0 comments Download
M third_party/crashpad/crashpad/util/misc/metrics.h View 4 chunks +6 lines, -43 lines 0 comments Download
M third_party/crashpad/crashpad/util/misc/metrics.cc View 1 chunk +0 lines, -8 lines 0 comments Download
D third_party/crashpad/crashpad/util/misc/zlib.h View 1 chunk +0 lines, -42 lines 0 comments Download
D third_party/crashpad/crashpad/util/misc/zlib.cc View 1 chunk +0 lines, -37 lines 0 comments Download
D third_party/crashpad/crashpad/util/net/http_body_gzip.h View 1 chunk +0 lines, -67 lines 0 comments Download
D third_party/crashpad/crashpad/util/net/http_body_gzip.cc View 1 chunk +0 lines, -126 lines 0 comments Download
D third_party/crashpad/crashpad/util/net/http_body_gzip_test.cc View 1 chunk +0 lines, -178 lines 0 comments Download
M third_party/crashpad/crashpad/util/net/http_headers.h View 1 chunk +0 lines, -3 lines 0 comments Download
M third_party/crashpad/crashpad/util/net/http_headers.cc View 1 chunk +1 line, -1 line 0 comments Download
M third_party/crashpad/crashpad/util/net/http_multipart_builder.h View 3 chunks +2 lines, -15 lines 0 comments Download
M third_party/crashpad/crashpad/util/net/http_multipart_builder.cc View 3 chunks +5 lines, -23 lines 0 comments Download
M third_party/crashpad/crashpad/util/net/http_multipart_builder_test.cc View 3 chunks +0 lines, -3 lines 0 comments Download
M third_party/crashpad/crashpad/util/net/http_transport_test.cc View 1 chunk +1 line, -15 lines 0 comments Download
M third_party/crashpad/crashpad/util/net/http_transport_test_server.py View 2 chunks +0 lines, -10 lines 0 comments Download
M third_party/crashpad/crashpad/util/thread/thread_posix.cc View 1 chunk +4 lines, -6 lines 0 comments Download
M third_party/crashpad/crashpad/util/thread/thread_win.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/crashpad/crashpad/util/util.gyp View 4 chunks +0 lines, -8 lines 0 comments Download
M third_party/crashpad/crashpad/util/util_test.gyp View 4 chunks +0 lines, -4 lines 0 comments Download
M third_party/crashpad/crashpad/util/win/scoped_handle.cc View 1 chunk +1 line, -1 line 0 comments Download
D third_party/crashpad/crashpad/util/win/session_end_watcher.h View 1 chunk +0 lines, -79 lines 0 comments Download
D third_party/crashpad/crashpad/util/win/session_end_watcher.cc View 1 chunk +0 lines, -243 lines 0 comments Download
D third_party/crashpad/crashpad/util/win/session_end_watcher_test.cc View 1 chunk +0 lines, -62 lines 0 comments Download
M tools/metrics/histograms/histograms.xml View 2 chunks +0 lines, -15 lines 0 comments Download

Messages

Total messages: 10 (6 generated)
tzik
3 years, 10 months ago (2017-02-23 05:38:26 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2705373005/1
3 years, 10 months ago (2017-02-23 05:40:19 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/cc032bd4e8986d7c14c09d9a04c856395b90fe0b
3 years, 10 months ago (2017-02-23 05:42:19 UTC) #9
Mark Mentovai
3 years, 10 months ago (2017-02-23 13:09:45 UTC) #10
Message was sent while issue was closed.
The fix was already in the CQ :(

https://codereview.chromium.org/2716563002/

Powered by Google App Engine
This is Rietveld 408576698