WebGL: Don't destroy mailbox textures in the destructor until they're released.

public/platform/WebExternalTextureMailbox.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 26 matching lines @@
     signed char name[64];
     unsigned syncPoint;
 
     WebExternalTextureMailbox()
         : syncPoint(0)
     {
         memset(name, 0, sizeof(name));
     }
 };
 
+inline bool operator==(const WebExternalTextureMailbox& a, const WebExternalTextureMailbox& b)
+{
+    return !memcmp(a.name, b.name, sizeof(a.name));

[abarth-chromium, 2014/04/24 14:59:06]

We don't need to compare the syncPoint as well?

[dshwang, 2014/04/24 15:03:23]

we don't need syncPoint comparison. The main purpose of operator==() is to
compare webgl's mailbox to compositor's mailbox. compositor usually changes sync
point.

[danakj, 2014/04/24 15:10:02]

If you're not comparing every field, might I suggest a method instead of
operator== such as NameEquals()?

[dshwang, 2014/04/24 15:19:11]

nameEquals() looks better. It's why I wait while don't click commit.
I'll update soon. :)

+}
+
+inline bool operator!=(const WebExternalTextureMailbox& a, const WebExternalTextureMailbox& b)
+{
+    return !(a == b);
+}
+
 } // namespace blink
 
 #endif // WebExternalTextureMailbox_h