Changeset 84956 in webkit


Ignore:
Timestamp:
Apr 26, 2011 2:21:53 PM (13 years ago)
Author:
mihaip@chromium.org
Message:

2011-04-26 Mihai Parparita <mihaip@chromium.org>

Reviewed by Eric Seidel.

InlineBox.h shouldn't include RenderBR.h
https://bugs.webkit.org/show_bug.cgi?id=59480

Per bug 59348 RenderBR.h is a "top header" by include * size, because
InlineBox.h includes it. There's no reason why it has to.

  • rendering/InlineBox.h:
  • rendering/InlineTextBox.cpp:
  • rendering/RenderBlock.h:
Location:
trunk/Source/WebCore
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r84954 r84956  
     12011-04-26  Mihai Parparita  <mihaip@chromium.org>
     2
     3        Reviewed by Eric Seidel.
     4
     5        InlineBox.h shouldn't include RenderBR.h
     6        https://bugs.webkit.org/show_bug.cgi?id=59480
     7
     8        Per bug 59348 RenderBR.h is a "top header" by include * size, because
     9        InlineBox.h includes it. There's no reason why it has to.
     10
     11        * rendering/InlineBox.h:
     12        * rendering/InlineTextBox.cpp:
     13        * rendering/RenderBlock.h:
     14
    1152011-04-26  Sam Weinig  <sam@webkit.org>
    216
  • trunk/Source/WebCore/rendering/InlineBox.h

    r83540 r84956  
    2222#define InlineBox_h
    2323
    24 #include "RenderBR.h"
    2524#include "RenderBoxModelObject.h"
    2625#include "TextDirection.h"
  • trunk/Source/WebCore/rendering/InlineTextBox.cpp

    r84631 r84956  
    3636#include "PaintInfo.h"
    3737#include "RenderArena.h"
     38#include "RenderBR.h"
    3839#include "RenderBlock.h"
    3940#include "RenderCombineText.h"
  • trunk/Source/WebCore/rendering/RenderBlock.h

    r84096 r84956  
    4040class LineWidth;
    4141class RenderInline;
     42class RenderText;
    4243
    4344struct BidiRun;
Note: See TracChangeset for help on using the changeset viewer.