api_proto_plugin: Remove debugging code

[phlax]

Signed-off-by: Ryan Northey ryan@synca.io

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

[wrowe]

Requesting review from @htuch who introduced the code now gated by cprofile_enabled.

From the slack, further comments from phlax;

To give more context, these are FileDescriptorProtos ie files including source codek, atm they are created as outputs but not used anywhere (afaict)
Also bazel recently added FileDescriptorProtos to ProtoSets so where we do actually need the source, which is going through this code we can grab it.
Re cprofile_enabled - mho is just remove the code guarded by it as it would require use of an action_env var to get the var there and that is not documented anywhere

It sounds to me that these are redundant and this was early diagnostic logic that never got mopped up from #8309?

[htuch]

Yeah, we can drop the cprofile stuff I think. But the this PR is now only about this, so the title no longer makes sense?

[phlax]

Yeah, we can drop the cprofile stuff I think.

so iiuc just remove - ill add patch now...

[phlax]

title updated

[htuch]

LGTM, thanks!