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

Side by Side Diff: Source/web/PluginPlaceholderImpl.h

Issue 1230533002: Fix virtual/override/final usage in Source/web/. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 5 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 PluginPlaceholderImpl_h 5 #ifndef PluginPlaceholderImpl_h
6 #define PluginPlaceholderImpl_h 6 #define PluginPlaceholderImpl_h
7 7
8 #include "core/plugins/PluginPlaceholder.h" 8 #include "core/plugins/PluginPlaceholder.h"
9 #include "platform/heap/Handle.h" 9 #include "platform/heap/Handle.h"
10 #include "platform/heap/Visitor.h" 10 #include "platform/heap/Visitor.h"
(...skipping 15 matching lines...) Expand all
26 public: 26 public:
27 static PassOwnPtrWillBeRawPtr<PluginPlaceholderImpl> create(PassOwnPtr<WebPl uginPlaceholder> webPluginPlaceholder, Document& document) 27 static PassOwnPtrWillBeRawPtr<PluginPlaceholderImpl> create(PassOwnPtr<WebPl uginPlaceholder> webPluginPlaceholder, Document& document)
28 { 28 {
29 ASSERT(webPluginPlaceholder); 29 ASSERT(webPluginPlaceholder);
30 return adoptPtrWillBeNoop(new PluginPlaceholderImpl(webPluginPlaceholder , document)); 30 return adoptPtrWillBeNoop(new PluginPlaceholderImpl(webPluginPlaceholder , document));
31 } 31 }
32 32
33 #if ENABLE(OILPAN) 33 #if ENABLE(OILPAN)
34 ~PluginPlaceholderImpl(); 34 ~PluginPlaceholderImpl();
35 #else 35 #else
36 virtual ~PluginPlaceholderImpl() override; 36 ~PluginPlaceholderImpl() override;
37 #endif 37 #endif
38 38
39 DECLARE_VIRTUAL_TRACE(); 39 DECLARE_VIRTUAL_TRACE();
40 40
41 // PluginPlaceholder methods 41 // PluginPlaceholder methods
42 virtual void loadIntoContainer(ContainerNode&) override; 42 void loadIntoContainer(ContainerNode&) override;
43 43
44 // Visible for testing. 44 // Visible for testing.
45 WebPluginPlaceholder* webPluginPlaceholder() const { return m_webPluginPlace holder.get(); } 45 WebPluginPlaceholder* webPluginPlaceholder() const { return m_webPluginPlace holder.get(); }
46 46
47 private: 47 private:
48 PluginPlaceholderImpl(PassOwnPtr<WebPluginPlaceholder>, Document&); 48 PluginPlaceholderImpl(PassOwnPtr<WebPluginPlaceholder>, Document&);
49 49
50 // Update the placeholder element with fresh content. 50 // Update the placeholder element with fresh content.
51 void update(); 51 void update();
52 52
53 OwnPtr<WebPluginPlaceholder> m_webPluginPlaceholder; 53 OwnPtr<WebPluginPlaceholder> m_webPluginPlaceholder;
54 RefPtrWillBeMember<PluginPlaceholderElement> m_placeholderElement; 54 RefPtrWillBeMember<PluginPlaceholderElement> m_placeholderElement;
55 }; 55 };
56 56
57 } // namespace blink 57 } // namespace blink
58 58
59 #endif // PluginPlaceholderImpl_h 59 #endif // PluginPlaceholderImpl_h
OLDNEW
« no previous file with comments | « Source/web/NotificationPermissionClientImpl.cpp ('k') | Source/web/PluginPlaceholderImplTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698