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

Issue 835439 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocking:
issue 820783



Sign in to add a comment

Change minimum RLZ ping delay on chromeos to 60sec

Project Member Reported by stephenlin@chromium.org, Apr 20 2018

Issue description

Current default of 20secs is sometimes not enough time to setup chrome://net-internals to capture outgoing ping. Increase to 60s.

This has been fixed in ToT: https://chromium-review.googlesource.com/c/chromium/src/+/1022513

 
Pasted from crbug.com/820783

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

commit a7523d1f37db7dbd6cb2bcb6dbe0487deb72b462
Author: Roger Tawa <rogerta@chromium.org>
Date: Fri Apr 20 20:40:29 2018

Change minimum RLZ ping delay on chromeos to 60sec.

Bug: 820783
Change-Id: If60b92464261fbf6523153f9cab9bba5bbf61dcd
Reviewed-on: https://chromium-review.googlesource.com/1022513
Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>
Commit-Queue: Roger Tawa <rogerta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552460}
[modify] https://crrev.com/a7523d1f37db7dbd6cb2bcb6dbe0487deb72b462/components/rlz/rlz_tracker.cc

Merge request is to make it easier for the Test team to test and verify the feature in crbug.com/820783.
Project Member

Comment 3 by sheriffbot@chromium.org, Apr 21 2018

Labels: -Merge-Request-67 Merge-Approved-67 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M67. Please go ahead and merge the CL to branch 3396 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 4 by sheriffbot@chromium.org, Apr 25 2018

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by sheriffbot@chromium.org, Apr 30 2018

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: merge-merged-3396
This change was merged on Apr 23rd under a different bug id:
https://bugs.chromium.org/p/chromium/issues/detail?id=820783#c40

Project Member

Comment 7 by sheriffbot@chromium.org, Jun 12 2018

Labels: -Merge-Approved-67
This issue hasn't been updated in the last 6 weeks, so removing its merge approval label. Please re-request a merge if needed.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Assigned)

Sign in to add a comment