DescriptionFix content verifier problem with content scripts (reland)
The problem is that we were creating the ContentVerifier too late
in ExtensionSystemImpl::Shared::Init, after the script loader had
already tried grabbing a reference to it for use on another
thead. Therefore at run time, for all content scripts we were was
seeing a null pointer for the content verifier and skipping
verification of those scripts.
Note: this is a reland of https://codereview.chromium.org/1226163010/
which was reverted in b9e8b50e4c45d8aefacac58e28c50d83bb257b7b due
to test failures on the Vista Tests bot.
BUG=509130
Committed: https://crrev.com/678123d286ab15d26064122c33f88a9d43673392
Cr-Commit-Position: refs/heads/master@{#341435}
Patch Set 1 : Original version of CL #Patch Set 2 : added extra logging to perhaps get more info from try servers on failures #Patch Set 3 : fix compile errors #Patch Set 4 : removed logging and reworked test to elimnate flakiness #
Messages
Total messages: 8 (3 generated)
|