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

Side by Side Diff: third_party/WebKit/Source/core/html/HTMLMediaElement.h

Issue 2713613002: Convert HTMLMediaElement timers to MediaElementEvent per-frame. (Closed)
Patch Set: 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 /* 1 /*
2 * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights 2 * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights
3 * reserved. 3 * reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 597 matching lines...) Expand 10 before | Expand all | Expand 10 after
608 // The load is deferred, and waiting for the task to set the 608 // The load is deferred, and waiting for the task to set the
609 // delaying-the-load-event flag (to false). 609 // delaying-the-load-event flag (to false).
610 WaitingForStopDelayingLoadEventTask, 610 WaitingForStopDelayingLoadEventTask,
611 // The load is the deferred, and waiting for a triggering event. 611 // The load is the deferred, and waiting for a triggering event.
612 WaitingForTrigger, 612 WaitingForTrigger,
613 // The load is deferred, and waiting for the task to set the 613 // The load is deferred, and waiting for the task to set the
614 // delaying-the-load-event flag, after which the load will be executed. 614 // delaying-the-load-event flag, after which the load will be executed.
615 ExecuteOnStopDelayingLoadEventTask 615 ExecuteOnStopDelayingLoadEventTask
616 }; 616 };
617 DeferredLoadState m_deferredLoadState; 617 DeferredLoadState m_deferredLoadState;
618 Timer<HTMLMediaElement> m_deferredLoadTimer; 618 TaskRunnerTimer<HTMLMediaElement> m_deferredLoadTimer;
mlamouri (slow - plz ping) 2017/02/22 12:10:03 Why wasn't it a TaskRunnerTimer<> like the others?
619 619
620 std::unique_ptr<WebMediaPlayer> m_webMediaPlayer; 620 std::unique_ptr<WebMediaPlayer> m_webMediaPlayer;
621 WebLayer* m_webLayer; 621 WebLayer* m_webLayer;
622 622
623 DisplayMode m_displayMode; 623 DisplayMode m_displayMode;
624 624
625 Member<HTMLMediaSource> m_mediaSource; 625 Member<HTMLMediaSource> m_mediaSource;
626 626
627 // Stores "official playback position", updated periodically from "current 627 // Stores "official playback position", updated periodically from "current
628 // playback position". Official playback position should not change while 628 // playback position". Official playback position should not change while
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
754 754
755 inline bool isHTMLMediaElement(const HTMLElement& element) { 755 inline bool isHTMLMediaElement(const HTMLElement& element) {
756 return isHTMLAudioElement(element) || isHTMLVideoElement(element); 756 return isHTMLAudioElement(element) || isHTMLVideoElement(element);
757 } 757 }
758 758
759 DEFINE_HTMLELEMENT_TYPE_CASTS_WITH_FUNCTION(HTMLMediaElement); 759 DEFINE_HTMLELEMENT_TYPE_CASTS_WITH_FUNCTION(HTMLMediaElement);
760 760
761 } // namespace blink 761 } // namespace blink
762 762
763 #endif // HTMLMediaElement_h 763 #endif // HTMLMediaElement_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698