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 2330863002: Use modifed URLRequest API in net/url_request context builder, data job fuzzer and file dir test (Closed)

Created:
4 years, 3 months ago by maksims (do not use this acc)
Modified:
4 years, 3 months ago
Reviewers:
mmenke
CC:
chromium-reviews, cbentzel+watch_chromium.org, fuzzing_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Use modifed URLRequest API in net/url_request context builder, data job fuzzer and file dir test This cl splits the big CL with changes to net/ folder and has changes only to net/cert_net according to the changes to URLRequest API. What the big cl does: It modifies net/ clients that use URLRequest API as long as URLRequest::Read returns int net errors and URLRequest::Delegate and NetworkDelegate methods (for example, OnResponseStarted or OnCompleted) have int net_error in the arguments now. The reason behind splitting the CL into small one is that an android bot started to be unstable and unittests became flaky. It was not possible to locate the problem and the decision was to split the CL and upload small parts with a 6+ hours interval in order to make it possible to locate the problematic code. The big CL is located here - https://codereview.chromium.org/2265873002/ BUG=423484 Committed: https://crrev.com/a74d3cbc572fa1c85cc2c0434a9f8e2f9cba0cd9 Cr-Commit-Position: refs/heads/master@{#418014}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -19 lines) Patch
M net/url_request/url_request_context_builder.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M net/url_request/url_request_data_job_fuzzer.cc View 3 chunks +11 lines, -10 lines 0 comments Download
M net/url_request/url_request_file_dir_job_unittest.cc View 2 chunks +3 lines, -7 lines 0 comments Download

Messages

Total messages: 19 (14 generated)
maksims (do not use this acc)
please check
4 years, 3 months ago (2016-09-14 05:50:43 UTC) #8
mmenke
On 2016/09/14 05:50:43, maksims wrote: > please check LGTM
4 years, 3 months ago (2016-09-15 16:32:01 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2330863002/1
4 years, 3 months ago (2016-09-18 06:04:25 UTC) #15
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/d3e5dafc0d304f73ef5c53fcda33d5e95ea100c2 Cr-Commit-Position: refs/heads/master@{#419394}
4 years, 3 months ago (2016-09-18 07:03:55 UTC) #17
commit-bot: I haz the power
4 years, 3 months ago (2016-09-18 07:04:18 UTC) #19
Message was sent while issue was closed.
Committed patchset #1 (id:1)

Powered by Google App Engine
This is Rietveld 408576698