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

Issue 120183005: Update perf expectations to match improvement caused by CF code removal in r242400. (Closed)

Created:
6 years, 12 months ago by gab
Modified:
6 years, 12 months ago
Reviewers:
Nikita (slow)
CC:
chromium-reviews, chrome-speed-team+watch_google.com, grt (UTC plus 2)
Visibility:
Public.

Description

Update perf expectations to match improvement caused by CF code removal in r242400. Update expectations for setup.exe/chrome.exe. Remove expectations for npchrome_frame.dll as per https://codereview.chromium.org/117413007/ NOTRY=True Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=242443

Patch Set 1 #

Patch Set 2 : remove expctations for npchrome_frame.dll #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -3 lines) Patch
M tools/perf_expectations/perf_expectations.json View 1 1 chunk +2 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
gab
Nikita PTAL. Cheers, Gab
6 years, 12 months ago (2013-12-24 15:40:14 UTC) #1
Nikita (slow)
lgtm, thanks!
6 years, 12 months ago (2013-12-24 15:42:32 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gab@chromium.org/120183005/30001
6 years, 12 months ago (2013-12-24 15:44:01 UTC) #3
commit-bot: I haz the power
Change committed as 242443
6 years, 12 months ago (2013-12-24 15:44:11 UTC) #4
gab
6 years, 12 months ago (2013-12-24 15:44:51 UTC) #5
Message was sent while issue was closed.
CC grt, FYI.

Cheers,
Gab

Powered by Google App Engine
This is Rietveld 408576698