add is_gs_url(), split_gs_url() static methods to gs_utils.py

add is_gs_url(), split_gs_url() static methods to gs_utils.py

This will be helpful for dealing with user input in the gs://path format.

R=rmistry@google.com

Committed: https://skia.googlesource.com/common/+/f63e1cf

[epoger, 2014-07-29 20:14:14]

Ravi, PTAL at patchset 1

[rmistry, 2014-07-29 20:36:20]

LGTM

https://codereview.chromium.org/427893002/diff/1/py/utils/gs_utils_manualtest.py
File py/utils/gs_utils_manualtest.py (right):

https://codereview.chromium.org/427893002/diff/1/py/utils/gs_utils_manualtest...
py/utils/gs_utils_manualtest.py:60: ('gs://bucket/path1',       'bucket',
'path1'),
Could you also please add gs://bucket/path1/ and maybe gs://bucket/path1//

https://codereview.chromium.org/427893002/diff/1/py/utils/gs_utils_manualtest...
py/utils/gs_utils_manualtest.py:61: ('gs://bucket/path1/path2', 'bucket',
'path1/path2'),
Optional Nit: Instead of aligning some of these lets not align any of them. I
always prefer not aligning because if we need to add any new test cases that are
longer than the longest one then we will have to change all of them instead of
it being a one-line change.

[epoger, 2014-07-30 01:10:49]

Thanks, fixes at patchset 2.  Committing now.

https://codereview.chromium.org/427893002/diff/1/py/utils/gs_utils_manualtest.py
File py/utils/gs_utils_manualtest.py (right):

https://codereview.chromium.org/427893002/diff/1/py/utils/gs_utils_manualtest...
py/utils/gs_utils_manualtest.py:60: ('gs://bucket/path1',       'bucket',
'path1'),
On 2014/07/29 20:36:20, rmistry wrote:
> Could you also please add gs://bucket/path1/ and maybe gs://bucket/path1//

Done.

https://codereview.chromium.org/427893002/diff/1/py/utils/gs_utils_manualtest...
py/utils/gs_utils_manualtest.py:61: ('gs://bucket/path1/path2', 'bucket',
'path1/path2'),
On 2014/07/29 20:36:20, rmistry wrote:
> Optional Nit: Instead of aligning some of these lets not align any of them. I
> always prefer not aligning because if we need to add any new test cases that
are
> longer than the longest one then we will have to change all of them instead of
> it being a one-line change.

Yeah, that makes sense, and the style guide agrees with you, so you have THAT
going for you, which is nice...

Done.

[epoger, 2014-07-30 01:11:06]

Committed patchset #2 manually as rf63e1cf (presubmit successful).