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

Issue 11343041: Revert 164733 - Merge 164709 - Don't use CanonicalCookie::Create method for creating canonical cook… (Closed)

Created:
8 years, 1 month ago by karen
Modified:
8 years, 1 month ago
Reviewers:
markusheintz_
CC:
chromium-reviews, markusheintz_
Visibility:
Public.

Description

Revert 164733 - Merge 164709 - Don't use CanonicalCookie::Create method for creating canonical cookies in the BrowsingDataCookieHelper. This results in wrong default values if no cookie-domain attribute is given in the cookie-string. BUG=158353 Review URL: https://codereview.chromium.org/11341011 TBR=markusheintz@chromium.org Review URL: https://codereview.chromium.org/11273090 TBR=markusheintz@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=164907

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -90 lines) Patch
M chrome/browser/browsing_data/browsing_data_cookie_helper.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/browsing_data/browsing_data_cookie_helper_unittest.cc View 5 chunks +0 lines, -89 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
karen
8 years, 1 month ago (2012-10-30 13:51:05 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698