Rename 'dstView' to 'outputView' to avoid confusion w/ 'dstProxyView'
This is just a simple clean up from a CL request. Change-Id: Ibc09e19aeb8a160e4e4e03573fcf56cb1d58525b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/273983 Reviewed-by: Greg Daniel <egdaniel@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
This commit is contained in:
parent
20e7a53cd2
commit
fad1e0f47b
@ -163,7 +163,7 @@ private:
|
||||
}
|
||||
|
||||
void onPrePrepare(GrRecordingContext* context,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip* clip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView) final {
|
||||
SkArenaAlloc* arena = context->priv().recordTimeAllocator();
|
||||
@ -174,7 +174,7 @@ private:
|
||||
GrGeometryProcessor* geomProc = ClockwiseTestProcessor::Make(arena, fReadSkFragCoord);
|
||||
|
||||
// TODO: need to also give this to the recording context
|
||||
fProgramInfo = sk_gpu_test::CreateProgramInfo(context->priv().caps(), arena, dstView,
|
||||
fProgramInfo = sk_gpu_test::CreateProgramInfo(context->priv().caps(), arena, outputView,
|
||||
std::move(appliedClip), dstProxyView,
|
||||
geomProc, SkBlendMode::kPlus,
|
||||
GrPrimitiveType::kTriangleStrip);
|
||||
|
@ -171,7 +171,7 @@ private:
|
||||
}
|
||||
|
||||
void onPrePrepare(GrRecordingContext* context,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip* clip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView) final {
|
||||
SkArenaAlloc* arena = context->priv().recordTimeAllocator();
|
||||
@ -182,7 +182,7 @@ private:
|
||||
GrGeometryProcessor* geomProc = FwidthSquircleTestProcessor::Make(arena, fViewMatrix);
|
||||
|
||||
// TODO: need to also give this to the recording context
|
||||
fProgramInfo = sk_gpu_test::CreateProgramInfo(context->priv().caps(), arena, dstView,
|
||||
fProgramInfo = sk_gpu_test::CreateProgramInfo(context->priv().caps(), arena, outputView,
|
||||
std::move(appliedClip), dstProxyView,
|
||||
geomProc, SkBlendMode::kSrcOver,
|
||||
GrPrimitiveType::kTriangleStrip);
|
||||
|
@ -243,7 +243,7 @@ private:
|
||||
}
|
||||
|
||||
void onPrePrepare(GrRecordingContext* context,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip* clip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView) final {
|
||||
// We're going to create the GrProgramInfo (and the GrPipeline and geometry processor
|
||||
@ -255,7 +255,7 @@ private:
|
||||
|
||||
GrGeometryProcessor* geomProc = SampleLocationsTestProcessor::Make(arena, fGradType);
|
||||
|
||||
fProgramInfo = sk_gpu_test::CreateProgramInfo(context->priv().caps(), arena, dstView,
|
||||
fProgramInfo = sk_gpu_test::CreateProgramInfo(context->priv().caps(), arena, outputView,
|
||||
std::move(appliedClip), dstProxyView,
|
||||
geomProc, SkBlendMode::kSrcOver,
|
||||
GrPrimitiveType::kTriangleStrip,
|
||||
|
@ -51,7 +51,9 @@ public:
|
||||
}
|
||||
}
|
||||
|
||||
void onPrePrepare(GrRecordingContext*, const GrSurfaceProxyView*, GrAppliedClip*,
|
||||
void onPrePrepare(GrRecordingContext*,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip*,
|
||||
const GrXferProcessor::DstProxyView&) final;
|
||||
|
||||
void onPrepare(GrOpFlushState*) final;
|
||||
@ -93,7 +95,7 @@ private:
|
||||
// Create a GrProgramInfo object in the provided arena
|
||||
GrProgramInfo* createProgramInfo(const GrCaps*,
|
||||
SkArenaAlloc*,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip&&,
|
||||
const GrXferProcessor::DstProxyView&);
|
||||
|
||||
@ -555,7 +557,7 @@ static constexpr uint16_t kMSAAIndexData[] = {
|
||||
GR_DECLARE_STATIC_UNIQUE_KEY(gMSAAIndexBufferKey);
|
||||
|
||||
void FillRRectOp::onPrePrepare(GrRecordingContext* context,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip* clip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView) {
|
||||
SkArenaAlloc* arena = context->priv().recordTimeAllocator();
|
||||
@ -567,7 +569,7 @@ void FillRRectOp::onPrePrepare(GrRecordingContext* context,
|
||||
// in the record-time arena. Then, if the program info had already been seen, we could
|
||||
// get pointers back to the prior versions and be able to return the allocated space
|
||||
// back to the arena.
|
||||
fProgramInfo = this->createProgramInfo(context->priv().caps(), arena, dstView,
|
||||
fProgramInfo = this->createProgramInfo(context->priv().caps(), arena, outputView,
|
||||
std::move(appliedClip), dstProxyView);
|
||||
|
||||
context->priv().recordProgramInfo(fProgramInfo);
|
||||
@ -863,7 +865,7 @@ GrGLSLPrimitiveProcessor* FillRRectOp::Processor::createGLSLInstance(
|
||||
|
||||
GrProgramInfo* FillRRectOp::createProgramInfo(const GrCaps* caps,
|
||||
SkArenaAlloc* arena,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip&& appliedClip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView) {
|
||||
GrGeometryProcessor* geomProc = Processor::Make(arena, fAAType, fFlags);
|
||||
@ -875,7 +877,7 @@ GrProgramInfo* FillRRectOp::createProgramInfo(const GrCaps* caps,
|
||||
}
|
||||
initArgs.fCaps = caps;
|
||||
initArgs.fDstProxyView = dstProxyView;
|
||||
initArgs.fOutputSwizzle = dstView->swizzle();
|
||||
initArgs.fOutputSwizzle = outputView->swizzle();
|
||||
|
||||
GrPipeline::FixedDynamicState* fixedDynamicState = nullptr;
|
||||
|
||||
@ -888,11 +890,11 @@ GrProgramInfo* FillRRectOp::createProgramInfo(const GrCaps* caps,
|
||||
std::move(fProcessors),
|
||||
std::move(appliedClip));
|
||||
|
||||
GrRenderTargetProxy* dstProxy = dstView->asRenderTargetProxy();
|
||||
return arena->make<GrProgramInfo>(dstProxy->numSamples(),
|
||||
dstProxy->numStencilSamples(),
|
||||
dstProxy->backendFormat(),
|
||||
dstView->origin(),
|
||||
GrRenderTargetProxy* outputProxy = outputView->asRenderTargetProxy();
|
||||
return arena->make<GrProgramInfo>(outputProxy->numSamples(),
|
||||
outputProxy->numStencilSamples(),
|
||||
outputProxy->backendFormat(),
|
||||
outputView->origin(),
|
||||
pipeline,
|
||||
geomProc,
|
||||
fixedDynamicState,
|
||||
@ -906,11 +908,9 @@ void FillRRectOp::onExecute(GrOpFlushState* flushState, const SkRect& chainBound
|
||||
}
|
||||
|
||||
if (!fProgramInfo) {
|
||||
const GrSurfaceProxyView* dstView = flushState->view();
|
||||
|
||||
fProgramInfo = this->createProgramInfo(&flushState->caps(),
|
||||
flushState->allocator(),
|
||||
dstView,
|
||||
flushState->view(),
|
||||
flushState->detachAppliedClip(),
|
||||
flushState->dstProxyView());
|
||||
}
|
||||
|
@ -89,16 +89,16 @@ protected:
|
||||
|
||||
private:
|
||||
void onPrePrepare(GrRecordingContext* context,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip* clip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView) final {
|
||||
this->onPrePrepareDraws(context, dstView, clip, dstProxyView);
|
||||
this->onPrePrepareDraws(context, outputView, clip, dstProxyView);
|
||||
}
|
||||
void onPrepare(GrOpFlushState* state) final;
|
||||
|
||||
// Only the GrTextureOp currently overrides this virtual
|
||||
virtual void onPrePrepareDraws(GrRecordingContext*,
|
||||
const GrSurfaceProxyView*,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip*,
|
||||
const GrXferProcessor::DstProxyView&) {}
|
||||
|
||||
|
@ -109,7 +109,7 @@ GrProcessorSet::Analysis GrSimpleMeshDrawOpHelper::finalizeProcessors(
|
||||
const GrPipeline* GrSimpleMeshDrawOpHelper::CreatePipeline(
|
||||
const GrCaps* caps,
|
||||
SkArenaAlloc* arena,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip&& appliedClip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView,
|
||||
GrProcessorSet&& processorSet,
|
||||
@ -121,7 +121,7 @@ const GrPipeline* GrSimpleMeshDrawOpHelper::CreatePipeline(
|
||||
pipelineArgs.fDstProxyView = dstProxyView;
|
||||
pipelineArgs.fCaps = caps;
|
||||
pipelineArgs.fUserStencil = stencilSettings;
|
||||
pipelineArgs.fOutputSwizzle = dstView->swizzle();
|
||||
pipelineArgs.fOutputSwizzle = outputView->swizzle();
|
||||
|
||||
return arena->make<GrPipeline>(pipelineArgs,
|
||||
std::move(processorSet),
|
||||
|
@ -126,7 +126,7 @@ public:
|
||||
static const GrPipeline* CreatePipeline(
|
||||
const GrCaps*,
|
||||
SkArenaAlloc*,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip&&,
|
||||
const GrXferProcessor::DstProxyView&,
|
||||
GrProcessorSet&&,
|
||||
|
@ -643,7 +643,7 @@ private:
|
||||
}
|
||||
|
||||
void onPrePrepareDraws(GrRecordingContext* context,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip* clip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView) override {
|
||||
TRACE_EVENT0("skia.gpu", TRACE_FUNC);
|
||||
|
@ -57,7 +57,7 @@ sk_sp<GrTextureProxy> MakeTextureProxyFromData(GrContext* context,
|
||||
|
||||
GrProgramInfo* CreateProgramInfo(const GrCaps* caps,
|
||||
SkArenaAlloc* arena,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip&& appliedClip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView,
|
||||
GrGeometryProcessor* geomProc,
|
||||
@ -71,7 +71,7 @@ GrProgramInfo* CreateProgramInfo(const GrCaps* caps,
|
||||
initArgs.fUserStencil = stencil;
|
||||
initArgs.fCaps = caps;
|
||||
initArgs.fDstProxyView = dstProxyView;
|
||||
initArgs.fOutputSwizzle = dstView->swizzle();
|
||||
initArgs.fOutputSwizzle = outputView->swizzle();
|
||||
|
||||
GrPipeline::FixedDynamicState* fixedDynamicState = nullptr;
|
||||
|
||||
@ -94,11 +94,11 @@ GrProgramInfo* CreateProgramInfo(const GrCaps* caps,
|
||||
std::move(processors),
|
||||
std::move(appliedClip));
|
||||
|
||||
GrRenderTargetProxy* dstProxy = dstView->asRenderTargetProxy();
|
||||
return arena->make<GrProgramInfo>(dstProxy->numSamples(),
|
||||
dstProxy->numStencilSamples(),
|
||||
dstProxy->backendFormat(),
|
||||
dstView->origin(),
|
||||
GrRenderTargetProxy* outputProxy = outputView->asRenderTargetProxy();
|
||||
return arena->make<GrProgramInfo>(outputProxy->numSamples(),
|
||||
outputProxy->numStencilSamples(),
|
||||
outputProxy->backendFormat(),
|
||||
outputView->origin(),
|
||||
pipeline,
|
||||
geomProc,
|
||||
fixedDynamicState,
|
||||
|
@ -27,7 +27,7 @@ sk_sp<GrTextureProxy> MakeTextureProxyFromData(GrContext*,
|
||||
|
||||
GrProgramInfo* CreateProgramInfo(const GrCaps*,
|
||||
SkArenaAlloc*,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip&&,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView,
|
||||
GrGeometryProcessor*, SkBlendMode,
|
||||
|
@ -103,7 +103,7 @@ private:
|
||||
const SkMatrix& localMatrix);
|
||||
|
||||
void onPrePrepareDraws(GrRecordingContext*,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip*,
|
||||
const GrXferProcessor::DstProxyView&) final;
|
||||
void onPrepareDraws(Target*) override;
|
||||
@ -160,7 +160,7 @@ TestRectOp::TestRectOp(const GrCaps* caps,
|
||||
}
|
||||
|
||||
void TestRectOp::onPrePrepareDraws(GrRecordingContext* context,
|
||||
const GrSurfaceProxyView* dstView,
|
||||
const GrSurfaceProxyView* outputView,
|
||||
GrAppliedClip* clip,
|
||||
const GrXferProcessor::DstProxyView& dstProxyView) {
|
||||
SkArenaAlloc* arena = context->priv().recordTimeAllocator();
|
||||
@ -170,7 +170,7 @@ void TestRectOp::onPrePrepareDraws(GrRecordingContext* context,
|
||||
|
||||
fPipeline = GrSimpleMeshDrawOpHelper::CreatePipeline(context->priv().caps(),
|
||||
arena,
|
||||
dstView,
|
||||
outputView,
|
||||
std::move(appliedClip),
|
||||
dstProxyView,
|
||||
std::move(fProcessorSet),
|
||||
|
Loading…
Reference in New Issue
Block a user