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

Side by Side Diff: media/blink/webmediaplayer_impl.h

Issue 1850733003: Media: Report informational error messages to HTMLMediaElement (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@use_locking_in_rendermedialog
Patch Set: Fix null MediaLogEvent* dereference in RenderMediaLog::AddEvent() Created 4 years, 8 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 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 5 #ifndef MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
6 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 6 #define MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <string> 10 #include <string>
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 double duration() const override; 136 double duration() const override;
137 virtual double timelineOffset() const; 137 virtual double timelineOffset() const;
138 double currentTime() const override; 138 double currentTime() const override;
139 139
140 // Internal states of loading and network. 140 // Internal states of loading and network.
141 // TODO(hclam): Ask the pipeline about the state rather than having reading 141 // TODO(hclam): Ask the pipeline about the state rather than having reading
142 // them from members which would cause race conditions. 142 // them from members which would cause race conditions.
143 blink::WebMediaPlayer::NetworkState getNetworkState() const override; 143 blink::WebMediaPlayer::NetworkState getNetworkState() const override;
144 blink::WebMediaPlayer::ReadyState getReadyState() const override; 144 blink::WebMediaPlayer::ReadyState getReadyState() const override;
145 145
146 blink::WebString getErrorMessage() override;
146 bool didLoadingProgress() override; 147 bool didLoadingProgress() override;
147 148
148 bool hasSingleSecurityOrigin() const override; 149 bool hasSingleSecurityOrigin() const override;
149 bool didPassCORSAccessCheck() const override; 150 bool didPassCORSAccessCheck() const override;
150 151
151 double mediaTimeForTimeValue(double timeValue) const override; 152 double mediaTimeForTimeValue(double timeValue) const override;
152 153
153 unsigned decodedFrameCount() const override; 154 unsigned decodedFrameCount() const override;
154 unsigned droppedFrameCount() const override; 155 unsigned droppedFrameCount() const override;
155 size_t audioDecodedByteCount() const override; 156 size_t audioDecodedByteCount() const override;
(...skipping 287 matching lines...) Expand 10 before | Expand all | Expand 10 after
443 // started; prevents us from spuriously logging errors that are transient or 444 // started; prevents us from spuriously logging errors that are transient or
444 // unimportant. 445 // unimportant.
445 bool suppress_destruction_errors_; 446 bool suppress_destruction_errors_;
446 447
447 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl); 448 DISALLOW_COPY_AND_ASSIGN(WebMediaPlayerImpl);
448 }; 449 };
449 450
450 } // namespace media 451 } // namespace media
451 452
452 #endif // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_ 453 #endif // MEDIA_BLINK_WEBMEDIAPLAYER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698