Revert of HttpCache::Transaction layer allowing parallel validation (patchset #33 id:800001 of http…

Revert of HttpCache::Transaction layer allowing parallel validation (patchset #33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest: https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-chrome-pfq-informational/builds/4271

TBR=asanka@chromium.org,jkarlin@chromium.org
BUG=715913
Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when another
> transaction is the active reader/writer. After the validation phase, if its a match
> the transaction might wait till the response is written to the cache by the active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed: https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757849f76e8a808

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740
Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)

Review-Url: https://codereview.chromium.org/2850613002
Cr-Commit-Position: refs/branch-heads/3082@{#3}
Cr-Branched-From: 190ee07cecf53e3795197cc195b844f7cc15a9ea-refs/heads/master@{#467534}
Committed: https://chromium.googlesource.com/chromium/src/+/0e97dc17b154f15e5566ac5079548402ab02e238

[shivanisha, 2017-04-27 19:37:07]

Description was changed from

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

to

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=rdsmith@chromium.org

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

[shivanisha, 2017-04-27 19:41:17]

Description was changed from

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=rdsmith@chromium.org

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

to

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=asanka@chromium.org

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

[shivanisha, 2017-04-27 19:41:50]

Description was changed from

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=asanka@chromium.org

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

to

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=asanka@chromium.org,jkarlin@chromium.org

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

[shivanisha, 2017-04-27 19:42:03]

Merging into branch 3082

[shivanisha, 2017-04-27 19:42:06]

The CQ bit was checked by shivanisha@chromium.org

[commit-bot: I haz the power, 2017-04-27 19:42:48]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[shivanisha, 2017-04-27 19:43:21]

shivanisha@chromium.org changed reviewers:
- shivanisha@chromium.org

[shivanisha, 2017-04-27 19:46:45]

Description was changed from

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=asanka@chromium.org,jkarlin@chromium.org

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

to

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=asanka@chromium.org,jkarlin@chromium.org
BUG=715913

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

[commit-bot: I haz the power, 2017-04-27 20:03:48]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493322126809530, "parent_rev":
"c539c707c7536d21c017009634f63e1e53953426", "commit_rev":
"0e97dc17b154f15e5566ac5079548402ab02e238"}

[commit-bot: I haz the power, 2017-04-27 20:04:00]

Description was changed from

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=asanka@chromium.org,jkarlin@chromium.org
BUG=715913

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)
==========

to

==========
Revert of HttpCache::Transaction layer allowing parallel validation (patchset
#33 id:800001 of https://codereview.chromium.org/2721933002/ )

Reason for revert:
Breaks tricky-tot-chrome-pfq-informational audio_CrasSanity autotest:
https://uberchromegw.corp.google.com/i/chromeos.chrome/builders/tricky-tot-ch...

TBR=asanka@chromium.org,jkarlin@chromium.org
BUG=715913

Original issue's description:
> This CL is a precursor to allowing shared writing to fix cache lock.
>
> This CL allows transactions to continue to their validation phase even when
another
> transaction is the active reader/writer. After the validation phase, if its a
match
> the transaction might wait till the response is written to the cache by the
active
> writer. If its not a match the transaction will doom the entry and go to the
> network. In a subsequent CL, the not matching case will create a new entry as
well.
>
> BUG=472740
>
> Review-Url: https://codereview.chromium.org/2721933002
> Cr-Commit-Position: refs/heads/master@{#467426}
> Committed:
https://chromium.googlesource.com/chromium/src/+/1e2e347f957ef889aaee527bb757...

TBR=asanka@chromium.org,jkarlin@chromium.org,rdsmith@chromium.org,shivanisha@...
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=472740

Review-Url: https://codereview.chromium.org/2847653002
Cr-Commit-Position: refs/heads/master@{#467556}
(cherry picked from commit ae7fa093c74907efb6c2788208ac05495e960936)

Review-Url: https://codereview.chromium.org/2850613002
Cr-Commit-Position: refs/branch-heads/3082@{#3}
Cr-Branched-From:
190ee07cecf53e3795197cc195b844f7cc15a9ea-refs/heads/master@{#467534}
Committed:
https://chromium.googlesource.com/chromium/src/+/0e97dc17b154f15e5566ac507954...
==========

[commit-bot: I haz the power, 2017-04-27 20:04:01]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/0e97dc17b154f15e5566ac507954...