[DevTools] Move field initialization into header for HeadlessShell.

This makes it a little easier to read / removes an ifdef.

Change-Id: If1d6505bf4820508e4bdd212be54bb3cb110a786
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1593733
Commit-Queue: Johannes Henkel <johannes@chromium.org>
Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#664824}

headless/app/headless_shell.cc

@@ -172,15 +172,7 @@ int RunContentMain(
 
 }  // namespace
 
-HeadlessShell::HeadlessShell()
-    : browser_(nullptr),
-#if !defined(CHROME_MULTIPLE_DLL_CHILD)
-      web_contents_(nullptr),
-      browser_context_(nullptr),
-#endif
-      processed_page_ready_(false),
-      weak_factory_(this) {
-}
+HeadlessShell::HeadlessShell() : weak_factory_(this) {}
 
 HeadlessShell::~HeadlessShell() = default;
 

headless/app/headless_shell.h

@@ -97,13 +97,13 @@ class HeadlessShell : public HeadlessWebContents::Observer,
   bool RemoteDebuggingEnabled() const;
 
   GURL url_;
-  HeadlessBrowser* browser_;  // Not owned.
+  HeadlessBrowser* browser_ = nullptr;  // Not owned.
   std::unique_ptr<HeadlessDevToolsClient> devtools_client_;
 #if !defined(CHROME_MULTIPLE_DLL_CHILD)
-  HeadlessWebContents* web_contents_;
-  HeadlessBrowserContext* browser_context_;
+  HeadlessWebContents* web_contents_ = nullptr;
+  HeadlessBrowserContext* browser_context_ = nullptr;
 #endif
-  bool processed_page_ready_;
+  bool processed_page_ready_ = false;
   scoped_refptr<base::SequencedTaskRunner> file_task_runner_;
   std::unique_ptr<base::FileProxy> file_proxy_;
   base::WeakPtrFactory<HeadlessShell> weak_factory_;