DevTools: revert Persistence2.0 experiment by default.

DevTools: revert Persistence2.0 experiment by default.

It turned out that Persistence2.0 doesn't account for the important
usecase: develoers were using workspace feature to override hosted
stylesheets.

In this case, both Persistence2.0 and "prevalidateBindigns"
experiments stay on their way:
- Persistence2.0 doesn't allow to map files with different filenames
- prevalidateBindings doesn't allow to map network resources to files
  if contents are different.

This patch:
- disables Persistence2.0 experiment by default
- merges prevalidateBindings experiment into Persistence2.0 experiment

The latter is important since binding prevalidation only necessary for
Persistence2.0 to work properly.

BUG=707002
R=dgozman

Review-Url: https://codereview.chromium.org/2818713007
Cr-Commit-Position: refs/heads/master@{#465412}
Committed: https://chromium.googlesource.com/chromium/src/+/87e05d141b5b873725e95585e92f07a6b7ac1fb2

[lushnikov, 2017-04-14 23:06:54]

Description was changed from

==========
DevTools: revert Persistence2.0 experiment by default.

Since Persistence2.0 doesn't account for a few important use cases, we
should not release it as a replacement for old prefix mappings.

BUG=707002
R=dgozman
==========

to

==========
DevTools: revert Persistence2.0 experiment by default.

It turned out that Persistence2.0 doesn't account for the important
usecase: develoers were using workspace feature to override hosted
stylesheets.

In this case, both Persistence2.0 and "prevalidateBindigns"
experiments stay on their way:
- Persistence2.0 doesn't allow to map files with different filenames
- prevalidateBindings doesn't allow to map network resources to files
  if contents are different.

This patch:
- disables Persistence2.0 experiment by default
- merges prevalidateBindings experiment into Persistence2.0 experiment

The latter is important since binding prevalidation only necessary for
Persistence2.0 to work properly.

BUG=707002
R=dgozman
==========

[lushnikov, 2017-04-14 23:07:26]

please, take a look

[lushnikov, 2017-04-14 23:09:21]

The CQ bit was checked by lushnikov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-14 23:09:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-15 00:46:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-15 00:46:44]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[dgozman, 2017-04-15 01:45:56]

lgtm

[lushnikov, 2017-04-18 20:51:57]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2017-04-18 20:52:40]

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

[commit-bot: I haz the power, 2017-04-18 23:03:22]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492548717875620, "parent_rev":
"22b1a6c7072edd302152cf05a2526030aea4ff6d", "commit_rev":
"87e05d141b5b873725e95585e92f07a6b7ac1fb2"}

[commit-bot: I haz the power, 2017-04-18 23:03:34]

Description was changed from

==========
DevTools: revert Persistence2.0 experiment by default.

It turned out that Persistence2.0 doesn't account for the important
usecase: develoers were using workspace feature to override hosted
stylesheets.

In this case, both Persistence2.0 and "prevalidateBindigns"
experiments stay on their way:
- Persistence2.0 doesn't allow to map files with different filenames
- prevalidateBindings doesn't allow to map network resources to files
  if contents are different.

This patch:
- disables Persistence2.0 experiment by default
- merges prevalidateBindings experiment into Persistence2.0 experiment

The latter is important since binding prevalidation only necessary for
Persistence2.0 to work properly.

BUG=707002
R=dgozman
==========

to

==========
DevTools: revert Persistence2.0 experiment by default.

It turned out that Persistence2.0 doesn't account for the important
usecase: develoers were using workspace feature to override hosted
stylesheets.

In this case, both Persistence2.0 and "prevalidateBindigns"
experiments stay on their way:
- Persistence2.0 doesn't allow to map files with different filenames
- prevalidateBindings doesn't allow to map network resources to files
  if contents are different.

This patch:
- disables Persistence2.0 experiment by default
- merges prevalidateBindings experiment into Persistence2.0 experiment

The latter is important since binding prevalidation only necessary for
Persistence2.0 to work properly.

BUG=707002
R=dgozman

Review-Url: https://codereview.chromium.org/2818713007
Cr-Commit-Position: refs/heads/master@{#465412}
Committed:
https://chromium.googlesource.com/chromium/src/+/87e05d141b5b873725e95585e92f...
==========

[commit-bot: I haz the power, 2017-04-18 23:03:35]

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