Restore ScopedLogger and add documentation.

third_party/WebKit/Source/wtf/Assertions.h

 /*
  * Copyright (C) 2003, 2006, 2007 Apple Inc.  All rights reserved.
  * 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 71 matching lines @@
 
 WTF_EXPORT void WTFReportAssertionFailure(const char* file, int line, const char* function, const char* assertion);
 WTF_EXPORT void WTFLog(WTFLogChannel*, const char* format, ...) WTF_ATTRIBUTE_PRINTF(2, 3);
 WTF_EXPORT void WTFLogVerbose(const char* file, int line, const char* function, WTFLogChannel*, const char* format, ...) WTF_ATTRIBUTE_PRINTF(5, 6);
 WTF_EXPORT void WTFLogAlways(const char* format, ...) WTF_ATTRIBUTE_PRINTF(1, 2);
 
 WTF_EXPORT void WTFGetBacktrace(void** stack, int* size);
 WTF_EXPORT void WTFReportBacktrace(int framesToShow = 31);
 WTF_EXPORT void WTFPrintBacktrace(void** stack, int size);
 
+namespace WTF {
+
+// ScopedLogger wraps log messages in parentheses, with indentation proportional
+// to the number of instances. This makes it easy to see the flow of control in
+// the output, particularly when instrumenting recursive functions.

[tkent, 2016/03/28 23:16:30]

Please add a comment that this class is not used in checked-in code, but we'd
like to keep it.
Without it, someonw will remove ScopedLogger again.

[skobes, 2016/03/29 18:23:40]

Done.

+class WTF_EXPORT ScopedLogger {
+    WTF_MAKE_NONCOPYABLE(ScopedLogger);
+public:
+    // The first message is passed to the constructor.  Additional messages for
+    // the same scope can be added with log(). If condition is false, produce no
+    // output and do not create a scope.
+    ScopedLogger(bool condition, const char* format, ...) WTF_ATTRIBUTE_PRINTF(3, 4);
+    ~ScopedLogger();
+    void log(const char* format, ...) WTF_ATTRIBUTE_PRINTF(2, 3);
+
+private:
+    friend class AssertionsTest;

[tkent, 2016/03/28 23:16:30]

Please use FRIEND_TEST_ALL_PREFIXES defined in base/gtest_prod_util.h.

[skobes, 2016/03/29 18:23:40]

Done.

+    using PrintFunctionPtr = void (*)(const char* format, va_list args);
+    static void setPrintFuncForTests(PrintFunctionPtr p) { m_printFunc = p; } // Note: not thread safe.
+
+    void init(const char* format, va_list args);
+    void writeNewlineIfNeeded();
+    void indent();
+    void print(const char* format, ...);
+    static ScopedLogger*& current();
+
+    ScopedLogger* const m_parent;
+    bool m_multiline; // The ')' will go on the same line if there is only one entry.
+    static PrintFunctionPtr m_printFunc;
+};
+
+#if LOG_DISABLED

[tkent, 2016/03/28 23:16:30]

We'd like to remove LOG_DISABLED macro.  Can you make it !ENABLE(ASSERT)?

[skobes, 2016/03/29 18:23:40]

That breaks blink_logging_always_on.

+#define WTF_CREATE_SCOPED_LOGGER(...) ((void) 0)
+#define WTF_CREATE_SCOPED_LOGGER_IF(...) ((void) 0)
+#define WTF_APPEND_SCOPED_LOGGER(...) ((void) 0)
+#else
+#define WTF_CREATE_SCOPED_LOGGER(name, ...) ScopedLogger name(true, __VA_ARGS__)

[tkent, 2016/03/28 23:16:30]

ScopedLogger class should be declared here to avoid to ship it in production.

[skobes, 2016/03/29 18:23:40]

Done.

+#define WTF_CREATE_SCOPED_LOGGER_IF(name, condition, ...) ScopedLogger name(condition, __VA_ARGS__)
+#define WTF_APPEND_SCOPED_LOGGER(name, ...) (name.log(__VA_ARGS__))
+#endif
+
+} // namespace WTF
+
+using WTF::ScopedLogger;
+
 /* IMMEDIATE_CRASH() - Like CRASH() below but crashes in the fastest, simplest possible way with no attempt at logging. */
 #ifndef IMMEDIATE_CRASH
 #if COMPILER(GCC) || COMPILER(CLANG)
 #define IMMEDIATE_CRASH() __builtin_trap()
 #else
 #define IMMEDIATE_CRASH() ((void)(*(volatile char*)0 = 0))
 #endif
 #endif
 
 /* CRASH() - Raises a fatal error resulting in program termination and triggering either the debugger or the crash reporter.
@@ skipping 160 matching lines @@
 } \
 inline const thisType& to##thisType(const argumentType& argumentName) \
 { \
     ASSERT_WITH_SECURITY_IMPLICATION(referencePredicate); \
     return static_cast<const thisType&>(argumentName); \
 } \
 void to##thisType(const thisType*); \
 void to##thisType(const thisType&)
 
 #endif /* WTF_Assertions_h */