quipper: Serialize/deserialize read info |
|||||||
Issue descriptionRead info fields in sample info is being read by PerfReader but not serialized by PerfSerializer.
,
Dec 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform2/+/21f681f2e5bf3270f086b0b3dc2d746b6283dfde commit 21f681f2e5bf3270f086b0b3dc2d746b6283dfde Author: Simon Que <sque@google.com> Date: Tue Dec 13 22:34:17 2016 quipper: Add PerfReaderTest for sample info fields There is currently no comprehensive unit test for handling all the sample info fields in a sample event. By adding this test, we provide a place for additional fields to be tested in the future when quipper needs to support them. BUG= chromium:672918 TEST=unit tests pass Change-Id: I8a8f4320dfc11f8b140b6078dc6f0444e69ee09a Reviewed-on: https://chromium-review.googlesource.com/419699 Commit-Ready: Simon Que <sque@chromium.org> Tested-by: Simon Que <sque@chromium.org> Reviewed-by: Chong Jiang <chongjiang@chromium.org> [modify] https://crrev.com/21f681f2e5bf3270f086b0b3dc2d746b6283dfde/chromiumos-wide-profiling/perf_reader_test.cc
,
Dec 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform2/+/79f1ed5562db61c776ff13dcbb08106cb9922b7e commit 79f1ed5562db61c776ff13dcbb08106cb9922b7e Author: Simon Que <sque@google.com> Date: Tue Dec 13 19:26:53 2016 quipper: Serialize/deserialize read info BUG= chromium:672918 TEST=unit tests pass Change-Id: Ib276590e2ec1eebf6620087efd0db9cb13559823 Reviewed-on: https://chromium-review.googlesource.com/419799 Commit-Ready: Simon Que <sque@chromium.org> Tested-by: Simon Que <sque@chromium.org> Reviewed-by: Chong Jiang <chongjiang@chromium.org> [modify] https://crrev.com/79f1ed5562db61c776ff13dcbb08106cb9922b7e/chromiumos-wide-profiling/test_perf_data.cc [modify] https://crrev.com/79f1ed5562db61c776ff13dcbb08106cb9922b7e/chromiumos-wide-profiling/test_perf_data.h [modify] https://crrev.com/79f1ed5562db61c776ff13dcbb08106cb9922b7e/chromiumos-wide-profiling/perf_serializer.cc [modify] https://crrev.com/79f1ed5562db61c776ff13dcbb08106cb9922b7e/chromiumos-wide-profiling/perf_reader_test.cc
,
Dec 20 2016
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by bugdroid1@chromium.org
, Dec 13 2016