Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) | 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) |
| 3 * (C) 1999 Antti Koivisto (koivisto@kde.org) | 3 * (C) 1999 Antti Koivisto (koivisto@kde.org) |
| 4 * Copyright (C) 2003, 2006, 2007 Apple Inc. All rights reserved. | 4 * Copyright (C) 2003, 2006, 2007 Apple Inc. All rights reserved. |
| 5 * | 5 * |
| 6 * This library is free software; you can redistribute it and/or | 6 * This library is free software; you can redistribute it and/or |
| 7 * modify it under the terms of the GNU Library General Public | 7 * modify it under the terms of the GNU Library General Public |
| 8 * License as published by the Free Software Foundation; either | 8 * License as published by the Free Software Foundation; either |
| 9 * version 2 of the License, or (at your option) any later version. | 9 * version 2 of the License, or (at your option) any later version. |
| 10 * | 10 * |
| (...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 64 | 64 |
| 65 | 65 |
| 66 class RenderBox : public RenderBoxModelObject { | 66 class RenderBox : public RenderBoxModelObject { |
| 67 public: | 67 public: |
| 68 explicit RenderBox(ContainerNode*); | 68 explicit RenderBox(ContainerNode*); |
| 69 | 69 |
| 70 // hasAutoZIndex only returns true if the element is positioned or a flex-it em since | 70 // hasAutoZIndex only returns true if the element is positioned or a flex-it em since |
| 71 // position:static elements that are not flex-items get their z-index coerce d to auto. | 71 // position:static elements that are not flex-items get their z-index coerce d to auto. |
| 72 virtual LayerType layerTypeRequired() const OVERRIDE | 72 virtual LayerType layerTypeRequired() const OVERRIDE |
| 73 { | 73 { |
| 74 if (isRoot() || isPositioned() || createsGroup() || hasClipPath() || has Transform() || hasHiddenBackface() || hasReflection() || style()->specifiesColum ns() || !style()->hasAutoZIndex() || style()->hasWillChangeCompositingHint()) | 74 if (isRoot() || isPositioned() || createsGroup() || hasClipPath() || has Transform() || hasHiddenBackface() || hasReflection() || style()->specifiesColum ns() || !style()->hasAutoZIndex() || style()->hasWillChangeCompositingHint() || style()->hasWillChangeCompositingHint()) |
|
abarth-chromium
2014/03/07 16:04:39
Why do we need this twice? Maybe you meant hasWil
ajuma
2014/03/07 17:03:41
Indeed, thanks for catching that!
| |
| 75 return NormalLayer; | 75 return NormalLayer; |
| 76 if (hasOverflowClip()) | 76 if (hasOverflowClip()) |
| 77 return OverflowClipLayer; | 77 return OverflowClipLayer; |
| 78 | 78 |
| 79 return NoLayer; | 79 return NoLayer; |
| 80 } | 80 } |
| 81 | 81 |
| 82 virtual bool backgroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect) const OVERRIDE; | 82 virtual bool backgroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect) const OVERRIDE; |
| 83 | 83 |
| 84 // Use this with caution! No type checking is done! | 84 // Use this with caution! No type checking is done! |
| (...skipping 689 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 774 if (UNLIKELY(inlineBoxWrapper() != 0)) | 774 if (UNLIKELY(inlineBoxWrapper() != 0)) |
| 775 deleteLineBoxWrapper(); | 775 deleteLineBoxWrapper(); |
| 776 } | 776 } |
| 777 | 777 |
| 778 ensureRareData().m_inlineBoxWrapper = boxWrapper; | 778 ensureRareData().m_inlineBoxWrapper = boxWrapper; |
| 779 } | 779 } |
| 780 | 780 |
| 781 } // namespace WebCore | 781 } // namespace WebCore |
| 782 | 782 |
| 783 #endif // RenderBox_h | 783 #endif // RenderBox_h |
| OLD | NEW |