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

Issue 385113003: Revert 282546 "Enforce SafetyMode for YouTube if prefs::kForceSa..." (Closed)

Created:
6 years, 5 months ago by kustermann
Modified:
6 years, 5 months ago
Reviewers:
Marc Treib
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 282546 "Enforce SafetyMode for YouTube if prefs::kForceSa..." Reason for revert: [0711/004616:ERROR:unit_test_launcher.cc(360)] no test result for SafeSearchUtilTest.ModifyYoutubePrefCookie [ RUN ] SafeSearchUtilTest.ModifyYoutubePrefCookie c:\b\depot_tools\win_toolchain\vs2013_files\vc\include\xstring(79) : Assertion failed: string iterator not dereferencable c:\b\depot_tools\win_toolchain\vs2013_files\vc\include\xstring(80) : Assertion failed: "Standard C++ Libraries Out of Range" && 0 [7000/7003] SafeSearchUtilTest.ModifyYoutubePrefCookie (UNKNOWN) > Enforce SafetyMode for YouTube if prefs::kForceSafeSearch is on, by modifying (or adding) YouTube's PrefCookie in the HTTP header. > > This is essentially a followup to https://codereview.chromium.org/11186002/ > > BUG=344815 > > Review URL: https://codereview.chromium.org/354183002 TBR=treib@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282601

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+127 lines, -490 lines) Patch
M trunk/src/chrome/browser/extensions/api/web_request/web_request_api_helpers.cc View 4 chunks +66 lines, -5 lines 0 comments Download
M trunk/src/chrome/browser/net/chrome_network_delegate.cc View 6 chunks +60 lines, -8 lines 0 comments Download
D trunk/src/chrome/browser/net/safe_search_util.h View 1 chunk +0 lines, -29 lines 0 comments Download
D trunk/src/chrome/browser/net/safe_search_util.cc View 1 chunk +0 lines, -195 lines 0 comments Download
D trunk/src/chrome/browser/net/safe_search_util_unittest.cc View 1 chunk +0 lines, -90 lines 0 comments Download
M trunk/src/chrome/chrome_browser.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M trunk/src/chrome/chrome_tests_unit.gypi View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/net/cookies/cookie_util.h View 2 chunks +1 line, -22 lines 0 comments Download
M trunk/src/net/cookies/cookie_util.cc View 1 chunk +0 lines, -53 lines 0 comments Download
M trunk/src/net/cookies/cookie_util_unittest.cc View 2 chunks +0 lines, -85 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
kustermann
6 years, 5 months ago (2014-07-11 09:55:56 UTC) #1
kustermann
6 years, 5 months ago (2014-07-11 09:56:16 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r282601.

Powered by Google App Engine
This is Rietveld 408576698