context.addHitRegion should return if the DOMException occurs.

context.addHitRegion should return if the DOMException occurs.

It may randomly make a crash or an unknown error.

BUG=387561
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176795

[zino, 2014-06-23 09:35:18]

I'm not sure that this CL is related to crbug.com/387561.
Because I don't have the permission to access the report and the test case.
But if there was this CL, crash may occur.
(test codes : http://jsfiddle.net/romandev/zwu2D/2/)
Also, I think we don't need the layout tests in this CL.
Because the exception for clipping region is a temporary expedient.

Please take a look.

[zino, 2014-06-23 09:37:05]

On 2014/06/23 09:35:18, zino wrote:
> I'm not sure that this CL is related to crbug.com/387561.
> Because I don't have the permission to access the report and the test case.
> But if there was this CL, crash may occur.
> (test codes : http://jsfiddle.net/romandev/zwu2D/2/)
> Also, I think we don't need the layout tests in this CL.
> Because the exception for clipping region is a temporary expedient.
> 
> Please take a look.

Sorry, But if there wasn't this CL, crash may occur.

Thank you :-)

[Justin Novosad, 2014-06-23 14:16:42]

On 2014/06/23 09:37:05, zino wrote:
> On 2014/06/23 09:35:18, zino wrote:
> > I'm not sure that this CL is related to crbug.com/387561.
> > Because I don't have the permission to access the report and the test case.
> > But if there was this CL, crash may occur.
> > (test codes : http://jsfiddle.net/romandev/zwu2D/2/)
> > Also, I think we don't need the layout tests in this CL.
> > Because the exception for clipping region is a temporary expedient.
> > 
> > Please take a look.
> 
> Sorry, But if there wasn't this CL, crash may occur.
> 
> Thank you :-)

lgtm

[zino, 2014-06-23 17:23:20]

The CQ bit was checked by jinho.bang@samsung.com

[commit-bot: I haz the power, 2014-06-23 17:24:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jinho.bang@samsung.com/331803004/1

[commit-bot: I haz the power, 2014-06-23 18:27:07]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/13163)

[commit-bot: I haz the power, 2014-06-23 19:58:25]

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

[commit-bot: I haz the power, 2014-06-23 19:58:27]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/13163)

[zino, 2014-06-23 23:55:31]

The CQ bit was checked by jinho.bang@samsung.com

[commit-bot: I haz the power, 2014-06-23 23:56:25]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jinho.bang@samsung.com/331803004/1

[commit-bot: I haz the power, 2014-06-23 23:57:12]

Change committed as 176795