Reland of [stubs] KeyedStoreGeneric: inline dictionary property stores

Reland^2 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/9be747666ca0eb6b9fad81425c311d76b405e27a
Cr-Commit-Position: refs/heads/master@{#41272}

[Jakob Kummerow, 2016-11-24 14:52:41]

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

[Jakob Kummerow, 2016-11-24 14:54:17]

Description was changed from

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

Reason for revert:
Will fix and reland

Original issue's description:
> Revert of [stubs] KeyedStoreGeneric: inline dictionary property stores
(patchset #2 id:10002 of https://codereview.chromium.org/2524943002/ )
> 
> Reason for revert:
> Blocks current roll:
> https://codereview.chromium.org/2526753003/
> 
> Bisect results:
> https://codereview.chromium.org/2531483002
> 
> Original issue's description:
> > 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}
> 
> TBR=ishell@chromium.org,machenbach@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/474bbec73dacfca83582a99a4b558af4f23db48f
> Cr-Commit-Position: refs/heads/master@{#41251}

TBR=ishell@chromium.org,machenbach@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^2 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-24 15:17:21]

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

[Jakob Kummerow, 2016-11-24 15:17:21]

PTAL. The diff from patch set 1 -> 2 is the only change you haven't seen before.

[Jakob Kummerow, 2016-11-24 15:23:51]

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

[commit-bot: I haz the power, 2016-11-24 15:23:55]

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

[Igor Sheludko, 2016-11-24 15:24:23]

lgtm

[commit-bot: I haz the power, 2016-11-24 15:59:24]

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

[commit-bot: I haz the power, 2016-11-24 15:59:25]

Dry run: This issue passed the CQ dry run.

[Jakob Kummerow, 2016-11-24 16:20:50]

The CQ bit was checked by jkummerow@chromium.org

[commit-bot: I haz the power, 2016-11-24 16:20:58]

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-24 16:22:54]

CQ is committing da patch.

Bot data: {"patchset_id": 210001, "attempt_start_ts": 1480004450449780,
"parent_rev": "b519f9b6523929eeacc6e05b1d26a4c8d2b56ea4", "commit_rev":
"df884a6751a01d18cfafcc5220116cd0d4159c89"}

[commit-bot: I haz the power, 2016-11-24 16:22:57]

Description was changed from

==========
Reland^2 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^2 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-24 16:22:58]

Committed patchset #3 (id:210001)

[commit-bot: I haz the power, 2016-11-24 16:23:18]

Description was changed from

==========
Reland^2 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^2 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/9be747666ca0eb6b9fad81425c311d76b405e27a
Cr-Commit-Position: refs/heads/master@{#41272}
==========

[commit-bot: I haz the power, 2016-11-24 16:23:19]

Patchset 3 (id:??) landed as
https://crrev.com/9be747666ca0eb6b9fad81425c311d76b405e27a
Cr-Commit-Position: refs/heads/master@{#41272}