Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(183)

Side by Side Diff: Source/core/rendering/style/RenderStyleConstants.h

Issue 236203020: Separate repaint and layout requirements of StyleDifference (Step 1) (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase; Renaming of some methods and small changes in StyleDifference Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
3 * (C) 2000 Antti Koivisto (koivisto@kde.org) 3 * (C) 2000 Antti Koivisto (koivisto@kde.org)
4 * (C) 2000 Dirk Mueller (mueller@kde.org) 4 * (C) 2000 Dirk Mueller (mueller@kde.org)
5 * Copyright (C) 2003, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved. 5 * Copyright (C) 2003, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
6 * Copyright (C) 2006 Graham Dennis (graham.dennis@gmail.com) 6 * Copyright (C) 2006 Graham Dennis (graham.dennis@gmail.com)
7 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/) 7 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/)
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 26 matching lines...) Expand all
37 Reattach, 37 Reattach,
38 ReattachNoRenderer 38 ReattachNoRenderer
39 }; 39 };
40 40
41 static const size_t PrintColorAdjustBits = 1; 41 static const size_t PrintColorAdjustBits = 1;
42 enum PrintColorAdjust { 42 enum PrintColorAdjust {
43 PrintColorAdjustEconomy, 43 PrintColorAdjustEconomy,
44 PrintColorAdjustExact 44 PrintColorAdjustExact
45 }; 45 };
46 46
47 // FIXME: To be replaced by StyleDifference defined in StyleDifference.h.
47 // The difference between two styles. The following values are used: 48 // The difference between two styles. The following values are used:
48 // - StyleDifferenceEqual - The two styles are identical. 49 // - StyleDifferenceEqual - The two styles are identical.
49 // FIXME: When style difference is context sensitive, RenderStyle::visualInval idationDiff() may return 50 // FIXME: When style difference is context sensitive, RenderStyle::visualInval idationDiff() may return
50 // StyleDifferenceEqual even there is difference. Will resolve the issue when fixing crbug.com/358460. 51 // StyleDifferenceEqual even there is difference. Will resolve the issue when fixing crbug.com/358460.
51 // - StyleDifferenceRecompositeLayer - The layer needs its position and transfor m updated, but no repaint. 52 // - StyleDifferenceRecompositeLayer - The layer needs its position and transfor m updated, but no repaint.
52 // - StyleDifferenceRepaint - The object just needs to be repainted. 53 // - StyleDifferenceRepaint - The object just needs to be repainted.
53 // - StyleDifferenceRepaintLayer - The layer and its descendant layers need to b e repainted. 54 // - StyleDifferenceRepaintLayer - The layer and its descendant layers need to b e repainted.
54 // - StyleDifferenceLayoutPositionedMovementOnly - Only the position of this pos itioned object has been updated. 55 // - StyleDifferenceLayoutPositionedMovementOnly - Only the position of this pos itioned object has been updated.
55 // - StyleDifferenceLayout - A full layout is required. 56 // - StyleDifferenceLayout - A full layout is required.
56 enum StyleDifference { 57 enum StyleDifferenceLegacy {
57 StyleDifferenceEqual, 58 StyleDifferenceEqual,
58 StyleDifferenceRecompositeLayer, 59 StyleDifferenceRecompositeLayer,
59 StyleDifferenceRepaint, 60 StyleDifferenceRepaint,
60 StyleDifferenceRepaintLayer, 61 StyleDifferenceRepaintLayer,
61 StyleDifferenceLayoutPositionedMovementOnly, 62 StyleDifferenceLayoutPositionedMovementOnly,
62 StyleDifferenceLayout 63 StyleDifferenceLayout
63 }; 64 };
64 65
65 // When some style properties change, different amounts of work have to be done depending on 66 // When some style properties change, different amounts of work have to be done depending on
66 // context (e.g. whether the property is changing on an element which has a comp ositing layer). 67 // context (e.g. whether the property is changing on an element which has a comp ositing layer).
(...skipping 480 matching lines...) Expand 10 before | Expand all | Expand 10 after
547 // Reasonable maximum to prevent insane font sizes from causing crashes on some platforms (such as Windows). 548 // Reasonable maximum to prevent insane font sizes from causing crashes on some platforms (such as Windows).
548 static const float maximumAllowedFontSize = 1000000.0f; 549 static const float maximumAllowedFontSize = 1000000.0f;
549 550
550 enum TextIndentLine { TextIndentFirstLine, TextIndentEachLine }; 551 enum TextIndentLine { TextIndentFirstLine, TextIndentEachLine };
551 552
552 enum CSSBoxType { BoxMissing = 0, MarginBox, BorderBox, PaddingBox, ContentBox } ; 553 enum CSSBoxType { BoxMissing = 0, MarginBox, BorderBox, PaddingBox, ContentBox } ;
553 554
554 } // namespace WebCore 555 } // namespace WebCore
555 556
556 #endif // RenderStyleConstants_h 557 #endif // RenderStyleConstants_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698