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

Side by Side Diff: content/browser/storage_partition_impl.cc

Issue 2788813002: Try to flush DOMStorage too when ending the browsing session.
Patch Set: fixes Created 3 years, 8 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 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/storage_partition_impl.h" 5 #include "content/browser/storage_partition_impl.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <set> 9 #include <set>
10 #include <vector> 10 #include <vector>
(...skipping 876 matching lines...) Expand 10 before | Expand all | Expand 10 after
887 if (url_matcher.is_null()) { 887 if (url_matcher.is_null()) {
888 StoragePartitionHttpCacheDataRemover::CreateForRange(this, begin, end) 888 StoragePartitionHttpCacheDataRemover::CreateForRange(this, begin, end)
889 ->Remove(callback); 889 ->Remove(callback);
890 } else { 890 } else {
891 StoragePartitionHttpCacheDataRemover::CreateForURLsAndRange( 891 StoragePartitionHttpCacheDataRemover::CreateForURLsAndRange(
892 this, url_matcher, begin, end) 892 this, url_matcher, begin, end)
893 ->Remove(callback); 893 ->Remove(callback);
894 } 894 }
895 } 895 }
896 896
897 void StoragePartitionImpl::Flush() { 897 std::vector<scoped_refptr<base::SequencedTaskRunner>>
898 StoragePartitionImpl::Flush() {
898 DCHECK_CURRENTLY_ON(BrowserThread::UI); 899 DCHECK_CURRENTLY_ON(BrowserThread::UI);
899 if (GetDOMStorageContext()) 900 std::vector<scoped_refptr<base::SequencedTaskRunner>> flush_runners;
900 GetDOMStorageContext()->Flush(); 901 if (dom_storage_context_)
902 flush_runners = dom_storage_context_->Flush();
903 return flush_runners;
901 } 904 }
902 905
903 void StoragePartitionImpl::ClearBluetoothAllowedDevicesMapForTesting() { 906 void StoragePartitionImpl::ClearBluetoothAllowedDevicesMapForTesting() {
904 bluetooth_allowed_devices_map_->Clear(); 907 bluetooth_allowed_devices_map_->Clear();
905 } 908 }
906 909
907 BrowserContext* StoragePartitionImpl::browser_context() const { 910 BrowserContext* StoragePartitionImpl::browser_context() const {
908 return browser_context_; 911 return browser_context_;
909 } 912 }
910 913
(...skipping 22 matching lines...) Expand all
933 media_url_request_context_ = media_url_request_context; 936 media_url_request_context_ = media_url_request_context;
934 } 937 }
935 938
936 void StoragePartitionImpl::GetQuotaSettings( 939 void StoragePartitionImpl::GetQuotaSettings(
937 const storage::OptionalQuotaSettingsCallback& callback) { 940 const storage::OptionalQuotaSettingsCallback& callback) {
938 GetContentClient()->browser()->GetQuotaSettings(browser_context_, this, 941 GetContentClient()->browser()->GetQuotaSettings(browser_context_, this,
939 callback); 942 callback);
940 } 943 }
941 944
942 } // namespace content 945 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698