libFuzzer for blink::MHTMLParser

third_party/WebKit/Source/platform/testing/TestingPlatformSupport.h

 /*
  * Copyright (C) 2014 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 98 matching lines @@
     ~TestingPlatformSupportWithMockScheduler() override;
 
     // Platform:
     WebThread* currentThread() override;
     TestingPlatformMockScheduler* mockWebScheduler();
 
 protected:
     std::unique_ptr<TestingPlatformMockWebThread> m_mockWebThread;
 };
 
+class ScopedUnittestsEnvironmentSetup {
+    WTF_MAKE_NONCOPYABLE(ScopedUnittestsEnvironmentSetup);
+public:
+    enum class TestType {
+        UnitTests,
+        LibFuzzer,

[esprehn, 2016/08/04 04:37:53]

I'd rather not do this, what code are you trying to share here? Why do we need a
separate setup?

[Łukasz Anforowicz, 2016/08/04 17:37:30]

I am trying to 1) consolidate all one-time test setup into
ScopedUnittestsEnvironmentSetup, 2) avoid i18n/ICU initialization from
MHTMLFuzzer.cpp (i.e. I want to limit base/i18n dependency to platform/testing
directory).
We don't in the latest patchset - separate setup has been avoided by
introduction of InitializeICUForTesting in base/test/icu_test_util.h (and
described in a reply to your other feedback).

+    };
+
+    ScopedUnittestsEnvironmentSetup(int argc, char** argv, TestType);
+    ~ScopedUnittestsEnvironmentSetup();
+
+private:
+    class Impl;

[esprehn, 2016/08/04 04:37:53]

Why do you need the nested class? Maybe you want DEFINE_STATIC_LOCAL for your
static to avoid the destructors?

[Łukasz Anforowicz, 2016/08/04 17:37:30]

I wanted to avoid including base/ headers from TestingPlatformSupport.h (i.e.
limit including base/headers to TestingPlatformSupport.cpp).  This was before I
realized that most of the base classes will be wrapped in std::unique_ptr (and
therefore can be forward declared).  Still - there are some base objects
embedded in the Impl class - I think it is desirable to keep them there.

WDYT?
Hmmm... maybe.  Without the destructors (i.e. when used as a *static* variable),
RunAllTests.cpp behavior would change (i.e. no cleanup from ~Impl would happen).

+    std::unique_ptr<Impl> m_impl;
+};
+
 } // namespace blink
 
 #endif // TestingPlatformSupport_h