Improved heuristic for disable acceleration.

third_party/WebKit/Source/modules/canvas2d/CanvasRenderingContext2D.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2007 Alp Toker <alp@atoker.com>
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008 Dirk Schulze <krit@webkit.org>
  * Copyright (C) 2010 Torch Mobile (Beijing) Co. Ltd. All rights reserved.
  * Copyright (C) 2012, 2013 Intel Corporation. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
  *
@@ skipping 35 matching lines @@
 #include "core/frame/Settings.h"
 #include "core/html/TextMetrics.h"
 #include "core/html/canvas/CanvasFontCache.h"
 #include "core/layout/LayoutBox.h"
 #include "core/layout/LayoutTheme.h"
 #include "modules/canvas2d/CanvasStyle.h"
 #include "modules/canvas2d/HitRegion.h"
 #include "modules/canvas2d/Path2D.h"
 #include "platform/fonts/FontCache.h"
 #include "platform/graphics/DrawLooperBuilder.h"
-#include "platform/graphics/ExpensiveCanvasHeuristicParameters.h"
 #include "platform/graphics/ImageBuffer.h"
 #include "platform/graphics/StrokeData.h"
 #include "platform/graphics/skia/SkiaUtils.h"
 #include "platform/text/BidiTextRun.h"
 #include "public/platform/Platform.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/core/SkImageFilter.h"
 #include "wtf/MathExtras.h"
 #include "wtf/text/StringBuilder.h"
 #include "wtf/typed_arrays/ArrayBufferContents.h"
@@ skipping 994 matching lines @@
 }
 
 unsigned CanvasRenderingContext2D::hitRegionsCount() const
 {
     if (m_hitRegionManager)
         return m_hitRegionManager->getHitRegionsCount();
 
     return 0;
 }
 
+
+double CanvasRenderingContext2D::estimateRenderingCostForUsage(BaseRenderingContext2D::UsageCounters usage, ExpensiveCanvasHeuristicParameters::RenderingModeCostIndex index) const

[Justin Novosad, 2016/08/04 15:53:17]

double -> float   here and below

[sebastienlc, 2016/08/04 16:36:27]

Done.

