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

Issue 118293004: Revert 241883 "Revert 241876 "Add a check that we don't send a s..." (Closed)

Created:
7 years ago by jochen (gone - plz use gerrit)
Modified:
7 years ago
Reviewers:
Michael Courage
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 241883 "Revert 241876 "Add a check that we don't send a s..." Fixed the incorrect referrer handling in r242082 > Revert 241876 "Add a check that we don't send a secure referrer ..." > > This change looks like it broke ReferrerPolicyTest.History > > > Add a check that we don't send a secure referrer to an insecure URL > > > > Unless one of the non-default referrer policies is set, this should > > never happen. > > > > BUG=none > > R=mef@chromium.org > > > > Review URL: https://codereview.chromium.org/115803003 > > TBR=jochen@chromium.org > > Review URL: https://codereview.chromium.org/118473005 TBR=courage@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=242084

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -0 lines) Patch
M trunk/src/net/url_request/url_request.cc View 2 chunks +12 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jochen (gone - plz use gerrit)
7 years ago (2013-12-20 12:57:03 UTC) #1
jochen (gone - plz use gerrit)
7 years ago (2013-12-20 12:57:12 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r242084.

Powered by Google App Engine
This is Rietveld 408576698