Separate ContentSettingsClient out from LocalFrameClient

third_party/WebKit/Source/core/loader/EmptyClients.cpp

Index: third_party/WebKit/Source/core/loader/EmptyClients.cpp
diff --git a/third_party/WebKit/Source/core/loader/EmptyClients.cpp b/third_party/WebKit/Source/core/loader/EmptyClients.cpp
index 812954ab5784a6d87007177d4d270daf8cdc54ab..0ac568a143637df62e3d8e1e3dffc5b87d76b0d9 100644
--- a/third_party/WebKit/Source/core/loader/EmptyClients.cpp
+++ b/third_party/WebKit/Source/core/loader/EmptyClients.cpp
@@ -28,6 +28,7 @@
 #include "core/loader/EmptyClients.h"
 
 #include <memory>
+#include "core/frame/ContentSettingsClient.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/VisualViewport.h"
 #include "core/html/HTMLFormElement.h"
@@ -195,6 +196,10 @@ EmptyLocalFrameClient::createServiceWorkerProvider() {
   return nullptr;
 }
 
+ContentSettingsClient* EmptyLocalFrameClient::contentSettingsClient() {
+  return new ContentSettingsClient();

[dcheng, 2017/04/02 05:26:19]

It seems a little surprising that this creates a new one every time. Would it be
possible to just make this a member?

[kinuko, 2017/04/03 15:15:06]

Oh yes, this was a tentative code I was going to fix. Done.

+}
+
 std::unique_ptr<WebApplicationCacheHost>
 EmptyLocalFrameClient::createApplicationCacheHost(
     WebApplicationCacheHostClient*) {