Inline LayoutRect::enclosingIntRect

Inline LayoutRect::enclosingIntRect

Noticed this function was hot on blink_perf.paint's color-changes.html
microbenchmark. With this patch, color-changes.html runs 2.7% faster
(average -0.8ms) but that number should be taken with a grain of salt
since it's near the standard deviation of the test.

Manual profiling with and without the patch confirmed this function is
no longer hot.

Committed: https://crrev.com/af2c8d1e6a537c356b4cf931bb5aca1e5f09f119
Cr-Commit-Position: refs/heads/master@{#436323}

[pdr., 2016-12-05 04:28:45]

pdr@chromium.org changed reviewers:
+ wangxianzhu@chromium.org

[pdr., 2016-12-05 04:28:54]

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

[commit-bot: I haz the power, 2016-12-05 04:29:00]

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, 2016-12-05 06:10:21]

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

[commit-bot: I haz the power, 2016-12-05 06:10:22]

Dry run: This issue passed the CQ dry run.

[Xianzhu, 2016-12-05 16:44:44]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-12-05 16:44:44]

lgtm

[commit-bot: I haz the power, 2016-12-05 16:44:56]

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-12-05 16:48:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480956284129310, "parent_rev":
"829b7c2bde633923a29e86d23a0a800d759be632", "commit_rev":
"08e3b3e8b4c959aa57b6c90b6bb3459bf6fba7aa"}

[commit-bot: I haz the power, 2016-12-05 16:49:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-05 16:51:02]

Description was changed from

==========
Inline LayoutRect::enclosingIntRect

Noticed this function was hot on blink_perf.paint's color-changes.html
microbenchmark. With this patch, color-changes.html runs 2.7% faster
(average -0.8ms) but that number should be taken with a grain of salt
since it's near the standard deviation of the test.

Manual profiling with and without the patch confirmed this function is
no longer hot.
==========

to

==========
Inline LayoutRect::enclosingIntRect

Noticed this function was hot on blink_perf.paint's color-changes.html
microbenchmark. With this patch, color-changes.html runs 2.7% faster
(average -0.8ms) but that number should be taken with a grain of salt
since it's near the standard deviation of the test.

Manual profiling with and without the patch confirmed this function is
no longer hot.

Committed: https://crrev.com/af2c8d1e6a537c356b4cf931bb5aca1e5f09f119
Cr-Commit-Position: refs/heads/master@{#436323}
==========

[commit-bot: I haz the power, 2016-12-05 16:51:03]

Patchset 1 (id:??) landed as
https://crrev.com/af2c8d1e6a537c356b4cf931bb5aca1e5f09f119
Cr-Commit-Position: refs/heads/master@{#436323}