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

Issue 2410223002: Rename RequestFinishedInfo.Metrics.getResponseEnd() to getRequestEnd() (Closed)

Created:
4 years, 2 months ago by xunjieli
Modified:
4 years, 2 months ago
Reviewers:
mgersh, mef
CC:
chromium-reviews, cbentzel+watch_chromium.org, agrieve+watch_chromium.org
Target Ref:
refs/pending/branch-heads/2883
Project:
chromium
Visibility:
Public.

Description

Rename RequestFinishedInfo.Metrics.getResponseEnd() to getRequestEnd() The timestamp is available even when there is no response, so this name makes more sense. BUG=629194, 653661 TBR=mef@chromium.org NOPRESUBMIT=true NOTRY=true Review-Url: https://codereview.chromium.org/2401933002 Cr-Commit-Position: refs/heads/master@{#423872} (cherry picked from commit 1ad34d61a5dc3b4415c9448885a36e8f442b2c0b)

Patch Set 1 #

Messages

Total messages: 8 (4 generated)
xunjieli
Misha, this is to help Miriam merge her CL into M55. I will CQ it ...
4 years, 2 months ago (2016-10-11 15:41:12 UTC) #2
xunjieli
On 2016/10/11 15:41:12, xunjieli wrote: > Misha, this is to help Miriam merge her CL ...
4 years, 2 months ago (2016-10-11 17:51:12 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2410223002/1
4 years, 2 months ago (2016-10-11 17:51:31 UTC) #6
commit-bot: I haz the power
4 years, 2 months ago (2016-10-11 17:52:32 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1)

Powered by Google App Engine
This is Rietveld 408576698