New issue
Advanced search Search tips

Issue 879209 link

Starred by 1 user
Status: Fixed
Owner:
pauljensen@chromium.org
Closed: Oct 5
Cc:
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , iOS , Chrome , Mac , Fuchsia
Pri: 3
Type: Bug

Blocking:
issue 786559



Sign in to add a comment

Cronet Native API URLRequest GetStatus is not implemented

Project Member Reported by mef@chromium.org, Aug 30 
Cronet Native API URLRequest GetStatus is not implemented.

Comment 1 by mef@chromium.org, Aug 30

Blocking: 786559

Comment 2 by olivierrobin@chromium.org, Aug 31

Owner: mef@chromium.org
Status: Assigned (was: Untriaged)

Comment 3 by pauljensen@chromium.org, Sep 18

Owner: pauljensen@chromium.org
I'll take a crack at this.
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 5 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3fa635c9c7031e1f9a18ea1b27805008627e3cbb

commit 3fa635c9c7031e1f9a18ea1b27805008627e3cbb
Author: Paul Jensen <pauljensen@chromium.org>
Date: Fri Oct 05 14:42:41 2018

[Cronet] Add UrlRequest::GetStatus() support to native API

Bug:  879209 
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;master.tryserver.chromium.android:android_cronet_tester
Change-Id: Idf94d64fc56addd13b7fafbb3e49d477a4e8c4bd
Reviewed-on: https://chromium-review.googlesource.com/c/1231735
Commit-Queue: Paul Jensen <pauljensen@chromium.org>
Reviewed-by: Misha Efimov <mef@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597110}
[modify] https://crrev.com/3fa635c9c7031e1f9a18ea1b27805008627e3cbb/components/cronet/native/test/url_request_test.cc
[modify] https://crrev.com/3fa635c9c7031e1f9a18ea1b27805008627e3cbb/components/cronet/native/url_request.cc
[modify] https://crrev.com/3fa635c9c7031e1f9a18ea1b27805008627e3cbb/components/cronet/native/url_request.h

Comment 5 by pauljensen@chromium.org, Oct 5

Status: Fixed (was: Assigned)
►

Sign in to add a comment