Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

For dictionary-mode receivers, the KeyedStoreGeneric stub can store
properties directly in most cases. Doing so avoids the need to have
an entry in the stub cache for every map/property combination.

Original review: https://codereview.chromium.org/2504403005/

Committed: https://crrev.com/7a963deb85a0cc04623947a759534c48e2871901
Cr-Commit-Position: refs/heads/master@{#41218}

[Jakob Kummerow, 2016-11-23 09:38:05]

Created Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

[Jakob Kummerow, 2016-11-23 09:39:58]

Description was changed from

==========
Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores (patchset
#1 id:1 of https://codereview.chromium.org/2528583002/ )

Reason for revert:
Will fix and reland

Original issue's description:
> Revert of [stubs] KeyedStoreGeneric: inline dictionary property stores
(patchset #5 id:100001 of https://codereview.chromium.org/2504403005/ )
> 
> Reason for revert:
> Blocks roll: https://codereview.chromium.org/2526573002/
> 
> Original issue's description:
> > [stubs] KeyedStoreGeneric: inline dictionary property stores
> >
> > For dictionary-mode receivers, the KeyedStoreGeneric stub can store
> > properties directly in most cases. Doing so avoids the need to have
> > an entry in the stub cache for every map/property combination.
> >
> > Committed: https://crrev.com/af168e330e95c4460fd1bb7734f0e9a750f2e748
> > Cr-Commit-Position: refs/heads/master@{#41185}
> 
> TBR=ishell@chromium.org,rmcilroy@chromium.org,jkummerow@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> 
> Committed: https://crrev.com/e3f5c515faa7ec90f044a765a5cd6b3e3be0d90d
> Cr-Commit-Position: refs/heads/master@{#41207}

TBR=ishell@chromium.org,rmcilroy@chromium.org,hablich@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

For dictionary-mode receivers, the KeyedStoreGeneric stub can store
properties directly in most cases. Doing so avoids the need to have
an entry in the stub cache for every map/property combination.

Original review: https://codereview.chromium.org/2504403005/
==========

[Jakob Kummerow, 2016-11-23 09:39:58]

jkummerow@chromium.org changed reviewers:
- hablich@chromium.org, rmcilroy@chromium.org

[Jakob Kummerow, 2016-11-23 11:02:53]

PTAL.

[Igor Sheludko, 2016-11-23 12:06:54]

lgtm

[Jakob Kummerow, 2016-11-23 12:12:57]

The CQ bit was checked by jkummerow@chromium.org

[commit-bot: I haz the power, 2016-11-23 12:13:03]

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, 2016-11-23 12:43:30]

CQ is committing da patch.

Bot data: {"patchset_id": 10002, "attempt_start_ts": 1479903177875100,
"parent_rev": "a7ed77566a6d646f7fc492e163b640b0088a6a61", "commit_rev":
"955ba70a202146171106892c4ad0e9f7f7885704"}

[commit-bot: I haz the power, 2016-11-23 12:43:33]

Description was changed from

==========
Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

For dictionary-mode receivers, the KeyedStoreGeneric stub can store
properties directly in most cases. Doing so avoids the need to have
an entry in the stub cache for every map/property combination.

Original review: https://codereview.chromium.org/2504403005/
==========

to

==========
Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

For dictionary-mode receivers, the KeyedStoreGeneric stub can store
properties directly in most cases. Doing so avoids the need to have
an entry in the stub cache for every map/property combination.

Original review: https://codereview.chromium.org/2504403005/
==========

[commit-bot: I haz the power, 2016-11-23 12:43:34]

Committed patchset #2 (id:10002)

[commit-bot: I haz the power, 2016-11-23 12:43:54]

Description was changed from

==========
Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

For dictionary-mode receivers, the KeyedStoreGeneric stub can store
properties directly in most cases. Doing so avoids the need to have
an entry in the stub cache for every map/property combination.

Original review: https://codereview.chromium.org/2504403005/
==========

to

==========
Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

For dictionary-mode receivers, the KeyedStoreGeneric stub can store
properties directly in most cases. Doing so avoids the need to have
an entry in the stub cache for every map/property combination.

Original review: https://codereview.chromium.org/2504403005/

Committed: https://crrev.com/7a963deb85a0cc04623947a759534c48e2871901
Cr-Commit-Position: refs/heads/master@{#41218}
==========

[commit-bot: I haz the power, 2016-11-23 12:43:54]

Patchset 2 (id:??) landed as
https://crrev.com/7a963deb85a0cc04623947a759534c48e2871901
Cr-Commit-Position: refs/heads/master@{#41218}

[Michael Achenbach, 2016-11-24 08:28:54]

machenbach@chromium.org changed reviewers:
+ machenbach@chromium.org

[Michael Achenbach, 2016-11-24 08:28:55]

Blocks current roll:
https://codereview.chromium.org/2526753003/

Bisect results:
https://codereview.chromium.org/2531483002

[Michael Hablich, 2016-11-24 08:30:55]

A revert of this CL (patchset #2 id:10002) has been created in
https://codereview.chromium.org/2522393002/ by hablich@chromium.org.

The reason for reverting is: Blocks current roll:
https://codereview.chromium.org/2526753003/

Bisect results:
https://codereview.chromium.org/2531483002
.