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

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

Issue 57473002: Remove HTMLMediaElement::m_sendProgressEvents (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 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
« no previous file with comments | « Source/core/html/HTMLMediaElement.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved. 2 * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple 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
(...skipping 262 matching lines...) Expand 10 before | Expand all | Expand 10 after
273 , m_shouldDelayLoadEvent(false) 273 , m_shouldDelayLoadEvent(false)
274 , m_haveFiredLoadedData(false) 274 , m_haveFiredLoadedData(false)
275 , m_active(true) 275 , m_active(true)
276 , m_autoplaying(true) 276 , m_autoplaying(true)
277 , m_muted(false) 277 , m_muted(false)
278 , m_paused(true) 278 , m_paused(true)
279 , m_seeking(false) 279 , m_seeking(false)
280 , m_sentStalledEvent(false) 280 , m_sentStalledEvent(false)
281 , m_sentEndEvent(false) 281 , m_sentEndEvent(false)
282 , m_pausedInternal(false) 282 , m_pausedInternal(false)
283 , m_sendProgressEvents(true)
284 , m_closedCaptionsVisible(false) 283 , m_closedCaptionsVisible(false)
285 , m_loadInitiatedByUserGesture(false) 284 , m_loadInitiatedByUserGesture(false)
286 , m_completelyLoaded(false) 285 , m_completelyLoaded(false)
287 , m_havePreparedToPlay(false) 286 , m_havePreparedToPlay(false)
288 , m_parsingInProgress(createdByParser) 287 , m_parsingInProgress(createdByParser)
289 , m_tracksAreReady(true) 288 , m_tracksAreReady(true)
290 , m_haveVisibleTextTrack(false) 289 , m_haveVisibleTextTrack(false)
291 , m_processingPreferenceChange(false) 290 , m_processingPreferenceChange(false)
292 , m_lastTextTrackUpdateTime(-1) 291 , m_lastTextTrackUpdateTime(-1)
293 , m_textTracks(0) 292 , m_textTracks(0)
(...skipping 544 matching lines...) Expand 10 before | Expand all | Expand 10 after
838 837
839 // Set m_currentSrc *before* changing to the cache url, the fact that we are loading from the app 838 // Set m_currentSrc *before* changing to the cache url, the fact that we are loading from the app
840 // cache is an internal detail not exposed through the media element API. 839 // cache is an internal detail not exposed through the media element API.
841 m_currentSrc = url; 840 m_currentSrc = url;
842 841
843 LOG(Media, "HTMLMediaElement::loadResource - m_currentSrc -> %s", urlForLogg ingMedia(m_currentSrc).utf8().data()); 842 LOG(Media, "HTMLMediaElement::loadResource - m_currentSrc -> %s", urlForLogg ingMedia(m_currentSrc).utf8().data());
844 843
845 if (MediaStreamRegistry::registry().lookupMediaStreamDescriptor(url.string() )) 844 if (MediaStreamRegistry::registry().lookupMediaStreamDescriptor(url.string() ))
846 removeBehaviorRestriction(RequireUserGestureForRateChangeRestriction); 845 removeBehaviorRestriction(RequireUserGestureForRateChangeRestriction);
847 846
848 if (m_sendProgressEvents) 847 startProgressEventTimer();
849 startProgressEventTimer();
850 848
851 // Reset display mode to force a recalculation of what to show because we ar e resetting the player. 849 // Reset display mode to force a recalculation of what to show because we ar e resetting the player.
852 setDisplayMode(Unknown); 850 setDisplayMode(Unknown);
853 851
854 if (!autoplay()) 852 if (!autoplay())
855 m_player->setPreload(m_preload); 853 m_player->setPreload(m_preload);
856 854
857 if (fastHasAttribute(mutedAttr)) 855 if (fastHasAttribute(mutedAttr))
858 m_muted = true; 856 m_muted = true;
859 updateVolume(); 857 updateVolume();
(...skipping 3010 matching lines...) Expand 10 before | Expand all | Expand 10 after
3870 { 3868 {
3871 scheduleLayerUpdate(); 3869 scheduleLayerUpdate();
3872 } 3870 }
3873 3871
3874 bool HTMLMediaElement::isInteractiveContent() const 3872 bool HTMLMediaElement::isInteractiveContent() const
3875 { 3873 {
3876 return fastHasAttribute(controlsAttr); 3874 return fastHasAttribute(controlsAttr);
3877 } 3875 }
3878 3876
3879 } 3877 }
OLDNEW
« no previous file with comments | « Source/core/html/HTMLMediaElement.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698