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

Side by Side Diff: third_party/WebKit/Source/modules/webaudio/AudioScheduledSourceNode.cpp

Issue 2588603002: Specify TaskType of posted Task explicitly in media elements (12) (Closed)
Patch Set: rebase Created 4 years 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 /* 1 /*
2 * Copyright (C) 2012, Google Inc. All rights reserved. 2 * Copyright (C) 2012, Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
11 * documentation and/or other materials provided with the distribution. 11 * documentation and/or other materials provided with the distribution.
12 * 12 *
13 * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND 13 * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND
14 * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE 14 * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
15 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE 15 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
16 * ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE 16 * ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE
17 * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL 17 * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
18 * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR 18 * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
19 * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER 19 * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
20 * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT 20 * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
21 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY 21 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
22 * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH 22 * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH
23 * DAMAGE. 23 * DAMAGE.
24 */ 24 */
25 25
26 #include "modules/webaudio/AudioScheduledSourceNode.h" 26 #include "modules/webaudio/AudioScheduledSourceNode.h"
27 #include "bindings/core/v8/ExceptionState.h" 27 #include "bindings/core/v8/ExceptionState.h"
28 #include "core/dom/ExceptionCode.h" 28 #include "core/dom/ExceptionCode.h"
29 #include "core/dom/ExecutionContextTask.h" 29 #include "core/dom/ExecutionContextTask.h"
30 #include "core/dom/TaskRunnerHelper.h"
30 #include "modules/EventModules.h" 31 #include "modules/EventModules.h"
31 #include "modules/webaudio/BaseAudioContext.h" 32 #include "modules/webaudio/BaseAudioContext.h"
32 #include "platform/audio/AudioUtilities.h" 33 #include "platform/audio/AudioUtilities.h"
33 #include "wtf/MathExtras.h" 34 #include "wtf/MathExtras.h"
34 #include <algorithm> 35 #include <algorithm>
35 36
36 namespace blink { 37 namespace blink {
37 38
38 const double AudioScheduledSourceHandler::UnknownTime = -1; 39 const double AudioScheduledSourceHandler::UnknownTime = -1;
39 40
(...skipping 174 matching lines...) Expand 10 before | Expand all | Expand 10 after
214 context()->notifySourceNodeFinishedProcessing(this); 215 context()->notifySourceNodeFinishedProcessing(this);
215 setPlaybackState(FINISHED_STATE); 216 setPlaybackState(FINISHED_STATE);
216 } 217 }
217 } 218 }
218 219
219 void AudioScheduledSourceHandler::finish() { 220 void AudioScheduledSourceHandler::finish() {
220 finishWithoutOnEnded(); 221 finishWithoutOnEnded();
221 222
222 if (context()->getExecutionContext()) { 223 if (context()->getExecutionContext()) {
223 context()->getExecutionContext()->postTask( 224 context()->getExecutionContext()->postTask(
224 BLINK_FROM_HERE, 225 TaskType::MediaElementEvent, BLINK_FROM_HERE,
225 createCrossThreadTask(&AudioScheduledSourceHandler::notifyEnded, 226 createCrossThreadTask(&AudioScheduledSourceHandler::notifyEnded,
226 PassRefPtr<AudioScheduledSourceHandler>(this))); 227 PassRefPtr<AudioScheduledSourceHandler>(this)));
227 } 228 }
228 } 229 }
229 230
230 void AudioScheduledSourceHandler::notifyEnded() { 231 void AudioScheduledSourceHandler::notifyEnded() {
231 DCHECK(isMainThread()); 232 DCHECK(isMainThread());
232 if (node()) 233 if (node())
233 node()->dispatchEvent(Event::create(EventTypeNames::ended)); 234 node()->dispatchEvent(Event::create(EventTypeNames::ended));
234 } 235 }
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
274 // playback state if the context is closed. 275 // playback state if the context is closed.
275 if (context()->isContextClosed()) 276 if (context()->isContextClosed())
276 return false; 277 return false;
277 278
278 // If a node is scheduled or playing, do not collect the node prematurely 279 // If a node is scheduled or playing, do not collect the node prematurely
279 // even its reference is out of scope. Then fire onended event if assigned. 280 // even its reference is out of scope. Then fire onended event if assigned.
280 return audioScheduledSourceHandler().isPlayingOrScheduled(); 281 return audioScheduledSourceHandler().isPlayingOrScheduled();
281 } 282 }
282 283
283 } // namespace blink 284 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698