Make AttachBehavior a required argument.

Make AttachBehavior a required argument.

This way we have a clear list of DeprecatedAttachNow calls to work through.
This required making ExceptionState a required argument as well.

In a future patch, I'll make AttachBehavior default to AttachLazily and ExceptionState default to ASSERT_NO_EXCEPTION.

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

[esprehn, 2013-08-21 20:11:35]

LGTM, lots of these can actually be AttachLazily, I had just skipped over ones
on detached trees where I knew it didn't matter in my previous patches.

[ojan, 2013-08-21 20:24:11]

On 2013/08/21 20:11:35, esprehn wrote:
> LGTM, lots of these can actually be AttachLazily, I had just skipped over ones
> on detached trees where I knew it didn't matter in my previous patches.

Yup. I didn't want to make any behavior changes in this patch.

[commit-bot: I haz the power, 2013-08-21 20:24:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ojan@chromium.org/22880029/1

[eseidel, 2013-08-21 20:56:02]

Fantastic!

[commit-bot: I haz the power, 2013-08-21 23:02:42]

Change committed as 156514