Revert "Revert of Use the GrCacheable ID to eliminate the need for notifications to GrGpuGL when te…

src/gpu/GrCacheable.cpp

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "GrCacheable.h"
 
-uint32_t GrCacheable::getGenerationID() const {
-    static int32_t gPathRefGenerationID;
-    while (!fGenID) {
-        fGenID = static_cast<uint32_t>(sk_atomic_inc(&gPathRefGenerationID) + 1);
-    }
-    return fGenID;
+uint32_t GrCacheable::CreateUniqueID() {
+    static int32_t gCacheableID = SK_InvalidUniqueID;
+    uint32_t id;
+    do {
+        id = static_cast<uint32_t>(sk_atomic_inc(&gCacheableID) + 1);

[robertphillips, 2014/07/22 19:27:37]

Is this test right?

[bsalomon, 2014/07/22 19:49:09]

I'm losing it.

+    } while (id != SK_InvalidUniqueID);

[robertphillips, 2014/07/22 19:27:37]

SkASSERT(SK_InvalidUniqueID != id); ?

[bsalomon, 2014/07/22 19:49:09]

With the corrected loop condition it seems unnecessary. I realize I made the
typo before that would hit the assert, but had I run the code it would have been
obvious as any thread that reaches here goes into a practically infinite loop.

+    return id;
 }