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

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

Issue 2570463005: Retire ThreadState::registerPreFinalizer<T>() (Closed)
Patch Set: 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) 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 400 matching lines...) Expand 10 before | Expand all | Expand 10 after
411 m_playingRemotely(false), 411 m_playingRemotely(false),
412 m_inOverlayFullscreenVideo(false), 412 m_inOverlayFullscreenVideo(false),
413 m_mostlyFillingViewport(false), 413 m_mostlyFillingViewport(false),
414 m_audioTracks(this, AudioTrackList::create(*this)), 414 m_audioTracks(this, AudioTrackList::create(*this)),
415 m_videoTracks(this, VideoTrackList::create(*this)), 415 m_videoTracks(this, VideoTrackList::create(*this)),
416 m_textTracks(this, nullptr), 416 m_textTracks(this, nullptr),
417 m_audioSourceNode(nullptr), 417 m_audioSourceNode(nullptr),
418 m_autoplayUmaHelper(AutoplayUmaHelper::create(this)), 418 m_autoplayUmaHelper(AutoplayUmaHelper::create(this)),
419 m_remotePlaybackClient(nullptr), 419 m_remotePlaybackClient(nullptr),
420 m_autoplayVisibilityObserver(nullptr) { 420 m_autoplayVisibilityObserver(nullptr) {
421 ThreadState::current()->registerPreFinalizer(this);
422
423 BLINK_MEDIA_LOG << "HTMLMediaElement(" << (void*)this << ")"; 421 BLINK_MEDIA_LOG << "HTMLMediaElement(" << (void*)this << ")";
424 422
425 m_lockedPendingUserGesture = computeLockedPendingUserGesture(document); 423 m_lockedPendingUserGesture = computeLockedPendingUserGesture(document);
426 m_lockedPendingUserGestureIfCrossOriginExperimentEnabled = 424 m_lockedPendingUserGestureIfCrossOriginExperimentEnabled =
427 isDocumentCrossOrigin(document); 425 isDocumentCrossOrigin(document);
428 426
429 LocalFrame* frame = document.frame(); 427 LocalFrame* frame = document.frame();
430 if (frame) { 428 if (frame) {
431 m_remotePlaybackClient = 429 m_remotePlaybackClient =
432 frame->loader().client()->createWebRemotePlaybackClient(*this); 430 frame->loader().client()->createWebRemotePlaybackClient(*this);
(...skipping 3665 matching lines...) Expand 10 before | Expand all | Expand 10 after
4098 kMostlyFillViewportBecomeStableSeconds, BLINK_FROM_HERE); 4096 kMostlyFillViewportBecomeStableSeconds, BLINK_FROM_HERE);
4099 } 4097 }
4100 4098
4101 void HTMLMediaElement::viewportFillDebouncerTimerFired(TimerBase*) { 4099 void HTMLMediaElement::viewportFillDebouncerTimerFired(TimerBase*) {
4102 m_mostlyFillingViewport = true; 4100 m_mostlyFillingViewport = true;
4103 if (m_webMediaPlayer) 4101 if (m_webMediaPlayer)
4104 m_webMediaPlayer->becameDominantVisibleContent(m_mostlyFillingViewport); 4102 m_webMediaPlayer->becameDominantVisibleContent(m_mostlyFillingViewport);
4105 } 4103 }
4106 4104
4107 } // namespace blink 4105 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698