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

Unified Diff: extensions/renderer/api/display_source/wifi_display/wifi_display_transport_stream_packetizer.cc

Issue 2918113004: Replace deprecated base::NonThreadSafe in /extensions/renderer/api/display_source in favor of Threa… (Closed)
Patch Set: Use ThreadChecker to be safe for now... Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « extensions/renderer/api/display_source/wifi_display/wifi_display_transport_stream_packetizer.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/renderer/api/display_source/wifi_display/wifi_display_transport_stream_packetizer.cc
diff --git a/extensions/renderer/api/display_source/wifi_display/wifi_display_transport_stream_packetizer.cc b/extensions/renderer/api/display_source/wifi_display/wifi_display_transport_stream_packetizer.cc
index 22e1bcfa9a5375fc4fe412473647533a8ec6b6c7..824719f91f9acc560b6efd04a628a3fc42a64ddf 100644
--- a/extensions/renderer/api/display_source/wifi_display/wifi_display_transport_stream_packetizer.cc
+++ b/extensions/renderer/api/display_source/wifi_display/wifi_display_transport_stream_packetizer.cc
@@ -434,7 +434,9 @@ WiFiDisplayTransportStreamPacketizer::WiFiDisplayTransportStreamPacketizer(
CHECK(SetElementaryStreams(stream_infos));
}
-WiFiDisplayTransportStreamPacketizer::~WiFiDisplayTransportStreamPacketizer() {}
+WiFiDisplayTransportStreamPacketizer::~WiFiDisplayTransportStreamPacketizer() {
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
+}
bool WiFiDisplayTransportStreamPacketizer::EncodeElementaryStreamUnit(
unsigned stream_index,
@@ -444,7 +446,7 @@ bool WiFiDisplayTransportStreamPacketizer::EncodeElementaryStreamUnit(
base::TimeTicks pts,
base::TimeTicks dts,
bool flush) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
DCHECK_LT(stream_index, stream_states_.size());
ElementaryStreamState& stream_state = stream_states_[stream_index];
@@ -549,7 +551,7 @@ bool WiFiDisplayTransportStreamPacketizer::EncodeElementaryStreamUnit(
}
bool WiFiDisplayTransportStreamPacketizer::EncodeMetaInformation(bool flush) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
return (EncodeProgramAssociationTable(false) &&
EncodeProgramMapTables(false) && EncodeProgramClockReference(flush));
@@ -557,7 +559,7 @@ bool WiFiDisplayTransportStreamPacketizer::EncodeMetaInformation(bool flush) {
bool WiFiDisplayTransportStreamPacketizer::EncodeProgramAssociationTable(
bool flush) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
// Fill in a packet header.
uint8_t header_data[ts::kPacketHeaderSize];
@@ -576,7 +578,7 @@ bool WiFiDisplayTransportStreamPacketizer::EncodeProgramAssociationTable(
bool WiFiDisplayTransportStreamPacketizer::EncodeProgramClockReference(
bool flush) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
program_clock_reference_ = base::TimeTicks::Now();
@@ -607,7 +609,7 @@ bool WiFiDisplayTransportStreamPacketizer::EncodeProgramClockReference(
}
bool WiFiDisplayTransportStreamPacketizer::EncodeProgramMapTables(bool flush) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
DCHECK(!program_map_table_.empty());
// Fill in a packet header.
@@ -633,7 +635,7 @@ void WiFiDisplayTransportStreamPacketizer::
void WiFiDisplayTransportStreamPacketizer::NormalizeUnitTimeStamps(
base::TimeTicks* pts,
base::TimeTicks* dts) const {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
// Normalize a presentation time stamp.
if (!pts || pts->is_null())
@@ -651,7 +653,7 @@ void WiFiDisplayTransportStreamPacketizer::NormalizeUnitTimeStamps(
bool WiFiDisplayTransportStreamPacketizer::SetElementaryStreams(
const std::vector<WiFiDisplayElementaryStreamInfo>& stream_infos) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
std::vector<ElementaryStreamState> new_stream_states;
new_stream_states.reserve(stream_infos.size());
@@ -717,7 +719,7 @@ bool WiFiDisplayTransportStreamPacketizer::SetElementaryStreams(
void WiFiDisplayTransportStreamPacketizer::UpdateDelayForUnitTimeStamps(
const base::TimeTicks& pts,
const base::TimeTicks& dts) {
- DCHECK(CalledOnValidThread());
+ DCHECK_CALLED_ON_VALID_THREAD(thread_checker_);
if (pts.is_null())
return;
« no previous file with comments | « extensions/renderer/api/display_source/wifi_display/wifi_display_transport_stream_packetizer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698