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

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

Issue 2463113002: Let AutoplayUmaHelper listen to visibilitychange instead of unload (Closed)
Patch Set: nit Created 4 years, 1 month 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 697 matching lines...) Expand 10 before | Expand all | Expand 10 after
708 WebAudioSourceProvider* m_webAudioSourceProvider; 708 WebAudioSourceProvider* m_webAudioSourceProvider;
709 Member<AudioClientImpl> m_client; 709 Member<AudioClientImpl> m_client;
710 Mutex provideInputLock; 710 Mutex provideInputLock;
711 }; 711 };
712 712
713 AudioSourceProviderImpl m_audioSourceProvider; 713 AudioSourceProviderImpl m_audioSourceProvider;
714 714
715 class AutoplayHelperClientImpl; 715 class AutoplayHelperClientImpl;
716 716
717 friend class AutoplayUmaHelper; // for isAutoplayAllowedPerSettings 717 friend class AutoplayUmaHelper; // for isAutoplayAllowedPerSettings
718 friend class AutoplayUmaHelperTest;
718 friend class Internals; 719 friend class Internals;
719 friend class TrackDisplayUpdateScope; 720 friend class TrackDisplayUpdateScope;
720 friend class AutoplayExperimentHelper; 721 friend class AutoplayExperimentHelper;
721 friend class MediaControlsTest; 722 friend class MediaControlsTest;
722 723
723 Member<AutoplayExperimentHelper::Client> m_autoplayHelperClient; 724 Member<AutoplayExperimentHelper::Client> m_autoplayHelperClient;
724 Member<AutoplayExperimentHelper> m_autoplayHelper; 725 Member<AutoplayExperimentHelper> m_autoplayHelper;
725 Member<AutoplayUmaHelper> m_autoplayUmaHelper; 726 Member<AutoplayUmaHelper> m_autoplayUmaHelper;
726 727
727 WebRemotePlaybackClient* m_remotePlaybackClient; 728 WebRemotePlaybackClient* m_remotePlaybackClient;
728 729
729 // class AutoplayVisibilityObserver; 730 // class AutoplayVisibilityObserver;
730 Member<ElementVisibilityObserver> m_autoplayVisibilityObserver; 731 Member<ElementVisibilityObserver> m_autoplayVisibilityObserver;
731 732
732 static URLRegistry* s_mediaStreamRegistry; 733 static URLRegistry* s_mediaStreamRegistry;
733 }; 734 };
734 735
735 inline bool isHTMLMediaElement(const HTMLElement& element) { 736 inline bool isHTMLMediaElement(const HTMLElement& element) {
736 return isHTMLAudioElement(element) || isHTMLVideoElement(element); 737 return isHTMLAudioElement(element) || isHTMLVideoElement(element);
737 } 738 }
738 739
739 DEFINE_HTMLELEMENT_TYPE_CASTS_WITH_FUNCTION(HTMLMediaElement); 740 DEFINE_HTMLELEMENT_TYPE_CASTS_WITH_FUNCTION(HTMLMediaElement);
740 741
741 } // namespace blink 742 } // namespace blink
742 743
743 #endif // HTMLMediaElement_h 744 #endif // HTMLMediaElement_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/html/AutoplayUmaHelperTest.cpp ('k') | third_party/WebKit/Source/core/html/OWNERS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698