Cleanup: Rename SkOSPath functions.

Cleanup: Rename SkOSPath functions.

Mostly for brevity and matches better with Python:

Python           | Old C++                 | New C++
os.path.join     | SkOSPath::SkPathJoin    | SkOSPath::Join
os.path.basename | SkOSPath::SkBasename    | SkOSPath::Basename

BUG=None
TEST=make all
R=mtklein@google.com

Committed: https://skia.googlesource.com/skia/+/a8e2e1504b9af6ba791637f228debaa23953064a

[mtklein, 2014-07-28 11:32:14]

lgtm

[tfarina, 2014-07-28 12:37:17]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-07-28 12:37:29]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/tfarina@chromium.org/428443002/20001

[commit-bot: I haz the power, 2014-07-28 12:37:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-28 12:37:41]

Presubmit check for 428443002-20001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Since the CL is editing public API, you must have an LGTM from one of:
('reed@chromium.org', 'reed@google.com', 'bsalomon@chromium.org',
'bsalomon@google.com', 'djsollen@chromium.org', 'djsollen@google.com')

[tfarina, 2014-07-28 16:00:55]

On 2014/07/28 12:37:41, I haz the power (commit-bot) wrote:
> Presubmit check for 428443002-20001 failed and returned exit status 1.
> 
> Running presubmit commit checks ...
> 
> ** Presubmit ERRORS **
> Since the CL is editing public API, you must have an LGTM from one of:

Brian, could you approve this?

[bsalomon, 2014-07-28 16:58:13]

lgtm

[tfarina, 2014-07-28 17:37:37]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-07-28 17:37:50]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/tfarina@chromium.org/428443002/20001

[commit-bot: I haz the power, 2014-07-28 17:37:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-28 17:37:59]

Failed to apply patch for dm/DM.cpp:
While running git apply --index -p1;
  error: patch failed: dm/DM.cpp:131
  error: dm/DM.cpp: patch does not apply

Patch:       dm/DM.cpp
Index: dm/DM.cpp
diff --git a/dm/DM.cpp b/dm/DM.cpp
index
003145b92187a92743051a57bd1e5724612d5b2e..79b0a04cfd3111a42765680839525cee26a4b2f2
100644
--- a/dm/DM.cpp
+++ b/dm/DM.cpp
@@ -131,7 +131,7 @@ static void kick_off_skps(DM::Reporter* reporter,
DM::TaskRunner* tasks) {
             continue;
         }
 
-        const SkString path = SkOSPath::SkPathJoin(FLAGS_skps[0],
filename.c_str());
+        const SkString path = SkOSPath::Join(FLAGS_skps[0], filename.c_str());
 
         SkAutoTDelete<SkStream> stream(SkStream::NewFromFile(path.c_str()));
         if (stream.get() == NULL) {

[tfarina, 2014-07-29 01:16:44]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-07-29 01:17:33]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/tfarina@chromium.org/428443002/40001

[commit-bot: I haz the power, 2014-07-29 01:23:47]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  Build-Mac10.8-Clang-x86-Release-Trybot on tryserver.skia
(http://108.170.220.76:10117/builders/Build-Mac10.8-Clang-x86-Release-Trybot/b...)
  Build-Mac10.8-Clang-x86_64-Release-Trybot on tryserver.skia
(http://108.170.220.76:10117/builders/Build-Mac10.8-Clang-x86_64-Release-Trybo...)
  Build-Ubuntu13.10-Clang-x86_64-Debug-Trybot on tryserver.skia
(http://108.170.220.76:10117/builders/Build-Ubuntu13.10-Clang-x86_64-Debug-Try...)
  Build-Ubuntu13.10-GCC4.8-Arm7-Debug-Android-Trybot on tryserver.skia
(http://108.170.220.76:10117/builders/Build-Ubuntu13.10-GCC4.8-Arm7-Debug-Andr...)
  Build-Ubuntu13.10-GCC4.8-x86_64-Release-Trybot on tryserver.skia
(http://108.170.220.76:10117/builders/Build-Ubuntu13.10-GCC4.8-x86_64-Release-...)
  Build-Win-VS2013-x86-Debug-Trybot on tryserver.skia
(http://108.170.220.76:10117/builders/Build-Win-VS2013-x86-Debug-Trybot/builds...)
  Test-Ubuntu13.10-ShuttleA-NoGPU-x86_64-Debug-Trybot on tryserver.skia
(http://108.170.220.120:10117/builders/Test-Ubuntu13.10-ShuttleA-NoGPU-x86_64-...)

[commit-bot: I haz the power, 2014-07-29 01:25:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-29 01:25:38]

Try jobs failed on following builders:
  Test-Ubuntu13.10-ShuttleA-NoGPU-x86_64-Debug-Trybot on tryserver.skia
(http://108.170.220.120:10117/builders/Test-Ubuntu13.10-ShuttleA-NoGPU-x86_64-...)

[tfarina, 2014-07-29 02:14:26]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-07-29 02:14:57]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/tfarina@chromium.org/428443002/60001

[commit-bot: I haz the power, 2014-07-29 02:27:04]

Change committed as a8e2e1504b9af6ba791637f228debaa23953064a