DescriptionRevert of Fix content verifier problem with content scripts (patchset #3 id:40001 of https://codereview.chromium.org/1226163010/)
Reason for revert:
The ContentVerifierTest.ContentScripts test introduced in this change has been failing
consistently on Vista. See below
http://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/builds/57878
http://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%281%29/builds/57877
Original issue's description:
> Fix content verifier problem with content scripts
>
> 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.
>
> BUG=509130
>
> Committed: https://crrev.com/fe6dc62c549db502a0323ebbebab7ffd4f5cebd7
> Cr-Commit-Position: refs/heads/master@{#339283}
TBR=rockot@chromium.org,asargent@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=509130
Committed: https://crrev.com/b9e8b50e4c45d8aefacac58e28c50d83bb257b7b
Cr-Commit-Position: refs/heads/master@{#339361}
Patch Set 1 #
Created: 5 years, 5 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|