Rename cc_common_internal_do_not_use back to cc_common

[fmeum]

The name of CcModule is shown in Bazel docs and should match the name of the top-level cc_common built-in symbol.

[fmeum]

@buildbreaker2021 The original report was https://bazelbuild.slack.com/archives/CGA9QFQ8H/p1688479868511049. I am not sure I fully understand how the built-in private API restrictions work, but this does seem to pass all tests while not exposing private API.

[buildbreaker2021]

I am trying to remember why I named this cc_common_internal_do_not_use - maybe because of naming conflicts. But this could have just been my assumption that there would be a conflict. I will check the history and will provide the review.

[fmeum]

@buildbreaker2021 I looked into how java_common is doing this and found that it is not using a different name. My thinking was that every context (builtins vs non-builtins) would only ever see a single instance of the cc_common top-level symbol.

[buildbreaker2021]

Yeah I also did not find a compelling reason for why I've named it that way.