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

Issue 10541044: Base: Add another pc to the handle verifier to get to the (Closed)

Created:
8 years, 6 months ago by rvargas (doing something else)
Modified:
8 years, 6 months ago
CC:
chromium-reviews, erikwright (departed), brettw-cc_chromium.org
Visibility:
Public.

Description

Base: Add another pc to the handle verifier to get to the actual caller. I was hoping for an inlining that didn't happened. BUG=127931 TEST=none original review: https://chromiumcodereview.appspot.com/10537041 TBR=willchan Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140962

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -13 lines) Patch
M base/win/scoped_handle.h View 1 7 chunks +28 lines, -9 lines 0 comments Download
M base/win/scoped_handle.cc View 4 chunks +5 lines, -4 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
rvargas (doing something else)
Patch set 2 fixes the build :(.
8 years, 6 months ago (2012-06-07 00:53:02 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rvargas@chromium.org/10541044/4
8 years, 6 months ago (2012-06-07 03:47:12 UTC) #2
willchan no longer on Chromium
Lgtm On Jun 6, 2012 8:47 PM, <commit-bot@chromium.org> wrote: > CQ is trying da patch. ...
8 years, 6 months ago (2012-06-07 04:20:03 UTC) #3
commit-bot: I haz the power
8 years, 6 months ago (2012-06-07 05:32:35 UTC) #4
Change committed as 140962

Powered by Google App Engine
This is Rietveld 408576698