+{
+    double basicCostOfDrawCalls =
+        ExpensiveCanvasHeuristicParameters::FillRectFixedCost[index] * usage.numDrawCalls[BaseRenderingContext2D::FillRect] +
+        ExpensiveCanvasHeuristicParameters::FillConvexPathFixedCost[index] * (usage.numDrawCalls[BaseRenderingContext2D::FillPath] - usage.numNonConvexFillPathCalls) +
+        ExpensiveCanvasHeuristicParameters::FillNonConvexPathFixedCost[index] * usage.numNonConvexFillPathCalls +
+        ExpensiveCanvasHeuristicParameters::FillTextFixedCost[index] * usage.numDrawCalls[BaseRenderingContext2D::FillText] +
+
+        ExpensiveCanvasHeuristicParameters::StrokeRectFixedCost[index] * usage.numDrawCalls[BaseRenderingContext2D::StrokeRect] +
+        ExpensiveCanvasHeuristicParameters::StrokePathFixedCost[index] * usage.numDrawCalls[BaseRenderingContext2D::StrokePath] +
+        ExpensiveCanvasHeuristicParameters::StrokeTextFixedCost[index] * usage.numDrawCalls[BaseRenderingContext2D::StrokeText] +
+
+        ExpensiveCanvasHeuristicParameters::FillRectVariableCostPerArea[index] * usage.boundingBoxAreaDrawCalls[BaseRenderingContext2D::FillRect] +
+        ExpensiveCanvasHeuristicParameters::FillConvexPathVariableCostPerArea[index] * (usage.boundingBoxAreaDrawCalls[BaseRenderingContext2D::FillPath] - usage.nonConvexFillPathArea) +
+        ExpensiveCanvasHeuristicParameters::FillNonConvexPathVariableCostPerArea[index] * usage.nonConvexFillPathArea +
+        ExpensiveCanvasHeuristicParameters::FillTextVariableCostPerArea[index] * usage.boundingBoxAreaDrawCalls[BaseRenderingContext2D::FillText] +
+
+        ExpensiveCanvasHeuristicParameters::StrokeRectVariableCostPerArea[index] * usage.boundingBoxAreaDrawCalls[BaseRenderingContext2D::StrokeRect] +
+        ExpensiveCanvasHeuristicParameters::StrokePathVariableCostPerArea[index] * usage.boundingBoxAreaDrawCalls[BaseRenderingContext2D::StrokePath] +
+        ExpensiveCanvasHeuristicParameters::StrokeTextVariableCostPerArea[index] * usage.boundingBoxAreaDrawCalls[BaseRenderingContext2D::StrokeText] +
+
+        ExpensiveCanvasHeuristicParameters::PutImageDataFixedCost[index] * usage.numPutImageDataCalls +
+        ExpensiveCanvasHeuristicParameters::PutImageDataVariableCostPerArea[index] * usage.areaPutImageDataCalls +
+
+        ExpensiveCanvasHeuristicParameters::DrawSVGImageFixedCost[index] * usage.numDrawCalls[BaseRenderingContext2D::DrawSVGImage] +
+        ExpensiveCanvasHeuristicParameters::DrawPNGImageFixedCost[index] * usage.numDrawCalls[BaseRenderingContext2D::DrawPNGImage] +
+
+        ExpensiveCanvasHeuristicParameters::DrawSVGImageVariableCostPerArea[index] * usage.boundingBoxAreaDrawCalls[BaseRenderingContext2D::DrawSVGImage] +
+        ExpensiveCanvasHeuristicParameters::DrawPNGImageVariableCostPerArea[index] * usage.boundingBoxAreaDrawCalls[BaseRenderingContext2D::DrawPNGImage];
+
+    double fillTypeAdjustment =
+        ExpensiveCanvasHeuristicParameters::PatternFillTypeFixedCost[index] * usage.numPatterns +
+        ExpensiveCanvasHeuristicParameters::LinearGradientFillTypeFixedCost[index] * usage.numLinearGradients +
+        ExpensiveCanvasHeuristicParameters::RadialGradientFillTypeFixedCost[index] * usage.numRadialGradients +
+
+        ExpensiveCanvasHeuristicParameters::PatternFillTypeVariableCostPerArea[index] * usage.boundingBoxAreaFillType[BaseRenderingContext2D::PatternFillType] +
+        ExpensiveCanvasHeuristicParameters::LinearGradientFillVariableCostPerArea[index] * usage.boundingBoxAreaFillType[BaseRenderingContext2D::LinearGradientFillType] +
+        ExpensiveCanvasHeuristicParameters::RadialGradientFillVariableCostPerArea[index] * usage.boundingBoxAreaFillType[BaseRenderingContext2D::RadialGradientFillType];
+
+    double shadowAdjustment =
+        ExpensiveCanvasHeuristicParameters::ShadowFixedCost[index] * usage.numBlurredShadows +
+        ExpensiveCanvasHeuristicParameters::ShadowVariableCostPerAreaTimesShadowBlurSquared[index] * usage.boundingBoxAreaTimesShadowBlurSquared;
+
+    return basicCostOfDrawCalls + fillTypeAdjustment + shadowAdjustment;
+}
+
 bool CanvasRenderingContext2D::isAccelerationOptimalForCanvasContent() const
 {
-    // Simple heuristic to determine if the GPU accelerated pipeline should be
-    // used to maximize performance of 2D canvas based on past usage.
+    // Heuristic to determine if the GPU accelerated rendering pipeline is optimal
+    // for performance based on past usage. It has a bias towards suggesting that the
+    // accelerated pipeline is optimal.
 
-    int numDrawPathCalls =
-        m_usageCounters.numDrawCalls[StrokePath] +
-        m_usageCounters.numDrawCalls[FillPath] +
-        m_usageCounters.numDrawCalls[FillText] +
-        m_usageCounters.numDrawCalls[StrokeText] +
-        m_usageCounters.numDrawCalls[FillRect] +
-        m_usageCounters.numDrawCalls[StrokeRect];
+    double acceleratedCost = estimateRenderingCostForUsage(m_usageCounters, ExpensiveCanvasHeuristicParameters::AcceleratedModeIndex);

[Justin Novosad, 2016/08/04 15:53:17]

double -> float here and below

[sebastienlc, 2016/08/04 16:36:27]

Done.

 
-    double acceleratedCost =
-        numDrawPathCalls * ExpensiveCanvasHeuristicParameters::AcceleratedDrawPathApproximateCost +
-        m_usageCounters.numGetImageDataCalls * ExpensiveCanvasHeuristicParameters::AcceleratedGetImageDataApproximateCost+
-        m_usageCounters.numDrawCalls[DrawImage] * ExpensiveCanvasHeuristicParameters::AcceleratedDrawImageApproximateCost;
+    double recordingCost = estimateRenderingCostForUsage(m_usageCounters, ExpensiveCanvasHeuristicParameters::RecordingModeIndex);
 
-    double recordingCost =
-        numDrawPathCalls * ExpensiveCanvasHeuristicParameters::RecordingDrawPathApproximateCost +
-        m_usageCounters.numGetImageDataCalls * ExpensiveCanvasHeuristicParameters::UnacceleratedGetImageDataApproximateCost +
-        m_usageCounters.numDrawCalls[DrawImage] * ExpensiveCanvasHeuristicParameters::RecordingDrawImageApproximateCost;
+    double costDifference = acceleratedCost - recordingCost;
+    double percentCostReduction = costDifference / acceleratedCost * 100.0;
+    double costDifferencePerFrame = costDifference / m_usageCounters.numFramesSinceReset;
 
-    if (recordingCost * ExpensiveCanvasHeuristicParameters::AcceleratedHeuristicBias < acceleratedCost) {
+    if (percentCostReduction >= ExpensiveCanvasHeuristicParameters::MinPercentageImprovementToSuggestDisableAcceleration
+        && costDifferencePerFrame >= ExpensiveCanvasHeuristicParameters::MinCostPerFrameImprovementToSuggestDisableAcceleration) {
         return false;
     }
     return true;
 }
 
 void CanvasRenderingContext2D::resetUsageTracking()
 {
     UsageCounters newCounters;
     m_usageCounters = newCounters;
 }
 
 } // namespace blink