[webkit-reviews] review granted: [Bug 131390] Support "Live" streams in media controls. : [Attachment 229154] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Apr 16 12:29:17 PDT 2014
Brent Fulgham <bfulgham at webkit.org> has granted Jer Noble
<jer.noble at apple.com>'s request for review:
Bug 131390: Support "Live" streams in media controls.
https://bugs.webkit.org/show_bug.cgi?id=131390
Attachment 229154: Patch
https://bugs.webkit.org/attachment.cgi?id=229154&action=review
------- Additional Comments from Brent Fulgham <bfulgham at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=229154&action=review
r=me, but I have some concerns about not seeing "Stalled..." and so forth
anymore!
> Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:486
> + this.controls.panel.appendChild(this.controls.statusDisplay);
Why don't we want the statusDisplay for fullscreen non-live controls case? It
looks like we do include it in the non-fullscreen case (see line 442)
> Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:-488
> - this.controls.statusDisplay.innerText = this.UIString('Aborted');
We don't seem to get this text in 'updateStatusDisplay"
> Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:-493
> - this.controls.statusDisplay.innerText = this.UIString('Suspended');
Ditto...
> Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:-498
> - this.controls.statusDisplay.innerText = this.UIString('Stalled');
Ditto...
> Source/WebCore/Modules/mediacontrols/mediaControlsApple.js:-504
> - this.controls.statusDisplay.innerText = this.UIString('Waiting');
Ditto...
More information about the webkit-reviews
mailing list