[webkit-reviews] review granted: [Bug 227621] [Model] Allow disabling interaction : [Attachment 450122] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Antoine Quint <graouts at webkit.org>'s
request for review:
Bug 227621: [Model] Allow disabling interaction
https://bugs.webkit.org/show_bug.cgi?id=227621

Attachment 450122: Patch

https://bugs.webkit.org/attachment.cgi?id=450122&action=review




--- Comment #5 from Darin Adler <darin at apple.com> ---
Comment on attachment 450122
  --> https://bugs.webkit.org/attachment.cgi?id=450122
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=450122&action=review

> Source/WebCore/Modules/model-element/HTMLModelElement.cpp:347
> +void HTMLModelElement::attributeChanged(const QualifiedName& attributeName,
const AtomString& oldValue, const AtomString& newValue,
AttributeModificationReason modificationReason)

Would give these arguments single-word names, "name" and "reason", unless they
seem ambiguous in this context.