{"push_id":1145615673,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"b20272f4bed6bf1775113960a2f2fbe70f74562d","before":"242e247a554070bdf93b2de7c73fdfd4ea07342b","commits":[{"sha":"b20272f4bed6bf1775113960a2f2fbe70f74562d","author":{"name":"gabadie","email":"eb6b182d7c05a80c07fb472c0ca4f217014b7aec@chromium.org"},"message":"sandwich: Merge cache-validation.json and urls-for-resources.json tasks\n\nBefore in the NoState-Prefetch benchmark tasks graph, Sandwich had\nthe common/cache-validation.json tasks as a leaf. But it could\nlead to issue when selecting final tasks with -e REGEX, and\nbut forgetting to have the regex matching this important task as\nwell as the subset of task the user initially wanted to run.\n\nThis CL addresses this issue by merging the previous tasks\ncommon/cache-validation.json and common/urls-for-resources.json\ninto a single common/patched-cache-validation.json tasks, and\nhave all the benchmark subgraphs depending on them so we are\nsure to always run the cache validation at least once before the\nbenchmark runs.\n\nThis CL take this oportunity to output further statistics in the\nCSV related to the cache genration such as the number of\nsub-resources of an URL, the number of expected cached resources\nand the actual number of successfully cached resources, to\nidentify at scale on many URLs from the CSVs potential issues\nthe WPR patcher could still have on some webpages.\n\nMoreover, this CL take this change in the extract metrics task\nchange as an opportunity to parse the fat loading trace JSON\nfile only once per URL repeat row, in order to make theses tasks\nfaster.\n\nBUG=582080\n\nReview-Url: https://codereview.chromium.org/2033093002\nCr-Commit-Position: refs/heads/master@{#398021}","distinct":true,"url":"https://api.github.com/repos/dstockwell/chromium/commits/b20272f4bed6bf1775113960a2f2fbe70f74562d"}]}
{"actor":{"display_login":"dstockwell"}}