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

Issue 647057 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

[remoting host] Building remoting_me2me_host seems to pull in a lot of WebKit.

Project Member Reported by lambroslambrou@chromium.org, Sep 14 2016

Issue description

Tried building remoting_me2me_host on Linux and noticed that it spent a lot of time compiling WebKit code. The host process shouldn't need anything from WebKit.

The dependency-chain (from gn path) looks like this:

//remoting/host:remoting_me2me_host --[private]-->
//remoting/host:remoting_me2me_host_static --[private]-->
//remoting/host:host --[private]-->
//content/public/common:common --[public]-->
//content:content --[public]-->
//content/ppapi_plugin:ppapi_plugin_sources --[private]-->
//third_party/WebKit/public:blink --[private]-->
//third_party/WebKit/Source/web:web --[private]-->
//third_party/WebKit/Source/core:core

 
Looks like revision 546e9c2b6c57eff413ed939baa2e5af30afed0b7 introduced a dependency on //content/public/common

Cc: dpranke@chromium.org
From talking with Scott, this issue seems to be that both remoting/ and content/ define ParamTraits<net::IPAddress>, and somehow content/ was being pulled in by the Android build, leading to duplicate definitions.

Ideally we should track down that dependency and remove it, but that still leaves us with two definitions for ParamTraits<net::IPAddress>. I wonder if it makes sense to move that definition out of content; perhaps in net?

Dirk, I believe you and Scott discussed this briefly; WDYT?
Owner: nicho...@chromium.org
Status: Assigned (was: Untriaged)
I don't recall discussing this, but I agree that moving the definition into net seems at least plausible.
Status: Fixed (was: Assigned)
Fixed in https://codereview.chromium.org/2365213002

Sign in to add a comment