Syntax and coverage checking added to Network Traffic Annotations.

Syntax and coverage checking added to Network Traffic Annotations.

Tests added to check if network traffic annotations are syntactically
correct and all functions requiring them are properly annotated.

BUG=656607
Closing this CL as it was decided to proceed with another approach.

[Ramin Halavati, 2017-05-09 11:53:43]

The CQ bit was checked by rhalavati@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-09 11:54:02]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-09 13:43:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-09 13:43:10]

Dry run: This issue passed the CQ dry run.

[Ramin Halavati, 2017-05-10 09:04:42]

rhalavati@chromium.org changed reviewers:
+ battre@chromium.org, msramek@chromium.org

[Ramin Halavati, 2017-05-10 09:04:43]

This the first draft for Tests.

Changes in the following files is because we can no more have constexpr at the
beginning of definition. I am checking if it is required or not, but if you
think it is, do you have a suggestion?

chrome/browser/plugins/plugins_resource_service.cc
chrome/browser/safe_browsing/incident_reporting/incident_report_uploader_impl.cc
chrome/browser/safe_browsing/protocol_manager.cc
chrome/browser/sync_file_system/drive_backend/sync_engine.cc
chrome/browser/ui/passwords/account_avatar_fetcher.cc
components/search_engines/template_url_fetcher.cc
content/browser/loader/resource_dispatcher_host_impl.cc
headless/public/util/http_url_fetcher.cc

[Ramin Halavati, 2017-05-10 11:20:42]

Description was changed from

==========
Syntax and converage added to Network Traffic Annotations.

Tests added to code to check if network traffic annotations are syntactically
correct and all functions requiring them are properly annotated.

BUG=656607
==========

to

==========
Syntax and coverage checking added to Network Traffic Annotations.

Tests added to check if network traffic annotations are syntactically
correct and all functions requiring them are properly annotated.

BUG=656607
==========

[Ramin Halavati, 2017-05-10 13:58:06]

Patchset #2 (id:20001) has been deleted

[Ramin Halavati, 2017-05-16 13:07:51]

Patchset #5 (id:100001) has been deleted

[Ramin Halavati, 2017-05-16 13:09:31]

Patchset #5 (id:120001) has been deleted

[Ramin Halavati, 2017-05-16 13:09:39]

Patchset #4 (id:80001) has been deleted

[Ramin Halavati, 2017-06-20 08:10:04]

Description was changed from

==========
Syntax and coverage checking added to Network Traffic Annotations.

Tests added to check if network traffic annotations are syntactically
correct and all functions requiring them are properly annotated.

BUG=656607
==========

to

==========
Syntax and coverage checking added to Network Traffic Annotations.

Tests added to check if network traffic annotations are syntactically
correct and all functions requiring them are properly annotated.

BUG=656607

Closing this CL as it was decided to proceed with another approach.
==========