Issue 9810022: Remove DONTUSEME_GetCookieStore().

Issue 9810022: Remove DONTUSEME_GetCookieStore(). (Closed)

Created:
8 years, 9 months ago by eroman

Modified:
8 years, 9 months ago

Reviewers:
jam

CC:
chromium-reviews, joi+watch-content_chromium.org, jam, cbentzel+watch_chromium.org, darin-cc_chromium.org

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Remove DONTUSEME_GetCookieStore(). BUG=119293 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=128142

Patch Set 1 #

Patch Set 2 : moar #

Patch Set 3 : rebase #

Created: 8 years, 9 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+4 lines, -61 lines)			Patch
M	chrome/browser/net/chrome_url_request_context.h	View	1 2	5 chunks	+2 lines, -11 lines	0 comments	Download
M	chrome/browser/net/chrome_url_request_context.cc	View		3 chunks	+0 lines, -33 lines	0 comments	Download
M	content/shell/shell_url_request_context_getter.h	View	1	1 chunk	+0 lines, -1 line	0 comments	Download
M	content/shell/shell_url_request_context_getter.cc	View	1 2	1 chunk	+0 lines, -7 lines	0 comments	Download
M	net/url_request/url_request_context_getter.h	View	1 2	2 chunks	+1 line, -5 lines	0 comments	Download
M	net/url_request/url_request_context_getter.cc	View	1 2	2 chunks	+1 line, -4 lines	0 comments	Download

Messages

Total messages: 6 (0 generated)

Expand Messages | Collapse Messages

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/eroman@chromium.org/9810022/6001

8 years, 9 months ago (2012-03-22 00:24:48 UTC) #3

commit-bot: I haz the power

Presubmit check for 9810022-6001 failed and returned exit status 1. Running presubmit commit checks ...

8 years, 9 months ago (2012-03-22 00:25:26 UTC) #4

Presubmit check for 9810022-6001 failed and returned exit status 1.

Running presubmit commit checks ...
Traceback (most recent call last):
  File "/mnt/data/b/commit-queue/verification/presubmit_shim.py", line 43, in
<module>
    sys.exit(presubmit_support.Main(argv))
  File "/b/depot_tools/presubmit_support.py", line 1266, in Main
    rietveld_obj)
  File "/b/depot_tools/presubmit_support.py", line 1114, in DoPresubmitChecks
    results += executer.ExecPresubmitScript(presubmit_script, filename)
  File "/b/depot_tools/presubmit_support.py", line 1031, in ExecPresubmitScript
    result = eval(function_name + '(*__args)', context)
  File "<string>", line 1, in <module>
  File "<string>", line 284, in CheckChangeOnCommit
  File "<string>", line 190, in _CommonChecks
  File "/b/depot_tools/presubmit_canned_checks.py", line 912, in
PanProjectChecks
    input_api, output_api, source_file_filter=None))
  File "/b/depot_tools/presubmit_canned_checks.py", line 752, in CheckOwners
    approval_needed=input_api.is_committing)
  File "/b/depot_tools/presubmit_canned_checks.py", line 784, in
_RietveldOwnerAndReviewers
    int(input_api.change.issue), True)
  File "/b/depot_tools/rietveld.py", line 92, in get_issue_properties
    return json.loads(self.get(url))
  File "/b/depot_tools/rietveld.py", line 306, in get
    return self._send(request_path, **kwargs)
  File "/b/depot_tools/rietveld.py", line 331, in _send
    result = self.rpc_server.Send(request_path, **kwargs)
  File "/b/depot_tools/third_party/upload.py", line 409, in Send
    ErrorExit(e.read())
  File "/b/depot_tools/rietveld.py", line 323, in trap_http_500
    raise urllib2.HTTPError(request_path, m.group(1), msg, None, None)
urllib2.HTTPError: HTTP Error 500: 
<html><head>
<meta http-equiv="content-type" content="text/html;charset=utf-8">
<title>500 Server Error</title>
</head>
<body text=#000000 bgcolor=#ffffff>
<h1>Error: Server Error</h1>
<h2>The server encountered an error and could not complete your request.<p>If
the problem persists, please <A
HREF="http://code.google.com/appengine/community.html">report</A> your problem
and mention this error message and the query that caused it.</h2>
<h2></h2>
</body></html>

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/eroman@chromium.org/9810022/6001

8 years, 9 months ago (2012-03-22 00:29:50 UTC) #5

commit-bot: I haz the power

8 years, 9 months ago (2012-03-22 01:46:28 UTC) #6

Failed to request the patch to try. Please note that binary filesare still
unsupported at the moment, this is being worked on.

Thanks for your patience.

HTTP Error 500: 
<html><head>
<meta http-equiv="content-type" content="text/html;charset=utf-8">
<title>500 Server Error</title>
</head>
<body text=#000000 bgcolor=#ffffff>
<h1>Error: Server Error</h1>
<h2>The server encountered an error and could not complete your request.<p>If
the problem persists, please <A
HREF="http://code.google.com/appengine/community.html">report</A> your problem
and mention this error message and the query that caused it.</h2>
<h2></h2>
</body></html>

Expand Messages | Collapse Messages