Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(19)

Side by Side Diff: third_party/WebKit/Source/web/LocalFrameClientImpl.cpp

Issue 2847983002: Use unique_ptr for Create{ServiceWorkerNetworkProvider,ApplicationCacheHost} (Closed)
Patch Set: addressed comments Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009, 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2009, 2012 Google Inc. All rights reserved.
3 * Copyright (C) 2011 Apple Inc. All rights reserved. 3 * Copyright (C) 2011 Apple Inc. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are 6 * modification, are permitted provided that the following conditions are
7 * met: 7 * met:
8 * 8 *
9 * * Redistributions of source code must retain the above copyright 9 * * Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 889 matching lines...) Expand 10 before | Expand all | Expand 10 after
900 900
901 void LocalFrameClientImpl::DispatchWillInsertBody() { 901 void LocalFrameClientImpl::DispatchWillInsertBody() {
902 if (web_frame_->Client()) 902 if (web_frame_->Client())
903 web_frame_->Client()->WillInsertBody(web_frame_); 903 web_frame_->Client()->WillInsertBody(web_frame_);
904 } 904 }
905 905
906 std::unique_ptr<WebServiceWorkerProvider> 906 std::unique_ptr<WebServiceWorkerProvider>
907 LocalFrameClientImpl::CreateServiceWorkerProvider() { 907 LocalFrameClientImpl::CreateServiceWorkerProvider() {
908 if (!web_frame_->Client()) 908 if (!web_frame_->Client())
909 return nullptr; 909 return nullptr;
910 return WTF::WrapUnique(web_frame_->Client()->CreateServiceWorkerProvider()); 910 return web_frame_->Client()->CreateServiceWorkerProvider();
911 } 911 }
912 912
913 ContentSettingsClient& LocalFrameClientImpl::GetContentSettingsClient() { 913 ContentSettingsClient& LocalFrameClientImpl::GetContentSettingsClient() {
914 return web_frame_->GetContentSettingsClient(); 914 return web_frame_->GetContentSettingsClient();
915 } 915 }
916 916
917 SharedWorkerRepositoryClient* 917 SharedWorkerRepositoryClient*
918 LocalFrameClientImpl::GetSharedWorkerRepositoryClient() { 918 LocalFrameClientImpl::GetSharedWorkerRepositoryClient() {
919 return web_frame_->SharedWorkerRepositoryClient(); 919 return web_frame_->SharedWorkerRepositoryClient();
920 } 920 }
921 921
922 std::unique_ptr<WebApplicationCacheHost> 922 std::unique_ptr<WebApplicationCacheHost>
923 LocalFrameClientImpl::CreateApplicationCacheHost( 923 LocalFrameClientImpl::CreateApplicationCacheHost(
924 WebApplicationCacheHostClient* client) { 924 WebApplicationCacheHostClient* client) {
925 if (!web_frame_->Client()) 925 if (!web_frame_->Client())
926 return nullptr; 926 return nullptr;
927 return WTF::WrapUnique( 927 return web_frame_->Client()->CreateApplicationCacheHost(client);
928 web_frame_->Client()->CreateApplicationCacheHost(client));
929 } 928 }
930 929
931 void LocalFrameClientImpl::DispatchDidChangeManifest() { 930 void LocalFrameClientImpl::DispatchDidChangeManifest() {
932 if (web_frame_->Client()) 931 if (web_frame_->Client())
933 web_frame_->Client()->DidChangeManifest(); 932 web_frame_->Client()->DidChangeManifest();
934 } 933 }
935 934
936 unsigned LocalFrameClientImpl::BackForwardLength() { 935 unsigned LocalFrameClientImpl::BackForwardLength() {
937 WebViewBase* webview = web_frame_->ViewImpl(); 936 WebViewBase* webview = web_frame_->ViewImpl();
938 if (!webview || !webview->Client()) 937 if (!webview || !webview->Client())
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
992 void LocalFrameClientImpl::AbortClientNavigation() { 991 void LocalFrameClientImpl::AbortClientNavigation() {
993 if (web_frame_->Client()) 992 if (web_frame_->Client())
994 web_frame_->Client()->AbortClientNavigation(); 993 web_frame_->Client()->AbortClientNavigation();
995 } 994 }
996 995
997 TextCheckerClient& LocalFrameClientImpl::GetTextCheckerClient() const { 996 TextCheckerClient& LocalFrameClientImpl::GetTextCheckerClient() const {
998 return web_frame_->GetTextCheckerClient(); 997 return web_frame_->GetTextCheckerClient();
999 } 998 }
1000 999
1001 } // namespace blink 1000 } // namespace blink
OLDNEW
« no previous file with comments | « content/renderer/shared_worker/embedded_shared_worker_stub.cc ('k') | third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698