Merged all PointerToId functions into TraceHelper::PointerToString.

third_party/WebKit/Source/platform/scheduler/base/trace_helper.h

Index: third_party/WebKit/Source/platform/scheduler/base/trace_helper.h
diff --git a/third_party/WebKit/Source/platform/scheduler/base/trace_helper.h b/third_party/WebKit/Source/platform/scheduler/base/trace_helper.h
new file mode 100644
index 0000000000000000000000000000000000000000..372561988b9a579b7c64a68f0dbc08b031016dc8
--- /dev/null
+++ b/third_party/WebKit/Source/platform/scheduler/base/trace_helper.h
@@ -0,0 +1,22 @@
+// Copyright 2017 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 THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_BASE_TRACE_HELPER_H
+#define THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_BASE_TRACE_HELPER_H
+
+#include <string>
+
+namespace blink {
+namespace scheduler {
+
+class TraceHelper {

[jochen (gone - plz use gerrit), 2017/05/11 08:53:11]

nit. should be a namespace (all static classes in general should be namespaces)

[Daniel Bratell, 2017/05/11 11:51:20]

Done.

+ public:
+  static std::string PointerToString(const void* pointer,
+                                     bool add_hex_prefix = true);

[jochen (gone - plz use gerrit), 2017/05/11 08:53:11]

ideally that would be an enum class, no a bool

[Daniel Bratell, 2017/05/11 11:51:20]

I have been uncertain about this flag since it's only needed for a single call
site. Making it an enum would clearly pass the threshold of "not worth it" so I
dropped the flag completely instead and let that call site handle the conversion
itself.

+};
+
+}  // namespace scheduler
+}  // namespace blink
+
+#endif  // THIRD_PARTY_WEBKIT_SOURCE_PLATFORM_SCHEDULER_BASE_TRACE_HELPER_H