New issue
Advanced search Search tips

Issue 887054 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 3
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 821009



Sign in to add a comment

Migrate content::PpapiPluginProcessHost::PluginNetworkObserver to NetworkConnectionTracker

Project Member Reported by rmcelrath@chromium.org, Sep 19

Issue description

content::PpapiPluginProcessHost::PluginNetworkObserver currently uses net::NetworkChangeNotifier to be notified of network connection changes. It should be converted to use NetworkConnectionTracker, which will work when the network service is enabled.
 
Owner: rmcelrath@chromium.org
Status: Started (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Oct 3

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

commit 5c8bbe3d3ea20ca3db9d40a5b528e42a14ca7722
Author: Robbie McElrath <rmcelrath@chromium.org>
Date: Wed Oct 03 00:04:25 2018

Migrate content::PpapiPluginProcessHost::PluginNetworkObserver to NetworkConnectionTracker

This migrates content::PpapiPluginProcessHost::PluginNetworkObserver
from using NetworkChangeNotifier to NetworkConnectionTracker, which is
backed by the network service.

Bug:  887054 
Change-Id: Ieee103c35dd2adde840ea21890c03fe4f0cc76ed
Reviewed-on: https://chromium-review.googlesource.com/c/1238446
Commit-Queue: Robbie McElrath <rmcelrath@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596056}
[modify] https://crrev.com/5c8bbe3d3ea20ca3db9d40a5b528e42a14ca7722/content/browser/ppapi_plugin_process_host.cc

Status: Fixed (was: Started)

Sign in to add a comment