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

Issue 10982093: Merge 129397 - MediaStream API: Fix the incorrectly spelled RTCPeerConnection::onnegotiationneeded … (Closed)

Created:
8 years, 2 months ago by karen
Modified:
8 years, 2 months ago
Reviewers:
tommyw
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1271/
Visibility:
Public.

Description

Merge 129397 - MediaStream API: Fix the incorrectly spelled RTCPeerConnection::onnegotiationneeded callback https://bugs.webkit.org/show_bug.cgi?id=97456 Reviewed by Adam Barth. Renamed onnegotationneeded to onnegotiationneeded. Source/WebCore: Existing test updated. * Modules/mediastream/RTCPeerConnection.cpp: (WebCore::RTCPeerConnection::negotiationNeeded): * Modules/mediastream/RTCPeerConnection.h: (RTCPeerConnection): * Modules/mediastream/RTCPeerConnection.idl: * dom/EventNames.h: (WebCore): LayoutTests: * fast/mediastream/RTCPeerConnection-onnegotiationneeded-expected.txt: * fast/mediastream/RTCPeerConnection-onnegotiationneeded.html: TBR=tommyw@google.com Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=129926

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M LayoutTests/fast/mediastream/RTCPeerConnection-onnegotiationneeded.html View 2 chunks +3 lines, -3 lines 0 comments Download
M LayoutTests/fast/mediastream/RTCPeerConnection-onnegotiationneeded-expected.txt View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/Modules/mediastream/RTCPeerConnection.h View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/Modules/mediastream/RTCPeerConnection.idl View 1 chunk +1 line, -1 line 0 comments Download
M Source/WebCore/dom/EventNames.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
karen
8 years, 2 months ago (2012-09-28 18:27:34 UTC) #1
tommyw
8 years, 2 months ago (2012-10-01 07:39:01 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698