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

Issue 376403002: Update size expectations and shared lib expectations after clang switch (Closed)

Created:
6 years, 5 months ago by Nico
Modified:
6 years, 5 months ago
Reviewers:
Lei Zhang
CC:
chromium-reviews, grt+watch_chromium.org
Project:
chromium
Visibility:
Public.

Description

Update size expectations and shared lib expectations after clang switch BUG=360311 TBR=thestig@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282261

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -18 lines) Patch
M chrome/installer/linux/rpm/expected_deps_i386 View 3 chunks +1 line, -4 lines 0 comments Download
M tools/perf_expectations/perf_expectations.json View 2 chunks +14 lines, -14 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Nico
I never changed so expectations, please check that part carefully. (The 64bit build was happy ...
6 years, 5 months ago (2014-07-10 05:04:20 UTC) #1
Nico
Committed patchset #1 manually as r282261 (tree was closed).
6 years, 5 months ago (2014-07-10 05:04:48 UTC) #2
Lei Zhang
I'm not qualified to review perf expectations changes. expected_deps lgtm if the bots go green.
6 years, 5 months ago (2014-07-10 18:20:17 UTC) #3
Nico
6 years, 5 months ago (2014-07-10 18:24:54 UTC) #4
Message was sent while issue was closed.
On 2014/07/10 18:20:17, Lei Zhang wrote:
> I'm not qualified to review perf expectations changes.

Pretty sure those are right :-)

> expected_deps lgtm if the bots go green.

The 32bit bot is green at least. Still fighting 64…

Powered by Google App Engine
This is Rietveld 408576698