commit f703443b1f675f9205c8507e0bcf9d04e04c432e
author Jamie Madill <jmadill@chromium.org> Sat Sep 21 14:10:35 2019 -0400
committer Commit Bot <commit-bot@chromium.org> Wed Sep 25 23:39:34 2019 +0000
tree 9f2aadc68d624b87be2ade9acc94c9616e181c43
parent 29f7916fd56ef664fdf80ced890204569a3994c9

Use Resource IDs in RefCountObject.

This lets us use strongly typed IDs pretty much everywhere. Only one or
two additional places still use GLuint IDs. Mostly for external queries
and for Framebuffer Attachments.

With some clever type reflection helpers lets us define a single
template function for handling operator== and != for resource IDs.

Refactor in preparation for more Capture/Replay work.

Bug: angleproject:3611
Change-Id: I1c0c848e89eb8a4b769714d57686f816daf01634
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/1815550
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Tobin Ehlis <tobine@google.com>

src/libANGLE/Framebuffer.cpp

diff --git a/src/libANGLE/Framebuffer.cpp b/src/libANGLE/Framebuffer.cpp
index bfbebdd..7fa5a3a 100644
--- a/src/libANGLE/Framebuffer.cpp
+++ b/src/libANGLE/Framebuffer.cpp
@@ -1974,7 +1974,7 @@
         // Regardless of if enabled or masked.
         if (checkDepth)
         {
-            if (texture->id() == depth->id())
+            if (texture->getId() == depth->id())
             {
                 return true;
             }
@@ -1982,7 +1982,7 @@
 
         if (checkStencil)
         {
-            if (texture->id() == stencil->id())
+            if (texture->getId() == stencil->id())
             {
                 return true;
             }
@@ -2005,7 +2005,7 @@
     return false;
 }
 
-bool Framebuffer::formsCopyingFeedbackLoopWith(GLuint copyTextureID,
+bool Framebuffer::formsCopyingFeedbackLoopWith(TextureID copyTextureID,
                                                GLint copyTextureLevel,
                                                GLint copyTextureLayer) const
 {