New issue
Advanced search Search tips

Issue 869748 link

Starred by 3 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task

Blocking:
issue 911415


Participants' hotlists:
ServiceWorkerOnionSoup


Sign in to add a comment

Use mojom struct for content::RendererPreferences and blink::PrivacyPreferences

Project Member Reported by shimazu@chromium.org, Aug 1

Issue description

We are now using IPC_STRUCT_TRAITS macros for content::RendererPreferences, but we can use mojom struct instead of that.
Along with that, we are having a subset of the preferences - blink::PrivacyPreferences. When we move content::RendererPreferences to mojom, we can move blink::PrivacyPreferences to mojom and replace some fields in content::RendererPreferences to blink::PrivacyPreferences.
 
Labels: -Type-Bug Type-Task
It's not a bug.

Comment 2 by leon....@intel.com, Yesterday (31 hours ago)

Status: Started (was: Available)

Comment 3 by leon....@intel.com, Yesterday (31 hours ago)

Owner: leon....@intel.com

Comment 4 by shimazu@chromium.org, Yesterday (30 hours ago)

Blocking: 911415

Comment 5 by staphany@chromium.org, Today (9 hours ago)

Am I correct in thinking this task will move content::RendererPreferences somewhere into third_party/blink? I'm also running into this issue while Onion Souping AppCache.

Comment 6 by shimazu@chromium.org, Today (8 hours ago)

Currently Leon is working on just replacing RendererPreference with mojom struct (https://crrev.com/c/1426522), but I believe he'll move it under blink/ later for Onion Souping EmbeddedWorkerInstanceClient.

Sign in to add a comment