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

Side by Side Diff: webkit/plugins/ppapi/plugin_delegate.h

Issue 10416004: RefCounted types should not have public destructors, webkit/ edition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase to r140259 Created 8 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « webkit/plugins/ppapi/fullscreen_container.h ('k') | webkit/quota/quota_manager.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) 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 #ifndef WEBKIT_PLUGINS_PPAPI_PLUGIN_DELEGATE_H_ 5 #ifndef WEBKIT_PLUGINS_PPAPI_PLUGIN_DELEGATE_H_
6 #define WEBKIT_PLUGINS_PPAPI_PLUGIN_DELEGATE_H_ 6 #define WEBKIT_PLUGINS_PPAPI_PLUGIN_DELEGATE_H_
7 7
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 264 matching lines...) Expand 10 before | Expand all | Expand 10 after
275 public: 275 public:
276 virtual ~PlatformVideoCaptureEventHandler() {} 276 virtual ~PlatformVideoCaptureEventHandler() {}
277 277
278 virtual void OnInitialized(media::VideoCapture* capture, 278 virtual void OnInitialized(media::VideoCapture* capture,
279 bool succeeded) = 0; 279 bool succeeded) = 0;
280 }; 280 };
281 281
282 class PlatformVideoCapture : public media::VideoCapture, 282 class PlatformVideoCapture : public media::VideoCapture,
283 public base::RefCounted<PlatformVideoCapture> { 283 public base::RefCounted<PlatformVideoCapture> {
284 public: 284 public:
285 // Detaches the event handler and stops sending notifications to it.
286 virtual void DetachEventHandler() = 0;
287
288 protected:
285 virtual ~PlatformVideoCapture() {} 289 virtual ~PlatformVideoCapture() {}
286 290
287 // Detaches the event handler and stops sending notifications to it. 291 private:
288 virtual void DetachEventHandler() = 0; 292 friend class base::RefCounted<PlatformVideoCapture>;
289 }; 293 };
290 294
291 // Provides access to the ppapi broker. 295 // Provides access to the ppapi broker.
292 class Broker { 296 class Broker {
293 public: 297 public:
294 virtual void Connect(webkit::ppapi::PPB_Broker_Impl* client) = 0; 298 virtual void Connect(webkit::ppapi::PPB_Broker_Impl* client) = 0;
295 299
296 // Decrements the references to the broker. 300 // Decrements the references to the broker.
297 // When there are no more references, this renderer's dispatcher is 301 // When there are no more references, this renderer's dispatcher is
298 // destroyed, allowing the broker to shutdown if appropriate. 302 // destroyed, allowing the broker to shutdown if appropriate.
(...skipping 337 matching lines...) Expand 10 before | Expand all | Expand 10 after
636 virtual webkit_glue::ClipboardClient* CreateClipboardClient() const = 0; 640 virtual webkit_glue::ClipboardClient* CreateClipboardClient() const = 0;
637 641
638 // Returns a Device ID 642 // Returns a Device ID
639 virtual std::string GetDeviceID() = 0; 643 virtual std::string GetDeviceID() = 0;
640 }; 644 };
641 645
642 } // namespace ppapi 646 } // namespace ppapi
643 } // namespace webkit 647 } // namespace webkit
644 648
645 #endif // WEBKIT_PLUGINS_PPAPI_PLUGIN_DELEGATE_H_ 649 #endif // WEBKIT_PLUGINS_PPAPI_PLUGIN_DELEGATE_H_
OLDNEW
« no previous file with comments | « webkit/plugins/ppapi/fullscreen_container.h ('k') | webkit/quota/quota_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698