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

Issue 11040061: Change nss_revision from r159459 back to r158129. (Closed)

Created:
8 years, 2 months ago by wtc
Modified:
8 years, 2 months ago
Reviewers:
Nico, Ryan Sleevi
CC:
chromium-reviews
Visibility:
Public.

Description

Change nss_revision from r159459 back to r158129. This reverts r158841: Update nss_revision to 158748 to pick up NSS 3.14 pre-release snapshot 2012-09-25. https://chromiumcodereview.appspot.com/10987048 But it also reverts r159462 by thakis: Merge https://bugzilla.mozilla.org/show_bug.cgi?id=795213 to fix a warning. https://codereview.chromium.org/11019008 R=rsleevi@chromium.org,thakis@chromium.org BUG=153281, 151692 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=160647

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
wtc
rsleevi,thakis: please review. thakis: we need to downgrade nss_revision to see if it fixes a ...
8 years, 2 months ago (2012-10-05 17:07:12 UTC) #1
Ryan Sleevi
lgtm
8 years, 2 months ago (2012-10-05 17:14:57 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wtc@chromium.org/11040061/1
8 years, 2 months ago (2012-10-06 22:50:53 UTC) #3
Nico
lgtm. Let me know if this helps. If it doesn't and fixing nspr will take ...
8 years, 2 months ago (2012-10-07 10:59:14 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wtc@chromium.org/11040061/1
8 years, 2 months ago (2012-10-08 14:01:12 UTC) #5
commit-bot: I haz the power
8 years, 2 months ago (2012-10-08 14:01:30 UTC) #6
Change committed as 160647

Powered by Google App Engine
This is Rietveld 408576698