New issue
Advanced search Search tips

Issue 740202 link

Starred by 2 users
Status: Fixed
Owner:
btolsch@chromium.org
Closed: Aug 2017
Cc:
mfo...@chromium.org
btolsch@chromium.org
powerb@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Media Router network monitor should be in media_router namespace

Project Member Reported by btolsch@chromium.org, Jul 7 2017 
DiscoveryNetworkMonitor and related code should probably be in the media_router namespace since it's under //chrome/browser/media/router/discovery.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 18 2017 

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

commit f230125d53e3c22178fd2ebeaca8f9d599d6f7d4
Author: btolsch <btolsch@chromium.org>
Date: Fri Aug 18 22:09:12 2017

Put discovery network code in media_router namespace

Bug:  740202 
Change-Id: I55850de20379d2b12f9513ccff58910a12da53e1
Reviewed-on: https://chromium-review.googlesource.com/619113
Reviewed-by: Derek Cheng <imcheng@chromium.org>
Commit-Queue: Brandon Tolsch <btolsch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495712}
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_info.cc
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_info.h
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_list.h
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_list_posix.cc
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_list_unittest.cc
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_list_wifi.h
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_list_wifi_linux.cc
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_list_wifi_mac.mm
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_list_win.cc
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_monitor.cc
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_monitor.h
[modify] https://crrev.com/f230125d53e3c22178fd2ebeaca8f9d599d6f7d4/chrome/browser/media/router/discovery/discovery_network_monitor_unittest.cc

Comment 2 by btolsch@chromium.org, Aug 21 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment