Rename Views method add_signature() to addSignature()

Extra space in function get_aggregation_info() comment, otherwise looks fine

I removed the extra spaces in-front of the comment.

Looks fine for me, no more occurrences of add_signature() in core folder

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/query/QueryPluginBase.phpundefined
@@ -79,7 +79,7 @@ function execute(ViewExecutable $view) {  }
+  function addSignature(ViewExecutable $view) { }

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/query/Sql.phpundefined
@@ -1643,7 +1643,7 @@ function load_entities(&$results) {
+  function addSignature(ViewExecutable $view) {

Add public access modifier in front of the function to adher to the new OOP standards.

@oenie, what was the reason to change issue status from RTBC?

@valtthebald: as mentioned in my small review, we have to make sure that all methods are supplied with an access modifier.
See the META issue, comment #28.
Since we're doing these patches, we might as well make sure they get done as good as we can.

I added access modifier.

#9: 2002280-9-rename-add_signature.patch queued for re-testing.

#9: 2002280-9-rename-add_signature.patch queued for re-testing.

Looks good to me now !

Committed b880d8f and pushed to 8.x. Thanks!