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

Issue 1675243002: Fix usage of deprecated method sendSynchronousRequest:returningResponse:error:. (Closed)

Created:
4 years, 10 months ago by sdefresne
Modified:
4 years, 10 months ago
Reviewers:
Mark Mentovai
CC:
google-breakpad-dev_googlegroups.com
Base URL:
https://chromium.googlesource.com/breakpad/breakpad.git@fix-string-by-adding-percent-escapes-using-encoding
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix usage of deprecated method sendSynchronousRequest:returningResponse:error:. The method -[NSURLConnection sendSynchronousRequest:returningResponse:error:] has been deprecated in 10.11 OS X SDK and 9.0 iOS SDK without replacement. So emulate a synchronous request by using an asynchronous request and waiting on a semaphore for the request completion. BUG=https://bugs.chromium.org/p/google-breakpad/issues/detail?id=675 BUG=569158 R=mark@chromium.org Committed: https://chromium.googlesource.com/breakpad/breakpad/+/4becb1170940e78bfbc255aa164aca5c9523671c

Patch Set 1 #

Patch Set 2 : Fix compilation #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -3 lines) Patch
M src/common/mac/HTTPMultipartUpload.m View 1 2 chunks +43 lines, -3 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 5 (2 generated)
sdefresne
This is a partial reland of https://codereview.chromium.org/1563223004/. The original CL did not properly manage the ...
4 years, 10 months ago (2016-02-08 13:03:31 UTC) #2
Mark Mentovai
LGTM
4 years, 10 months ago (2016-02-08 15:12:56 UTC) #3
sdefresne
4 years, 10 months ago (2016-02-08 16:39:47 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
4becb1170940e78bfbc255aa164aca5c9523671c (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698