Chromium Code Reviews
DescriptionRevert of Reland: Initialize and reset V4LocalDBManager. Instantiate V4Stores. (patchset #6 id:100001 of https://codereview.chromium.org/1983723002/ )
Reason for revert:
Speculative revert for http://crbug.com/612523; will re-land if that builder still fails the same tests after the revert
Original issue's description:
> Reland: Initialize and reset V4LocalDBManager. Instantiate V4Stores.
> A rough sketch of how this works is here: https://goto.google.com/pver4-LoadUpdateDatabase-doc
>
> This relands https://codereview.chromium.org/1954393002 which was reverted by https://codereview.chromium.org/1983603002
>
> Reason for failure: StopOnIOThread ran first and reset v4_update_protocol_manager_, which we were later tried to dereference in DatabaseReady()
>
> --
> Original commit: http://crrev.com/32b9468ab5177534cb155b2d222611a6bca16c98
> Which was reverted as: http://crrev.com/517133dfc95a9ca2b599bfe7452205e5b5d81fd4
>
> This CL reverts commit http://crrev.com/517133dfc95a9ca2b599bfe7452205e5b5d81fd4
> --
>
> BUG=543161
>
> Committed: https://crrev.com/26771182e06d28496aca4c73e095d3c9a43713ce
> Cr-Commit-Position: refs/heads/master@{#393989}
TBR=shess@chromium.org,nparker@chromium.org,vakh@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=543161
Committed: https://crrev.com/ed2c1c1767e9eb657d804b32088ba141502b645e
Cr-Commit-Position: refs/heads/master@{#394227}
Patch Set 1 #Messages
Total messages: 6 (1 generated)
|