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

Issue 10836255: Revert 124798 - MediaSource should use it's own list of source URLs (Closed)

Created:
8 years, 4 months ago by annacc
Modified:
8 years, 4 months ago
Reviewers:
annacc
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1229/
Visibility:
Public.

Description

Revert 124798 - MediaSource should use it's own list of source URLs https://bugs.webkit.org/show_bug.cgi?id=93285 Reviewed by Eric Carlson. r124780 added MediaSource objects to PublicURLManager's streamURLs list, piggy-backing on MEDIA_STREAM. This patch creates a sourceURLs list so that MediaSource can stand on its own. Test: builds on Andriod even when MEDIA_STREAM is disabled. * html/DOMURL.cpp: (WebCore::DOMURL::createObjectURL): * html/PublicURLManager.h: (WebCore::PublicURLManager::contextDestroyed): (PublicURLManager): (WebCore::PublicURLManager::sourceURLs): TBR=annacc@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=125645

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -17 lines) Patch
M Source/WebCore/html/DOMURL.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/html/PublicURLManager.h View 2 chunks +0 lines, -16 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
annacc
8 years, 4 months ago (2012-08-15 04:20:32 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698