Fix #260: break datastore cycles by moving 'from_protobuf' classmethods

Third time's the charm for #260 in datastore:

'gcloud.datastore._helpers' -> 'gcloud.datastore.helpers' (now API).

'gcloud.datastore.entity.Entity.from_protobuf' classmethod -> 'gcloud.datastore.helpers.entity_from_protobuf'.

'gcloud.datastore.key.Key.from_protobuf' classmethod -> 'gcloud.datastore.helpers.key_from_protobuf'.

Hmm, pylint doesn't whine about too-short names for list-comprehension placeholders.  I can make the change, but I don't see that it adds clarity (the name isn't used, or supposed to be, outside that expression).