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

Issue 1286383006: Fix port of CrashReportUploadThread to windows (Closed)

Created:
5 years, 4 months ago by scottmg
Modified:
5 years, 4 months ago
Reviewers:
Mark Mentovai
CC:
crashpad-dev_chromium.org, mihnea_arkaos.net
Base URL:
https://chromium.googlesource.com/crashpad/crashpad@master
Target Ref:
refs/heads/master
Project:
crashpad
Visibility:
Public.

Description

Fix port of CrashReportUploadThread to windows util/thread/ doesn't automatically start, contrary to previous pthread implementation. Thanks to mihnea@arkaos.net for the patch. R=mark@chromium.org BUG=https://codereview.chromium.org/1295363002/#msg5 Committed: https://chromium.googlesource.com/crashpad/crashpad/+/4f2d2018de3d898529d7f3ce2bfdf11eb7b543c2

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M handler/crash_report_upload_thread.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
scottmg
5 years, 4 months ago (2015-08-19 16:33:04 UTC) #1
Mark Mentovai
LGTM
5 years, 4 months ago (2015-08-19 16:39:52 UTC) #2
scottmg
5 years, 4 months ago (2015-08-19 16:41:57 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
4f2d2018de3d898529d7f3ce2bfdf11eb7b543c2 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698