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

Issue 12330079: Revert 184133 (Closed)

Created:
7 years, 10 months ago by vandebo (ex-Chrome)
Modified:
7 years, 10 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 184133 It failed in this way: [ RUN ] HKDFTest.HKDFConstruct c:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\vector(932) : Assertion failed: vector subscript out of range c:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\vector(933) : Assertion failed: "Standard C++ Libraries Out of Range" && 0 http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%28dbg%29%281%29/builds/33019/steps/crypto_unittests/logs/stdio > Porting of HKDF changes from server. > > Merge internal CL: 40300624 > > > Review URL: https://chromiumcodereview.appspot.com/12326029 TBR=rtenneti@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=184146

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -251 lines) Patch
MM crypto/crypto.gyp View 2 chunks +0 lines, -3 lines 0 comments Download
D crypto/hkdf.h View 1 chunk +0 lines, -64 lines 0 comments Download
D crypto/hkdf.cc View 1 chunk +0 lines, -92 lines 0 comments Download
D crypto/hkdf_unittest.cc View 1 chunk +0 lines, -92 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
vandebo (ex-Chrome)
7 years, 10 months ago (2013-02-22 19:27:26 UTC) #1
vandebo (ex-Chrome)
Committed patchset #1 manually as r184146.
7 years, 10 months ago (2013-02-22 19:27:48 UTC) #2
ramant (doing other things)
7 years, 10 months ago (2013-02-22 22:16:44 UTC) #3
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698