Make cc::SurfaceId unguessable

cc/surfaces/surface_id.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_SURFACES_SURFACE_ID_H_
 #define CC_SURFACES_SURFACE_ID_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <functional>
 
+#include "base/format_macros.h"
+#include "base/hash.h"
+#include "base/strings/stringprintf.h"
+
 namespace cc {
 
-struct SurfaceId {
-  SurfaceId() : id(0) {}
-  explicit SurfaceId(uint64_t id) : id(id) {}
+class SurfaceId {
+ public:
+  SurfaceId() : id_namespace_(0), local_id_(0), nonce_(0) {}
 
-  bool is_null() const { return id == 0; }
+  SurfaceId(const SurfaceId& other)
+      : id_namespace_(other.id_namespace_),
+        local_id_(other.local_id_),
+        nonce_(other.nonce_) {}
 
+  SurfaceId(uint32_t id_namespace, uint32_t local_id, uint64_t nonce)
+      : id_namespace_(id_namespace),
+        local_id_(local_id),
+        nonce_(nonce)
+
+  {}
+
+  bool is_null() const {
+    return id_namespace_ == 0 && nonce_ == 0 && local_id_ == 0;
+  }
+
+  size_t hash() const {
+    size_t interim = base::HashInts(id_namespace_, local_id_);
+    return base::HashInts(interim, nonce_);
+  }
+
+  uint32_t id_namespace() const { return id_namespace_; }
+
+  uint32_t local_id() const { return local_id_; }
+
+  uint64_t nonce() const { return nonce_; }
+
+  std::string ToString() const {
+    return base::StringPrintf("%d:%" PRIuS ":%d", id_namespace_, nonce_,
+                              local_id_);
+  }
+
+  bool operator==(const SurfaceId& other) const {
+    return (id_namespace_ == other.id_namespace_) &&

[Tom Sepez, 2016/05/25 00:09:53]

nit: overparenthesized.

[Fady Samuel, 2016/05/25 00:57:45]

Done.

+           (local_id_ == other.local_id_) && (nonce_ == other.nonce_);
+  }
+
+  bool operator!=(const SurfaceId& other) const { return !(*this == other); }
+
+  bool operator<(const SurfaceId& other) const {
+    if (id_namespace_ < other.id_namespace_)

[Tom Sepez, 2016/05/25 00:09:53]

nit: this is fine, but still reads a little weird.  Maybe

if (id_namespace != other.id_namespace)
  return id_namespace < other.id_namespace;

if (local_id != other.local_id)
  return local_id < other.local_id;

return nonce < other.nonce;

[Fady Samuel, 2016/05/25 00:57:45]

Nice! Much nicer! Done.

[danakj, 2016/05/25 00:58:55]

The "modern" way to do that is with std::tie(), and compare 2 tuples.

[Fady Samuel, 2016/05/25 01:07:41]

I totally forgot about std::tie. Thanks! Let's go modern! Done. Thanks!

+      return true;
+
+    if (local_id_ < other.local_id_)
+      return id_namespace_ == other.id_namespace_;
+
+    if (nonce_ < other.nonce_) {
+      return id_namespace_ == other.id_namespace_ &&
+             local_id_ == other.local_id_;
+    }
+
+    return false;
+  }
+
+ private:
   // See SurfaceIdAllocator::GenerateId.
-  uint32_t id_namespace() const { return id >> 32; }
-
-  uint64_t id;
+  uint32_t id_namespace_;
+  uint32_t local_id_;
+  uint64_t nonce_;
 };
 
-inline bool operator==(const SurfaceId& a, const SurfaceId& b) {
-  return a.id == b.id;
-}
-
-inline bool operator!=(const SurfaceId& a, const SurfaceId& b) {
-  return !(a == b);
-}
-
-inline bool operator<(const SurfaceId& a, const SurfaceId& b) {
-  return a.id < b.id;
-}
-
 struct SurfaceIdHash {
-  size_t operator()(const SurfaceId& key) const {
-    return std::hash<uint64_t>()(key.id);
-  }
+  size_t operator()(const SurfaceId& key) const { return key.hash(); }
 };
 
 }  // namespace cc
 
 #endif  // CC_SURFACES_SURFACE_ID_H_