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

Issue 9700056: Remove experimental support for MAC cookies (Closed)

Created:
8 years, 9 months ago by Ryan Sleevi
Modified:
8 years, 9 months ago
CC:
chromium-reviews, cbentzel+watch_chromium.org, darin-cc_chromium.org
Visibility:
Public.

Description

Remove experimental support for MAC cookies Remove the experimental support for MAC cookies, as defined in http://tools.ietf.org/html/draft-hammer-oauth-v2-mac-token . Support for strongly-bound cookies will be provided by support for TLS channel-bound cookies, as defined in http://tools.ietf.org/html/draft-balfanz-tls-obc BUG=none TEST=none R=abarth@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=126928

Patch Set 1 #

Patch Set 2 : Remove unused global #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -429 lines) Patch
M chrome/browser/chrome_browser_main.cc View 1 chunk +0 lines, -3 lines 0 comments Download
M chrome/common/chrome_switches.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/chrome_switches.cc View 1 chunk +0 lines, -5 lines 0 comments Download
D net/http/http_mac_signature.h View 1 chunk +0 lines, -77 lines 0 comments Download
D net/http/http_mac_signature.cc View 1 chunk +0 lines, -181 lines 0 comments Download
D net/http/http_mac_signature_unittest.cc View 1 chunk +0 lines, -109 lines 0 comments Download
M net/net.gyp View 2 chunks +0 lines, -3 lines 0 comments Download
M net/url_request/url_request.h View 1 chunk +0 lines, -4 lines 0 comments Download
M net/url_request/url_request.cc View 1 2 chunks +0 lines, -12 lines 0 comments Download
M net/url_request/url_request_http_job.cc View 3 chunks +0 lines, -34 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Ryan Sleevi
willchan: For stamps! abarth: As discussed on e-mail, removing the dead code. This reverts http://crrev.com/83600 ...
8 years, 9 months ago (2012-03-15 05:40:30 UTC) #1
willchan no longer on Chromium
lgtm
8 years, 9 months ago (2012-03-15 05:57:31 UTC) #2
abarth-chromium
Thanks.
8 years, 9 months ago (2012-03-15 08:15:01 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rsleevi@chromium.org/9700056/2003
8 years, 9 months ago (2012-03-15 14:22:01 UTC) #4
commit-bot: I haz the power
8 years, 9 months ago (2012-03-15 17:05:47 UTC) #5
Change committed as 126928

Powered by Google App Engine
This is Rietveld 408576698