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

Unified Diff: Source/platform/graphics/filters/FilterOperation.h

Issue 305883002: Switch arguments to const Length references for performance (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/platform/LengthSize.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/graphics/filters/FilterOperation.h
diff --git a/Source/platform/graphics/filters/FilterOperation.h b/Source/platform/graphics/filters/FilterOperation.h
index 4b87be4bf307724cf921f42c1b9e206100d54e4a..b0559ddb0fa7eb798a222d5c0dccf0bc547cdfd7 100644
--- a/Source/platform/graphics/filters/FilterOperation.h
+++ b/Source/platform/graphics/filters/FilterOperation.h
@@ -235,12 +235,12 @@ DEFINE_TYPE_CASTS(BasicComponentTransferFilterOperation, FilterOperation, op, is
class PLATFORM_EXPORT BlurFilterOperation : public FilterOperation {
public:
- static PassRefPtr<BlurFilterOperation> create(Length stdDeviation)
+ static PassRefPtr<BlurFilterOperation> create(const Length& stdDeviation)
{
return adoptRef(new BlurFilterOperation(stdDeviation));
}
- Length stdDeviation() const { return m_stdDeviation; }
+ const Length& stdDeviation() const { return m_stdDeviation; }
virtual bool affectsOpacity() const OVERRIDE { return true; }
virtual bool movesPixels() const OVERRIDE { return true; }
@@ -256,7 +256,7 @@ private:
return m_stdDeviation == other->m_stdDeviation;
}
- BlurFilterOperation(Length stdDeviation)
+ BlurFilterOperation(const Length& stdDeviation)
: FilterOperation(BLUR)
, m_stdDeviation(stdDeviation)
{
« no previous file with comments | « Source/platform/LengthSize.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698