Make old coverage scripts point to the new one |
||||
Issue descriptionWe probably don't need https://cs.chromium.org/chromium/src/ios/tools/coverage/coverage.py and https://cs.chromium.org/chromium/src/testing/libfuzzer/coverage.py anymore, as there is a unified script: https://cs.chromium.org/chromium/src/tools/code_coverage/coverage.py
,
Feb 21 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3cfd62612ebd53b51c46ad3170ca94b01a88414e commit 3cfd62612ebd53b51c46ad3170ca94b01a88414e Author: Jonathan Metzman <metzman@chromium.org> Date: Wed Feb 21 04:29:08 2018 [Coverage] Remove //testing/libfuzzer/coverage.py and update docs. Change-Id: Ic977dd1109087f7e74408b54bdaa45ab3014c32a Bug: 800876 Reviewed-on: https://chromium-review.googlesource.com/927112 Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by: Max Moroz <mmoroz@chromium.org> Reviewed-by: Abhishek Arya <inferno@chromium.org> Cr-Commit-Position: refs/heads/master@{#538021} [delete] https://crrev.com/9cb3bed9a0b960fc777a729a29c3677fa4955682/testing/libfuzzer/coverage.py [modify] https://crrev.com/3cfd62612ebd53b51c46ad3170ca94b01a88414e/testing/libfuzzer/efficient_fuzzer.md
,
Feb 22 2018
liaoyuke@, can you remove the ios scripts/directory and update ios docs to also use our new script. then close this as fixed. thanks!
,
Feb 22 2018
,
Feb 22 2018
Sure, I've filed a blocking bug to support iOS. Thanks!
,
Mar 6 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6616221895764efd35d48c2a508dccbf8629e296 commit 6616221895764efd35d48c2a508dccbf8629e296 Author: Yuke Liao <liaoyuke@chromium.org> Date: Tue Mar 06 18:37:36 2018 Remove ios coverage docs. Now that code coverage script works on iOS platform no different from other platforms, it doesn't make sense to keep the iOS specific code coverage docs anymore. Bug: 800876 Change-Id: If9344e7283dae05255cc5518609dced2f4367593 Reviewed-on: https://chromium-review.googlesource.com/950469 Reviewed-by: Eugene But <eugenebut@chromium.org> Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#541158} [delete] https://crrev.com/36518401def37caf59a85686b1c06110646befc0/docs/ios/coverage.md [delete] https://crrev.com/36518401def37caf59a85686b1c06110646befc0/docs/ios/images/coverage_xcode.png [delete] https://crrev.com/36518401def37caf59a85686b1c06110646befc0/docs/ios/images/llvm-cov_report.png [delete] https://crrev.com/36518401def37caf59a85686b1c06110646befc0/docs/ios/images/llvm-cov_report_folder.png [delete] https://crrev.com/36518401def37caf59a85686b1c06110646befc0/docs/ios/images/llvm-cov_show.png [delete] https://crrev.com/36518401def37caf59a85686b1c06110646befc0/docs/ios/images/llvm-cov_show_file.png
,
Mar 6 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by mmoroz@chromium.org
, Jan 11 2018Components: Tools>CodeCoverage