[extensions] Add async initialization support for extensions [chromium/src : main]

0 views

Skip to first unread message

woxxom (Gerrit)

unread,

9:03 AM (6 hours ago) 9:03 AM

to Sven Zheng, Chromium Metrics Reviews, chromium...@chromium.org, Chromium LUCI CQ, asvitkine...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org, ipc-securi...@chromium.org

Attention needed from Sven Zheng

woxxom added 1 comment

File chrome/test/data/extensions/api_test/async_initialization/basic/manifest.json

Line 5, Patchset 1 (Latest): "async_initialization": true,

woxxom . unresolved

Judging by the code this is specific to the background context, so it might be better to move the value inside the `background` object to explicitly indicate the affected scope. BTW is this an evolution of https://crrev.com/c/7682242?

Open in Gerrit

Related details

Attention is currently required from:

Sven Zheng

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Reply all

Reply to author

Forward

0 new messages