Changeset 155364 in webkit


Ignore:
Timestamp:
Sep 9, 2013 11:19:47 AM (11 years ago)
Author:
eric.carlson@apple.com
Message:

MediaStream API: Stop means stop
​https://bugs.webkit.org/show_bug.cgi?id=120873

Reviewed by Jer Noble.

Merge blink ​https://chromium.googlesource.com/chromium/blink/+/9a97b0f3892320fa5741485e06a78183b77c3635
by Tommy Widenflycht.

Removing notification functionality from MediaStreamTrack::stop because the world is being torn down.

No new tests, the original author was unable to come up with a reliable test case.

  • Modules/mediastream/MediaStreamTrack.cpp:

(WebCore::MediaStreamTrack::stop): Don't call didEndTrack, the world is going away.

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r155363 r155364  
     12013-09-09  Eric Carlson  <eric.carlson@apple.com>
     2
     3        MediaStream API: Stop means stop
     4        https://bugs.webkit.org/show_bug.cgi?id=120873
     5
     6        Reviewed by Jer Noble.
     7
     8        Merge blink https://chromium.googlesource.com/chromium/blink/+/9a97b0f3892320fa5741485e06a78183b77c3635
     9        by Tommy Widenflycht.
     10
     11        Removing notification functionality from MediaStreamTrack::stop because the world is being torn down.
     12
     13        No new tests, the original author was unable to come up with a reliable test case.
     14
     15        * Modules/mediastream/MediaStreamTrack.cpp:
     16        (WebCore::MediaStreamTrack::stop): Don't call didEndTrack, the world is going away.
     17
    1182013-09-09  Filip Pizlo  <fpizlo@apple.com>
    219

  • trunk/Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp

    r155249 r155364  
    158158{
    159159    m_stopped = true;
    160     didEndTrack();
    161160}
    162161
Note: See TracChangeset for help on using the changeset viewer.