Let dirty rect inflate after its unite with the affected area

Let dirty rect inflate after its unite with the affected area

In our previous CL: https://codereview.chromium.org/1983863002/, we let
the dirty rect inflate by one pixel. However, this makes the dirty rect
keeps growing. Instead, we should inflate the dirty rect after computing
its intersection with the rect, which is the actual affected area.

BUG=609820
TBR=junov@chromium.org

Committed: https://crrev.com/0e9fa080a3438b3e4e8f2c525eaa43fff1393125
Cr-Commit-Position: refs/heads/master@{#394429}

[xidachen, 2016-05-18 13:47:55]

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

[commit-bot: I haz the power, 2016-05-18 13:48:04]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1994613002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1994613002/1

[xidachen, 2016-05-18 15:15:05]

Description was changed from

==========
Let dirty rect inflate after its unite with the affected area

In our previous CL: https://codereview.chromium.org/1983863002/, we let
the dirty rect inflate by one pixel. However, this makes the dirty rect
keeps growing. Instead, we should inflate the dirty rect after computing
its intersection with the rect, which is the actual affected area.

BUG=609820
==========

to

==========
Let dirty rect inflate after its unite with the affected area

In our previous CL: https://codereview.chromium.org/1983863002/, we let
the dirty rect inflate by one pixel. However, this makes the dirty rect
keeps growing. Instead, we should inflate the dirty rect after computing
its intersection with the rect, which is the actual affected area.

BUG=609820
TBR=junov@chromium.org
==========

[xidachen, 2016-05-18 15:25:03]

The CQ bit was unchecked by xidachen@chromium.org

[xidachen, 2016-05-18 15:35:23]

The CQ bit was checked by xidachen@chromium.org

[commit-bot: I haz the power, 2016-05-18 15:35:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1994613002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1994613002/1

[commit-bot: I haz the power, 2016-05-18 15:39:24]

Description was changed from

==========
Let dirty rect inflate after its unite with the affected area

In our previous CL: https://codereview.chromium.org/1983863002/, we let
the dirty rect inflate by one pixel. However, this makes the dirty rect
keeps growing. Instead, we should inflate the dirty rect after computing
its intersection with the rect, which is the actual affected area.

BUG=609820
TBR=junov@chromium.org
==========

to

==========
Let dirty rect inflate after its unite with the affected area

In our previous CL: https://codereview.chromium.org/1983863002/, we let
the dirty rect inflate by one pixel. However, this makes the dirty rect
keeps growing. Instead, we should inflate the dirty rect after computing
its intersection with the rect, which is the actual affected area.

BUG=609820
TBR=junov@chromium.org
==========

[commit-bot: I haz the power, 2016-05-18 15:39:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-18 15:41:02]

Description was changed from

==========
Let dirty rect inflate after its unite with the affected area

In our previous CL: https://codereview.chromium.org/1983863002/, we let
the dirty rect inflate by one pixel. However, this makes the dirty rect
keeps growing. Instead, we should inflate the dirty rect after computing
its intersection with the rect, which is the actual affected area.

BUG=609820
TBR=junov@chromium.org
==========

to

==========
Let dirty rect inflate after its unite with the affected area

In our previous CL: https://codereview.chromium.org/1983863002/, we let
the dirty rect inflate by one pixel. However, this makes the dirty rect
keeps growing. Instead, we should inflate the dirty rect after computing
its intersection with the rect, which is the actual affected area.

BUG=609820
TBR=junov@chromium.org

Committed: https://crrev.com/0e9fa080a3438b3e4e8f2c525eaa43fff1393125
Cr-Commit-Position: refs/heads/master@{#394429}
==========

[commit-bot: I haz the power, 2016-05-18 15:41:03]

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

[Justin Novosad, 2016-05-18 19:11:41]

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

[Justin Novosad, 2016-05-18 19:11:42]

https://codereview.chromium.org/1994613002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/html/HTMLCanvasElement.cpp (right):

https://codereview.chromium.org/1994613002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/html/HTMLCanvasElement.cpp:274:
m_dirtyRect.inflate(1);
what you meant to do is inflate "rect" before the union. As opposed to inflate
m_dirtyRect, which contains an accumulation