Remove some useless null checks in Element/NamedNodeMap

Remove some useless null checks in Element/NamedNodeMap
    
Make shouldIgnoreAttributeCase() a method of Element, thus avoiding
unnecessary element null checks.
    
This is based on WebKit r122767 by <akling@apple.com>;.
    
R=tkent

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

[eseidel, 2013-10-15 21:09:10]

Meh.

*this looks pretty ugly.  But I guess I'm just not used to references.

We're actually typing more now. :)  Since the one place had "e &&".  We could
have just made that ASSERT(e) or made this a private method on Element.

But this is also OK.

[adamk, 2013-10-15 21:15:52]

Yeah, I don't see much win here. The main thing that sticks out to me is the
duplication; could just have this as a method on Element.

[adamk, 2013-10-15 21:16:47]

Though I guess it's a kinda sad method. Maybe I'm just sad about NamedNodeMap :)

[Inactive, 2013-10-15 21:31:36]

On 2013/10/15 21:15:52, adamk wrote:
> Yeah, I don't see much win here. The main thing that sticks out to me is the
> duplication; could just have this as a method on Element.

Ok, I think making it a method of Element makes sense, at least to avoid the
duplication.

[Inactive, 2013-10-15 21:47:18]

On 2013/10/15 21:31:36, Chris Dumez wrote:
> On 2013/10/15 21:15:52, adamk wrote:
> > Yeah, I don't see much win here. The main thing that sticks out to me is the
> > duplication; could just have this as a method on Element.
> 
> Ok, I think making it a method of Element makes sense, at least to avoid the
> duplication.

Done.

[adamk, 2013-10-15 21:49:57]

lgtm

[commit-bot: I haz the power, 2013-10-16 13:19:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/26955005/9001

[commit-bot: I haz the power, 2013-10-16 17:39:05]

Change committed as 159789