New issue
Advanced search Search tips

Issue 793273 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

RTCPeerConnectionHandler::DestructAllHandlers is unused

Project Member Reported by eladalon@chromium.org, Dec 8 2017

Issue description

Dead code; should probably be removed.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e8c9fc291ff16bfa54f4ba1da82694800d85df44

commit e8c9fc291ff16bfa54f4ba1da82694800d85df44
Author: Elad Alon <eladalon@chromium.org>
Date: Mon Dec 11 12:41:08 2017

Remove RTCPeerConnectionHandler::DestructAllHandlers

The function is not used; removing it and its unit-test.

Bug:  793273 
Change-Id: I9419ce7c25e0d319dd10ae5d2014d7c3da517c3d
Reviewed-on: https://chromium-review.googlesource.com/817117
Reviewed-by: Henrik Boström <hbos@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Elad Alon <eladalon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523081}
[modify] https://crrev.com/e8c9fc291ff16bfa54f4ba1da82694800d85df44/content/renderer/media/rtc_peer_connection_handler.cc
[modify] https://crrev.com/e8c9fc291ff16bfa54f4ba1da82694800d85df44/content/renderer/media/rtc_peer_connection_handler.h
[modify] https://crrev.com/e8c9fc291ff16bfa54f4ba1da82694800d85df44/content/renderer/media/rtc_peer_connection_handler_unittest.cc

Status: Verified (was: Started)

Sign in to add a comment