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

Side by Side Diff: third_party/WebKit/Source/modules/remoteplayback/RemotePlayback.h

Issue 2675943003: Remove custom left overs from object grouping (Closed)
Patch Set: Rebase (0 changes) Created 3 years, 10 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 RemotePlayback_h 5 #ifndef RemotePlayback_h
6 #define RemotePlayback_h 6 #define RemotePlayback_h
7 7
8 #include "bindings/core/v8/ActiveScriptWrappable.h" 8 #include "bindings/core/v8/ActiveScriptWrappable.h"
9 #include "bindings/core/v8/ScriptPromise.h" 9 #include "bindings/core/v8/ScriptPromise.h"
10 #include "bindings/core/v8/TraceWrapperMember.h" 10 #include "bindings/core/v8/TraceWrapperMember.h"
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 // Calls the specified availability callback with the current availability. 82 // Calls the specified availability callback with the current availability.
83 // Need a void() method to post it as a task. 83 // Need a void() method to post it as a task.
84 void notifyInitialAvailability(int callbackId); 84 void notifyInitialAvailability(int callbackId);
85 85
86 // WebRemotePlaybackClient implementation. 86 // WebRemotePlaybackClient implementation.
87 void stateChanged(WebRemotePlaybackState) override; 87 void stateChanged(WebRemotePlaybackState) override;
88 void availabilityChanged(WebRemotePlaybackAvailability) override; 88 void availabilityChanged(WebRemotePlaybackAvailability) override;
89 void promptCancelled() override; 89 void promptCancelled() override;
90 bool remotePlaybackAvailable() const override; 90 bool remotePlaybackAvailable() const override;
91 91
92 // Prevent v8 from garbage collecting the availability callbacks.
93 // TODO(avayvod): remove when crbug.com/468240 is fixed and the references
94 // are maintained automatically.
95 void setV8ReferencesForCallbacks(v8::Isolate*,
96 const v8::Persistent<v8::Object>& wrapper);
97
98 WebRemotePlaybackState m_state; 92 WebRemotePlaybackState m_state;
99 WebRemotePlaybackAvailability m_availability; 93 WebRemotePlaybackAvailability m_availability;
100 HeapHashMap<int, TraceWrapperMember<RemotePlaybackAvailabilityCallback>> 94 HeapHashMap<int, TraceWrapperMember<RemotePlaybackAvailabilityCallback>>
101 m_availabilityCallbacks; 95 m_availabilityCallbacks;
102 Member<HTMLMediaElement> m_mediaElement; 96 Member<HTMLMediaElement> m_mediaElement;
103 Member<ScriptPromiseResolver> m_promptPromiseResolver; 97 Member<ScriptPromiseResolver> m_promptPromiseResolver;
104 }; 98 };
105 99
106 } // namespace blink 100 } // namespace blink
107 101
108 #endif // RemotePlayback_h 102 #endif // RemotePlayback_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698