Update how SkPath handles fGenerationID and fSourcePath, and add tests to cover.

src/core/SkPath.cpp

Index: src/core/SkPath.cpp
diff --git a/src/core/SkPath.cpp b/src/core/SkPath.cpp
index ed2f555ca56bfcca932c7be2a33621faffa35748..d11ac7aac6e8c90198bbd1813235a856fe91379b 100644
--- a/src/core/SkPath.cpp
+++ b/src/core/SkPath.cpp
@@ -180,7 +180,8 @@ void SkPath::resetFields() {
     fIsOval = false;
 #ifdef SK_BUILD_FOR_ANDROID
     GEN_ID_INC;
-    fSourcePath = NULL;
+    // We don't touch fSourcePath.  It's ued to track texture garbage collection, so we don't

[bungeman-skia, 2013/08/12 19:13:31]

s/ued/used

[mtklein, 2013/08/12 19:19:36]

Done.

+    // want to muck with it if it's been set to something non-NULL.
 #endif
 }
 
@@ -189,7 +190,7 @@ SkPath::SkPath(const SkPath& that)
     this->copyFields(that);
 #ifdef SK_BUILD_FOR_ANDROID
     fGenerationID = that.fGenerationID;
-    fSourcePath   = NULL;  // TODO(mtklein): follow up with Android: do we want to copy this too?
+    fSourcePath   = that.fSourcePath;
 #endif
     SkDEBUGCODE(that.validate();)
 }
@@ -206,7 +207,7 @@ SkPath& SkPath::operator=(const SkPath& that) {
         this->copyFields(that);
 #ifdef SK_BUILD_FOR_ANDROID
         GEN_ID_INC;  // Similar to swap, we can't just copy this or it could go back in time.
-        fSourcePath = NULL;  // TODO(mtklein): follow up with Android: do we want to copy this too?
+        fSourcePath = that.fSourcePath;
 #endif
     }
     SkDEBUGCODE(this->validate();)