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

Side by Side Diff: Source/core/rendering/RenderListMarker.cpp

Issue 217563003: Turn styleWillChange |newStyle| into a reference (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 9 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) 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, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserv ed. 4 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserv ed.
5 * Copyright (C) 2006 Andrew Wellington (proton@wiretapped.net) 5 * Copyright (C) 2006 Andrew Wellington (proton@wiretapped.net)
6 * Copyright (C) 2010 Daniel Bates (dbates@intudata.com) 6 * Copyright (C) 2010 Daniel Bates (dbates@intudata.com)
7 * 7 *
8 * This library is free software; you can redistribute it and/or 8 * This library is free software; you can redistribute it and/or
9 * modify it under the terms of the GNU Library General Public 9 * modify it under the terms of the GNU Library General Public
10 * License as published by the Free Software Foundation; either 10 * License as published by the Free Software Foundation; either
(...skipping 1060 matching lines...) Expand 10 before | Expand all | Expand 10 after
1071 } 1071 }
1072 1072
1073 RenderListMarker* RenderListMarker::createAnonymous(RenderListItem* item) 1073 RenderListMarker* RenderListMarker::createAnonymous(RenderListItem* item)
1074 { 1074 {
1075 Document& document = item->document(); 1075 Document& document = item->document();
1076 RenderListMarker* renderer = new RenderListMarker(item); 1076 RenderListMarker* renderer = new RenderListMarker(item);
1077 renderer->setDocumentForAnonymous(&document); 1077 renderer->setDocumentForAnonymous(&document);
1078 return renderer; 1078 return renderer;
1079 } 1079 }
1080 1080
1081 void RenderListMarker::styleWillChange(StyleDifference diff, const RenderStyle* newStyle) 1081 void RenderListMarker::styleWillChange(StyleDifference diff, const RenderStyle& newStyle)
1082 { 1082 {
1083 if (style() && (newStyle->listStylePosition() != style()->listStylePosition( ) || newStyle->listStyleType() != style()->listStyleType())) 1083 if (style() && (newStyle.listStylePosition() != style()->listStylePosition() || newStyle.listStyleType() != style()->listStyleType()))
1084 setNeedsLayoutAndPrefWidthsRecalc(); 1084 setNeedsLayoutAndPrefWidthsRecalc();
1085 1085
1086 RenderBox::styleWillChange(diff, newStyle); 1086 RenderBox::styleWillChange(diff, newStyle);
1087 } 1087 }
1088 1088
1089 void RenderListMarker::styleDidChange(StyleDifference diff, const RenderStyle* o ldStyle) 1089 void RenderListMarker::styleDidChange(StyleDifference diff, const RenderStyle* o ldStyle)
1090 { 1090 {
1091 RenderBox::styleDidChange(diff, oldStyle); 1091 RenderBox::styleDidChange(diff, oldStyle);
1092 1092
1093 if (m_image != style()->listStyleImage()) { 1093 if (m_image != style()->listStyleImage()) {
(...skipping 747 matching lines...) Expand 10 before | Expand all | Expand 10 after
1841 1841
1842 if (clipToVisibleContent) 1842 if (clipToVisibleContent)
1843 computeRectForRepaint(repaintContainer, rect); 1843 computeRectForRepaint(repaintContainer, rect);
1844 else 1844 else
1845 rect = localToContainerQuad(FloatRect(rect), repaintContainer).enclosing BoundingBox(); 1845 rect = localToContainerQuad(FloatRect(rect), repaintContainer).enclosing BoundingBox();
1846 1846
1847 return rect; 1847 return rect;
1848 } 1848 }
1849 1849
1850 } // namespace WebCore 1850 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698