[Webkit-unassigned] [Bug 121221] RenderBR should not be RenderText

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=121221


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #211899|review?                     |review+
               Flag|                            |




--- Comment #35 from Darin Adler <darin at apple.com>  2013-09-17 07:54:24 PST ---
(From update of attachment 211899)
View in context: https://bugs.webkit.org/attachment.cgi?id=211899&action=review

Sorry, I later figured out I was wrong about OVERRIDE, but forgot to tell you.

> Source/WebCore/dom/Range.cpp:1651
> +        if (r->isBR()) {
> +            r->absoluteQuads(quads, &isFixed);
> +        } else if (r->isText()) {

Normally our code style says to omit the braces in a case like this.

> Source/WebCore/rendering/RenderBR.cpp:3
> + * Copyright (C) 2006, 2013 Apple Computer, Inc.

Should be Apple Inc. Should say All rights reserved.

> Source/WebCore/rendering/RenderBR.cpp:192
> +    IntRect boundingBox = linesBoundingBox();
> +    return IntRect(0, 0, boundingBox.width(), boundingBox.height());

I think you can do it like one of these two:

    return IntRect(IntPoint(), linesBoundingBox().size();
    return IntRect(IntPoint(0, 0), linesBoundingBox().size();

I like those slightly better than the two line form.

> LayoutTests/ChangeLog:16
> +            Changes in render tree dumb that don't affect rendering.

dump, not dumb

> LayoutTests/ChangeLog:25
> +            Changes in render tree dumb that don't affect rendering.

Ditto.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.