win: Fix a few sign mismatch warnings in crashpad.

win: Fix a few sign mismatch warnings in crashpad.

BUG=chromium:567877
R=mark@chromium.org, scottmg@chromium.org

Committed: https://chromium.googlesource.com/crashpad/crashpad/+/b9e732d3180f9eadc74d7229090e613d0700b8a6

[Nico, 2015-12-08 21:58:00]

thakis@chromium.org changed reviewers:
+ scottmg@chromium.org

[Mark Mentovai, 2015-12-08 22:05:58]

mark@chromium.org changed reviewers:
+ mark@chromium.org

[Mark Mentovai, 2015-12-08 22:05:59]

https://codereview.chromium.org/1503403003/diff/1/snapshot/win/process_reader...
File snapshot/win/process_reader_win.cc (right):

https://codereview.chromium.org/1503403003/diff/1/snapshot/win/process_reader...
snapshot/win/process_reader_win.cc:139: if (previous_suspend_count ==
static_cast<DWORD>(-1)) {
-1u?

[scottmg, 2015-12-08 22:06:00]

scottmg@chromium.org changed reviewers:
- mark@chromium.org

[scottmg, 2015-12-08 22:06:00]

lgtm

[Nico, 2015-12-08 22:07:12]

https://codereview.chromium.org/1503403003/diff/1/snapshot/win/process_reader...
File snapshot/win/process_reader_win.cc (right):

https://codereview.chromium.org/1503403003/diff/1/snapshot/win/process_reader...
snapshot/win/process_reader_win.cc:139: if (previous_suspend_count ==
static_cast<DWORD>(-1)) {
On 2015/12/08 22:05:59, Mark Mentovai wrote:
> -1u?

MSDN says that SuspendThread returns "(DWORD)-1" on error, I figured it's nicer
to look closer to that.

[Mark Mentovai, 2015-12-08 22:09:41]

mark@chromium.org changed reviewers:
+ mark@chromium.org

[Mark Mentovai, 2015-12-08 22:09:41]

OK, LGTM too

[Nico, 2015-12-08 22:20:26]

Looks like I don't have commit permissions. Please land.


error: failed to push some refs to
'https://chromium.googlesource.com/crashpad/crashpad.git
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
....

[scottmg, 2015-12-08 22:21:35]

Description was changed from

==========
win: Fix a few sign mismatch warnings in crashpad.

BUG=chromium:567877
==========

to

==========
win: Fix a few sign mismatch warnings in crashpad.

BUG=chromium:567877
R=mark@chromium.org, scottmg@chromium.org

Committed:
https://chromium.googlesource.com/crashpad/crashpad/+/b9e732d3180f9eadc74d722...
==========

[scottmg, 2015-12-08 22:21:37]

Committed patchset #1 (id:1) manually as
b9e732d3180f9eadc74d7229090e613d0700b8a6 (presubmit successful).