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

Side by Side Diff: content/browser/renderer_host/pepper/pepper_file_system_browser_host.h

Issue 667943003: Standardize usage of virtual/override/final in content/browser/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 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 CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_SYSTEM_BROWSER_HOST_H_ 5 #ifndef CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_SYSTEM_BROWSER_HOST_H_
6 #define CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_SYSTEM_BROWSER_HOST_H_ 6 #define CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_SYSTEM_BROWSER_HOST_H_
7 7
8 #include <queue> 8 #include <queue>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 19 matching lines...) Expand all
30 class CONTENT_EXPORT PepperFileSystemBrowserHost 30 class CONTENT_EXPORT PepperFileSystemBrowserHost
31 : public ppapi::host::ResourceHost, 31 : public ppapi::host::ResourceHost,
32 public base::SupportsWeakPtr<PepperFileSystemBrowserHost> { 32 public base::SupportsWeakPtr<PepperFileSystemBrowserHost> {
33 public: 33 public:
34 // Creates a new PepperFileSystemBrowserHost for a file system of a given 34 // Creates a new PepperFileSystemBrowserHost for a file system of a given
35 // |type|. The host must be opened before use. 35 // |type|. The host must be opened before use.
36 PepperFileSystemBrowserHost(BrowserPpapiHost* host, 36 PepperFileSystemBrowserHost(BrowserPpapiHost* host,
37 PP_Instance instance, 37 PP_Instance instance,
38 PP_Resource resource, 38 PP_Resource resource,
39 PP_FileSystemType type); 39 PP_FileSystemType type);
40 virtual ~PepperFileSystemBrowserHost(); 40 ~PepperFileSystemBrowserHost() override;
41 41
42 // Opens the PepperFileSystemBrowserHost to use an existing file system at the 42 // Opens the PepperFileSystemBrowserHost to use an existing file system at the
43 // given |root_url|. The file system at |root_url| must already be opened and 43 // given |root_url|. The file system at |root_url| must already be opened and
44 // have the type given by GetType(). 44 // have the type given by GetType().
45 // Calls |callback| when complete. 45 // Calls |callback| when complete.
46 void OpenExisting(const GURL& root_url, const base::Closure& callback); 46 void OpenExisting(const GURL& root_url, const base::Closure& callback);
47 47
48 // ppapi::host::ResourceHost overrides. 48 // ppapi::host::ResourceHost overrides.
49 virtual int32_t OnResourceMessageReceived( 49 int32_t OnResourceMessageReceived(
50 const IPC::Message& msg, 50 const IPC::Message& msg,
51 ppapi::host::HostMessageContext* context) override; 51 ppapi::host::HostMessageContext* context) override;
52 virtual bool IsFileSystemHost() override; 52 bool IsFileSystemHost() override;
53 53
54 // Supports FileRefs direct access on the host side. 54 // Supports FileRefs direct access on the host side.
55 PP_FileSystemType GetType() const { return type_; } 55 PP_FileSystemType GetType() const { return type_; }
56 bool IsOpened() const { return opened_; } 56 bool IsOpened() const { return opened_; }
57 GURL GetRootUrl() const { return root_url_; } 57 GURL GetRootUrl() const { return root_url_; }
58 scoped_refptr<storage::FileSystemContext> GetFileSystemContext() const { 58 scoped_refptr<storage::FileSystemContext> GetFileSystemContext() const {
59 return file_system_context_; 59 return file_system_context_;
60 } 60 }
61 61
62 // Supports FileIOs direct access on the host side. 62 // Supports FileIOs direct access on the host side.
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
165 std::string fsid_; // used only for isolated filesystems. 165 std::string fsid_; // used only for isolated filesystems.
166 166
167 base::WeakPtrFactory<PepperFileSystemBrowserHost> weak_factory_; 167 base::WeakPtrFactory<PepperFileSystemBrowserHost> weak_factory_;
168 168
169 DISALLOW_COPY_AND_ASSIGN(PepperFileSystemBrowserHost); 169 DISALLOW_COPY_AND_ASSIGN(PepperFileSystemBrowserHost);
170 }; 170 };
171 171
172 } // namespace content 172 } // namespace content
173 173
174 #endif // CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_SYSTEM_BROWSER_HOST_ H_ 174 #endif // CONTENT_BROWSER_RENDERER_HOST_PEPPER_PEPPER_FILE_SYSTEM_BROWSER_HOST_ H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698