New issue
Advanced search Search tips

Issue 872876 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 10
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: ----
Type: ----



Sign in to add a comment

Migrate chrome/browser/android/digital_asset_links/digital_asset_links_handler.cc using SimpleURLLoader

Project Member Reported by dxie@google.com, Aug 9

Issue description


 
Labels: Proj-Servicification Proj-Servicification-VendorBug Hotlist-KnownIssue
Labels: OS-Android
Labels: Proj-Servicification-Network-Url
Owner: svil...@igalia.com
Status: Assigned (was: Available)
Taking this...
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 10

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

commit edefa9e114b1459695990145343d605e7dcbf9fd
Author: Sergio Villar Senin <svillar@igalia.com>
Date: Mon Sep 10 08:44:06 2018

[android] Migrate digital_asset_links_handler.cc using SimpleURLLoader

It's currently using URLFetcher. It should use SimpleURLLoader
instead to make it eventually work with the network service.

Bug:  872876 
Change-Id: I1b0a0b3fcab7f96ca576d8dae47c62f7218a31e2
Reviewed-on: https://chromium-review.googlesource.com/1203955
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Commit-Queue: Sergio Villar <svillar@igalia.com>
Cr-Commit-Position: refs/heads/master@{#589859}
[modify] https://crrev.com/edefa9e114b1459695990145343d605e7dcbf9fd/chrome/browser/android/customtabs/origin_verifier.cc
[modify] https://crrev.com/edefa9e114b1459695990145343d605e7dcbf9fd/chrome/browser/android/digital_asset_links/digital_asset_links_handler.cc
[modify] https://crrev.com/edefa9e114b1459695990145343d605e7dcbf9fd/chrome/browser/android/digital_asset_links/digital_asset_links_handler.h
[modify] https://crrev.com/edefa9e114b1459695990145343d605e7dcbf9fd/chrome/browser/android/digital_asset_links/digital_asset_links_handler_unittest.cc

Status: Fixed (was: Assigned)
closing

Sign in to add a comment