Introduced WebFactory and WebFactoryImpl for constructing web/ classes

third_party/WebKit/Source/web/WebViewImpl.h

Index: third_party/WebKit/Source/web/WebViewImpl.h
diff --git a/third_party/WebKit/Source/web/WebViewImpl.h b/third_party/WebKit/Source/web/WebViewImpl.h
index 2898f9c3d2d7c1b29a0ac4778ffdec4edceb3105..d860876a440450d25b713c97727f98300b636141 100644
--- a/third_party/WebKit/Source/web/WebViewImpl.h
+++ b/third_party/WebKit/Source/web/WebViewImpl.h
@@ -34,6 +34,7 @@
 #include <memory>
 #include "core/exported/WebViewBase.h"
 #include "core/frame/ResizeViewportAnchor.h"
+#include "core/page/ChromeClient.h"
 #include "core/page/ContextMenuProvider.h"
 #include "core/page/EventWithHitTestResults.h"
 #include "platform/animation/CompositorAnimationTimeline.h"
@@ -60,7 +61,6 @@
 #include "public/platform/WebVector.h"
 #include "public/web/WebNavigationPolicy.h"
 #include "public/web/WebPageImportanceSignals.h"
-#include "web/ChromeClientImpl.h"
 #include "web/ContextMenuClientImpl.h"
 #include "web/EditorClientImpl.h"
 #include "web/MediaKeysClientImpl.h"
@@ -498,7 +498,7 @@ class WEB_EXPORT WebViewImpl final
   }
 
   class ChromeClient& ChromeClient() const override {
-    return *chrome_client_impl_.Get();
+    return *chrome_client_.Get();
   }
 
   // Returns the currently active WebInputMethodController which is the one
@@ -606,7 +606,7 @@ class WEB_EXPORT WebViewImpl final
   WebViewClient* client_;  // Can be 0 (e.g. unittests, shared workers, etc.)
   WebSpellCheckClient* spell_check_client_;
 
-  Persistent<ChromeClientImpl> chrome_client_impl_;
+  Persistent<blink::ChromeClient> chrome_client_;

[haraken, 2017/05/23 01:56:35]

Drop blink::.

[sashab, 2017/05/29 04:34:01]

Can't, name clash with method above :( Added comment.

   ContextMenuClientImpl context_menu_client_impl_;
   EditorClientImpl editor_client_impl_;
   SpellCheckerClientImpl spell_checker_client_impl_;