New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 685897 link

Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Update component updater URLs to update.googleapis.com.

Project Member Reported by waff...@chromium.org, Jan 27 2017

Issue description

The component updater should move off clients2 and to newer hostnames that are more globally compatible.
 
Summary: Update component updater URLs to update.googleapis.com. (was: Update component updater URLs to update.googleapi.com.)
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 3 2017

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

commit af271de2bcbaaec00b9b2d68f41fd0c0903f5e05
Author: dtu <dtu@chromium.org>
Date: Fri Feb 03 04:48:49 2017

Revert of Update component updater URLs. (patchset #2 id:20001 of https://codereview.chromium.org/2654033009/ )

Reason for revert:
Lots of perf benchmarks started failing on Android. Bisect pointed to this change.

https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus5%20Perf%20%281%29/builds/5173
https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus5X%20Perf%20%281%29/builds/4443
https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus6%20Perf%20%281%29/builds/4826
https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus7v2%20Perf%20%281%29/builds/4667
https://build.chromium.org/p/chromium.perf/builders/Android%20Nexus9%20Perf%20%281%29/builds/4012

Original issue's description:
> Update component updater URLs.
>
> BUG= 685897 
>
> Review-Url: https://codereview.chromium.org/2654033009
> Cr-Commit-Position: refs/heads/master@{#447564}
> Committed: https://chromium.googlesource.com/chromium/src/+/9fefd20253ceb6af4fc3604ca29fcafc4c8454cd

TBR=sorin@chromium.org,rkaplow@chromium.org,pastarmovj@chromium.org,waffles@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 685897 

Review-Url: https://codereview.chromium.org/2675903002
Cr-Commit-Position: refs/heads/master@{#447940}

[modify] https://crrev.com/af271de2bcbaaec00b9b2d68f41fd0c0903f5e05/chrome/browser/policy/policy_browsertest.cc
[modify] https://crrev.com/af271de2bcbaaec00b9b2d68f41fd0c0903f5e05/components/component_updater/component_updater_url_constants.cc
[modify] https://crrev.com/af271de2bcbaaec00b9b2d68f41fd0c0903f5e05/components/component_updater/component_updater_url_constants.h
[modify] https://crrev.com/af271de2bcbaaec00b9b2d68f41fd0c0903f5e05/components/component_updater/configurator_impl.cc
[modify] https://crrev.com/af271de2bcbaaec00b9b2d68f41fd0c0903f5e05/testing/variations/fieldtrial_testing_config.json

Cc: jasontiller@chromium.org
 Issue 688168  has been merged into this issue.
Cc: waff...@chromium.org rbasuvula@chromium.org
 Issue 657304  has been merged into this issue.
Project Member

Comment 7 by bugdroid1@chromium.org, Jul 19 2017

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

commit c0fffee8cff1abcce506f86bb192349c62e6e3a7
Author: Joshua Pawlicki <waffles@google.com>
Date: Wed Jul 19 18:08:58 2017

Switch component updater to use new URL by default. (Launching.)

Bug:  685897 
Change-Id: Ib08627d9b1f99baa7f50d97e30b7f6ba271d3495
Reviewed-on: https://chromium-review.googlesource.com/576707
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487915}
[modify] https://crrev.com/c0fffee8cff1abcce506f86bb192349c62e6e3a7/chrome/browser/component_updater/chrome_component_updater_configurator_unittest.cc
[modify] https://crrev.com/c0fffee8cff1abcce506f86bb192349c62e6e3a7/components/component_updater/configurator_impl.cc

Status: Fixed (was: Started)
Project Member

Comment 9 by bugdroid1@chromium.org, May 15 2018

Sign in to add a comment