[Code Coverage] Refactor the script logic into Py2/Py3 compatible module.

- Move all report post processing logic to a separate coverage_utils file. - Add switch for disabling component view. - Fix shared libraries regexp to match versioned libraries. - Move HTML post processing methods into a class (to reduce passing a bunch of parameters back and forth). - Add _Main() to coverage_utils.py that can be used outside of Chromium. For example, to use that script on OSS-Fuzz we would need to do the following: 1) git clone https://chromium.googlesource.com/chromium/src/tools/code_coverage somewhere 2) echo -en '#!/bin/bash\npython3 somewhere/coverage_utils.py $@\n' \ > /usr/local/bin/coverage_helper Bug: 817798 Change-Id: I8de8cec2d118f022eb43634dd85b96e4126f5e63 Reviewed-on: https://chromium-review.googlesource.com/1180364 Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by: Oliver Chang <ochang@chromium.org> Cr-Commit-Position: refs/heads/master@{#584726}

[Code Coverage] Refactor the script logic into Py2/Py3 compatible module.
- Move all report post processing logic to a separate coverage_utils file. - Add switch for disabling component view. - Fix shared libraries regexp to match versioned libraries. - Move HTML post processing methods into a class (to reduce passing a bunch of parameters back and forth). - Add _Main() to coverage_utils.py that can be used outside of Chromium. For example, to use that script on OSS-Fuzz we would need to do the following: 1) git clone https://chromium.googlesource.com/chromium/src/tools/code_coverage somewhere 2) echo -en '#!/bin/bash\npython3 somewhere/coverage_utils.py $@\n' \ > /usr/local/bin/coverage_helper Bug: 817798 Change-Id: I8de8cec2d118f022eb43634dd85b96e4126f5e63 Reviewed-on: https://chromium-review.googlesource.com/1180364 Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by: Oliver Chang <ochang@chromium.org> Cr-Commit-Position: refs/heads/master@{#584726}
1de68d73 · Max Moroz · Commit Bot · 921cd240 · 1de68d73 · 1de68d73
Commit 1de68d73 authored Aug 21, 2018 by Max Moroz Committed by Commit Bot Aug 21, 2018
3 changed files
--- a/tools/code_coverage/coverage.py
+++ b/tools/code_coverage/coverage.py
--- a/tools/code_coverage/coverage_utils.py
+++ b/tools/code_coverage/coverage_utils.py
--- a/tools/code_coverage/html_templates/header.html
+++ b/tools/code_coverage/html_templates/header.html
@@ -12,7 +12,11 @@
  <body>
    <h2>Coverage Report</h2>
    <p>
-      View results by: <a href='{{ component_view_href }}'>Components</a> | <a href='{{ directory_view_href }}'>Directories</a>
+      View results by:
+      {% if component_view_href %}
+        <a href='{{ component_view_href }}'>Components</a> |
+      {% endif %}
+      <a href='{{ directory_view_href }}'>Directories</a>
      {% if file_view_href %}
        | <a href='{{ file_view_href }}'>Files</a>
      {% endif %}