GitLab

This CL changes the following:
1- Make ProtocolEvent.ToValue() a member method instead of a static
method.
2- Changes most of the methods in ProtocolEvent to be private since
there aren't used outside the class.
3- Replaces some deprecated methods.
3- Some general cleanup here and there in the touched files.

I suggest starting the review from protocol_event.h and
protocol_event.cc

This is mostly a mechanical change.

Bug: 929100
Change-Id: I32d86265fa1b1ac63af993ff34ef46df18b0c2ac
Reviewed-on: https://chromium-review.googlesource.com/c/1456004
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#629565}