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

Side by Side Diff: storage/browser/blob/blob_data_handle.h

Issue 669603008: Standardize usage of virtual/override/final in storage/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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
« no previous file with comments | « no previous file | storage/browser/blob/blob_url_request_job.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 #ifndef STORAGE_BROWSER_BLOB_BLOB_DATA_HANDLE_H_ 5 #ifndef STORAGE_BROWSER_BLOB_BLOB_DATA_HANDLE_H_
6 #define STORAGE_BROWSER_BLOB_BLOB_DATA_HANDLE_H_ 6 #define STORAGE_BROWSER_BLOB_BLOB_DATA_HANDLE_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/memory/ref_counted.h" 10 #include "base/memory/ref_counted.h"
(...skipping 11 matching lines...) Expand all
22 class BlobStorageContext; 22 class BlobStorageContext;
23 23
24 // A scoper object for use in chrome's main browser process, ensures 24 // A scoper object for use in chrome's main browser process, ensures
25 // the underlying BlobData and its uuid remain in BlobStorageContext's 25 // the underlying BlobData and its uuid remain in BlobStorageContext's
26 // collection for the duration. This object has delete semantics and 26 // collection for the duration. This object has delete semantics and
27 // maybe deleted on any thread. 27 // maybe deleted on any thread.
28 class STORAGE_EXPORT BlobDataHandle 28 class STORAGE_EXPORT BlobDataHandle
29 : public base::SupportsUserData::Data { 29 : public base::SupportsUserData::Data {
30 public: 30 public:
31 BlobDataHandle(const BlobDataHandle& other); // May be copied on any thread. 31 BlobDataHandle(const BlobDataHandle& other); // May be copied on any thread.
32 virtual ~BlobDataHandle(); // Maybe be deleted on any thread. 32 ~BlobDataHandle() override; // Maybe be deleted on any thread.
33 BlobData* data() const; // May only be accessed on the IO thread. 33 BlobData* data() const; // May only be accessed on the IO thread.
34 34
35 std::string uuid() const; // May be accessed on any thread. 35 std::string uuid() const; // May be accessed on any thread.
36 36
37 private: 37 private:
38 class BlobDataHandleShared 38 class BlobDataHandleShared
39 : public base::RefCountedThreadSafe<BlobDataHandleShared> { 39 : public base::RefCountedThreadSafe<BlobDataHandleShared> {
40 public: 40 public:
41 BlobDataHandleShared(BlobData* blob_data, 41 BlobDataHandleShared(BlobData* blob_data,
42 BlobStorageContext* context, 42 BlobStorageContext* context,
(...skipping 19 matching lines...) Expand all
62 BlobDataHandle(BlobData* blob_data, BlobStorageContext* context, 62 BlobDataHandle(BlobData* blob_data, BlobStorageContext* context,
63 base::SequencedTaskRunner* task_runner); 63 base::SequencedTaskRunner* task_runner);
64 64
65 scoped_refptr<base::SequencedTaskRunner> io_task_runner_; 65 scoped_refptr<base::SequencedTaskRunner> io_task_runner_;
66 scoped_refptr<BlobDataHandleShared> shared_; 66 scoped_refptr<BlobDataHandleShared> shared_;
67 }; 67 };
68 68
69 } // namespace storage 69 } // namespace storage
70 70
71 #endif // STORAGE_BROWSER_BLOB_BLOB_DATA_HANDLE_H_ 71 #endif // STORAGE_BROWSER_BLOB_BLOB_DATA_HANDLE_H_
OLDNEW
« no previous file with comments | « no previous file | storage/browser/blob/blob_url_request_job.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698