DescriptionRevert 210433 "Revert 210423 "base: Make SequencedWorkerPool iss..."
Undoing speculative revert. Still failed after revert.
> Revert 210423 "base: Make SequencedWorkerPool issue globally uni..."
>
> Speculative revert for failing PageCyclerTest.FailProvisionalLoads on
> linux_clang.
>
> > base: Make SequencedWorkerPool issue globally unique SequenceTokens.
> >
> > SequencedWorkerPool currently issues SequenceTokens out of an internal member counter. This means that two different SequencedWorkerPool instances can issue identical SequenceTokens, which mucks up any attempt to distinguish sequences using only SequenceTokens.
> >
> > This change makes the SequenceTokens issued from an StaticAtomicSequenceNumber, which is globally shared amongst all SequencedWorkerPools.
> >
> > This change also makes the SequencedWorkerPool included in the nacl_untrusted builds, as it is needed for SequenceChecker and WeakPtr to work correctly. It previously was excluded because it used base/metrics. I've #ifdefed the base/metrics usage out for nacl.
> >
> > This issue is a spinoff and pre-requisite of issue 18501008: Make WeakPtr use SequenceChecker instead of ThreadChecker.
> >
> > R=akalin,darin
> > BUG=165590
> >
> > Review URL: https://chromiumcodereview.appspot.com/18650006
>
> TBR=tommycli@chromium.org
>
> Review URL: https://codereview.chromium.org/18271011
TBR=scottmg@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=210434
Patch Set 1 #
Messages
Total messages: 2 (0 generated)
|