New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 751207 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Feature



Sign in to add a comment

[mDNS] Enable netlog for mDNS client

Project Member Reported by mfo...@chromium.org, Aug 1 2017

Issue description

This tracks the work to wire a netlog instance to the sockets bound by the mDNS client, to enable end users to provide more actionable information when mDNS doesn't seem to work (esp. to discover Cast devices).

Need to make sure this does not fill up the netlog with background discovery unless chrome://net-internals or some other observer is present.

 
Components: Internals>Network>DNS Internals>Cast
Labels: -Type-Bug Type-Feature
Labels: -OS-Android -OS-All OS-Chrome OS-Linux OS-Mac OS-Windows
Cc: imch...@chromium.org zhaobin@chromium.org

Comment 5 by sko...@chromium.org, Aug 14 2017

Labels: Hotlist-Fixit
Status: Available (was: Untriaged)
Cc: -imch...@chromium.org
Owner: imch...@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 12 2017

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

commit afd9e2271216549a52df558c469869e28d02b2f4
Author: Derek Cheng <imcheng@chromium.org>
Date: Tue Sep 12 19:31:57 2017

[mDNS] Add NetLog for mDNS / service_discovery_client_mdns.

Add option to pass in NetLog to socket used for mDNS.
Pass in ChromeNetLog from Chrome code service_discovery_client_mdns.

This will hopefully help us diagnose long standing Cast / mDNS
reliability issues.

The NetLog entries can be easily filtered by searching the IP endpoint
used for mDNS (defined in net/dns/mdns_client.cc?).

BUG= 751207 

Change-Id: Id649a03d498e077738c5e071214f31f6577401e6
Reviewed-on: https://chromium-review.googlesource.com/647490
Reviewed-by: Gene Gutnik <gene@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501360}
[modify] https://crrev.com/afd9e2271216549a52df558c469869e28d02b2f4/chrome/browser/local_discovery/service_discovery_client_mdns.cc
[modify] https://crrev.com/afd9e2271216549a52df558c469869e28d02b2f4/net/dns/mdns_client.cc
[modify] https://crrev.com/afd9e2271216549a52df558c469869e28d02b2f4/net/dns/mdns_client.h
[modify] https://crrev.com/afd9e2271216549a52df558c469869e28d02b2f4/net/dns/mdns_client_impl.cc

Status: Fixed (was: Assigned)

Sign in to add a comment