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

Issue 2802893002: Add gn args to metadata and metadata diffing (Closed)

Created:
3 years, 8 months ago by agrieve
Modified:
3 years, 8 months ago
Reviewers:
estevenson
CC:
chromium-reviews, wnwen+watch_chromium.org, estevenson+watch_chromium.org, agrieve+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Add gn args to metadata and metadata diffing When diffing, should help confirm we're comparing apples to apples. When just analyzing. Validating that the correct gn args were used as actually quite important, so they are worth recording. NOTRY=true BUG=681694 Review-Url: https://codereview.chromium.org/2802893002 Cr-Commit-Position: refs/heads/master@{#462559} Committed: https://chromium.googlesource.com/chromium/src/+/ca8dfa51db7ed68b7c8fca1ebc168539349a9675

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+145 lines, -70 lines) Patch
M tools/binary_size/describe.py View 4 chunks +39 lines, -14 lines 0 comments Download
M tools/binary_size/integration_test.py View 1 chunk +2 lines, -0 lines 0 comments Download
M tools/binary_size/map2size.py View 4 chunks +17 lines, -2 lines 0 comments Download
M tools/binary_size/models.py View 5 chunks +30 lines, -7 lines 0 comments Download
M tools/binary_size/testdata/ActualDiff.golden View 1 chunk +47 lines, -40 lines 0 comments Download
M tools/binary_size/testdata/ConsoleNullDiff.golden View 1 chunk +10 lines, -7 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 10 (6 generated)
agrieve
On 2017/04/06 01:31:14, agrieve wrote: > mailto:agrieve@chromium.org changed reviewers: > + mailto:estevenson@chromium.org 🚙
3 years, 8 months ago (2017-04-06 01:31:31 UTC) #3
estevenson
lgtm, forgot to send this one. Good call adding gn args to metadata!
3 years, 8 months ago (2017-04-06 17:02:15 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/2802893002/1
3 years, 8 months ago (2017-04-06 18:22:21 UTC) #7
commit-bot: I haz the power
3 years, 8 months ago (2017-04-06 18:45:45 UTC) #10
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/ca8dfa51db7ed68b7c8fca1ebc16...

Powered by Google App Engine
This is Rietveld 408576698