Commit c78f00a9 authored by Yi Xu's avatar Yi Xu Committed by Commit Bot

Remove Useless Variable |usage_counters_| in canvas


Change-Id: Ic2846811bcaadd612ed40475adae95f469044ff6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2050577
Commit-Queue: Yi Xu <yiyix@chromium.org>
Reviewed-by: default avatarFernando Serboncini <fserb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#742914}
parent 03a3575d
...@@ -855,8 +855,6 @@ void BaseRenderingContext2D::clearRect(double x, ...@@ -855,8 +855,6 @@ void BaseRenderingContext2D::clearRect(double x,
double y, double y,
double width, double width,
double height) { double height) {
usage_counters_.num_clear_rect_calls++;
if (!ValidateRectForCanvas(x, y, width, height)) if (!ValidateRectForCanvas(x, y, width, height))
return; return;
...@@ -1559,8 +1557,6 @@ ImageData* BaseRenderingContext2D::getImageData( ...@@ -1559,8 +1557,6 @@ ImageData* BaseRenderingContext2D::getImageData(
base::TimeTicks start_time = base::TimeTicks::Now(); base::TimeTicks start_time = base::TimeTicks::Now();
usage_counters_.num_get_image_data_calls++;
usage_counters_.area_get_image_data_calls += sw * sh;
if (!OriginClean()) { if (!OriginClean()) {
exception_state.ThrowSecurityError( exception_state.ThrowSecurityError(
"The canvas has been tainted by cross-origin data."); "The canvas has been tainted by cross-origin data.");
...@@ -1722,8 +1718,6 @@ void BaseRenderingContext2D::putImageData(ImageData* data, ...@@ -1722,8 +1718,6 @@ void BaseRenderingContext2D::putImageData(ImageData* data,
return; return;
} }
base::TimeTicks start_time = base::TimeTicks::Now(); base::TimeTicks start_time = base::TimeTicks::Now();
usage_counters_.num_put_image_data_calls++;
usage_counters_.area_put_image_data_calls += dirty_width * dirty_height;
if (data->BufferBase()->IsDetached()) { if (data->BufferBase()->IsDetached()) {
exception_state.ThrowDOMException(DOMExceptionCode::kInvalidStateError, exception_state.ThrowDOMException(DOMExceptionCode::kInvalidStateError,
...@@ -1998,11 +1992,6 @@ void BaseRenderingContext2D::setTextBaseline(const String& s) { ...@@ -1998,11 +1992,6 @@ void BaseRenderingContext2D::setTextBaseline(const String& s) {
ModifiableState().SetTextBaseline(baseline); ModifiableState().SetTextBaseline(baseline);
} }
const BaseRenderingContext2D::UsageCounters&
BaseRenderingContext2D::GetUsage() {
return usage_counters_;
}
void BaseRenderingContext2D::Trace(Visitor* visitor) { void BaseRenderingContext2D::Trace(Visitor* visitor) {
visitor->Trace(state_stack_); visitor->Trace(state_stack_);
} }
......
...@@ -366,8 +366,6 @@ class MODULES_EXPORT BaseRenderingContext2D : public GarbageCollectedMixin, ...@@ -366,8 +366,6 @@ class MODULES_EXPORT BaseRenderingContext2D : public GarbageCollectedMixin,
HeapVector<Member<CanvasRenderingContext2DState>> state_stack_; HeapVector<Member<CanvasRenderingContext2DState>> state_stack_;
AntiAliasingMode clip_antialiasing_; AntiAliasingMode clip_antialiasing_;
mutable UsageCounters usage_counters_;
virtual void FinalizeFrame() {} virtual void FinalizeFrame() {}
float GetFontBaseline(const SimpleFontData&) const; float GetFontBaseline(const SimpleFontData&) const;
......
...@@ -678,7 +678,6 @@ void CanvasRenderingContext2D::FinalizeFrame() { ...@@ -678,7 +678,6 @@ void CanvasRenderingContext2D::FinalizeFrame() {
TRACE_EVENT0("blink", "CanvasRenderingContext2D::FinalizeFrame"); TRACE_EVENT0("blink", "CanvasRenderingContext2D::FinalizeFrame");
if (IsPaintable()) if (IsPaintable())
canvas()->GetCanvas2DLayerBridge()->FinalizeFrame(); canvas()->GetCanvas2DLayerBridge()->FinalizeFrame();
usage_counters_.num_frames_since_reset++;
} }
bool CanvasRenderingContext2D::ParseColorOrCurrentColor( bool CanvasRenderingContext2D::ParseColorOrCurrentColor(
...@@ -1012,7 +1011,6 @@ bool CanvasRenderingContext2D::FocusRingCallIsValid(const Path& path, ...@@ -1012,7 +1011,6 @@ bool CanvasRenderingContext2D::FocusRingCallIsValid(const Path& path,
} }
void CanvasRenderingContext2D::DrawFocusRing(const Path& path) { void CanvasRenderingContext2D::DrawFocusRing(const Path& path) {
usage_counters_.num_draw_focus_calls++;
if (!GetOrCreatePaintCanvas()) if (!GetOrCreatePaintCanvas())
return; return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment