Remove a bunch of pointless null checks in FrameLoaderClientImpl.

third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.h

 /*
  * 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:
  *
  *     * 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 112 matching lines @@
   RefPtr<WorkerScriptLoader> m_mainScriptLoader;
 
   Persistent<ParentFrameTaskRunners> m_mainThreadTaskRunners;
 
   std::unique_ptr<WorkerThread> m_workerThread;
   RefPtr<WorkerLoaderProxy> m_loaderProxy;
   Persistent<ServiceWorkerGlobalScopeProxy> m_workerGlobalScopeProxy;
   Persistent<WorkerInspectorProxy> m_workerInspectorProxy;
 
   // 'shadow page' - created to proxy loading requests from the worker.
-  // Both WebView and WebFrame objects are close()'ed (where they're
-  // deref'ed) when this EmbeddedWorkerImpl is destructed, therefore they
-  // are guaranteed to exist while this object is around.
+  // WebView is guarantee to be exist while this object exists, as close() is
+  // called in the destructor.
   WebView* m_webView;
   Persistent<WebLocalFrameImpl> m_mainFrame;
 
   bool m_loadingShadowPage;
   bool m_askedToTerminate;
 
   enum WaitingForDebuggerState { WaitingForDebugger, NotWaitingForDebugger };
 
   enum {
     DontPauseAfterDownload,
     DoPauseAfterDownload,
     IsPausedAfterDownload
   } m_pauseAfterDownloadState;
 
   WaitingForDebuggerState m_waitingForDebuggerState;
 };
 
 }  // namespace blink
 
 #endif  // WebEmbeddedWorkerImpl_h