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

Side by Side Diff: src/core/SkXfermode.cpp

Issue 759713002: Make all blending up to GrOptDrawState be handled by the xp/xp factory. (Closed) Base URL: https://skia.googlesource.com/skia.git@xferFactorySolo
Patch Set: Clean up Created 6 years 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
OLDNEW
1 1
2 /* 2 /*
3 * Copyright 2006 The Android Open Source Project 3 * Copyright 2006 The Android Open Source Project
4 * 4 *
5 * Use of this source code is governed by a BSD-style license that can be 5 * Use of this source code is governed by a BSD-style license that can be
6 * found in the LICENSE file. 6 * found in the LICENSE file.
7 */ 7 */
8 8
9 #include "SkXfermode.h" 9 #include "SkXfermode.h"
10 #include "SkXfermode_opts_SSE2.h" 10 #include "SkXfermode_opts_SSE2.h"
(...skipping 671 matching lines...) Expand 10 before | Expand all | Expand 10 after
682 bool SkXfermode::asXPFactory(GrXPFactory**) const { 682 bool SkXfermode::asXPFactory(GrXPFactory**) const {
683 return false; 683 return false;
684 } 684 }
685 685
686 686
687 #if SK_SUPPORT_GPU 687 #if SK_SUPPORT_GPU
688 #include "effects/GrPorterDuffXferProcessor.h" 688 #include "effects/GrPorterDuffXferProcessor.h"
689 689
690 bool SkXfermode::AsFragmentProcessorOrXPFactory(SkXfermode* xfermode, 690 bool SkXfermode::AsFragmentProcessorOrXPFactory(SkXfermode* xfermode,
691 GrFragmentProcessor** fp, 691 GrFragmentProcessor** fp,
692 GrXPFactory** xpf, 692 GrXPFactory** xpf) {
693 Coeff* src, Coeff* dst) { 693 Coeff src, dst;
694 Mode mode; 694 Mode mode;
695 if (NULL == xfermode) { 695 if (NULL == xfermode) {
696 SkAssertResult(ModeAsCoeff(kSrcOver_Mode, src, dst)); 696 *xpf = GrPorterDuffXPFactory::Create(kSrcOver_Mode);
697 *xpf = GrPorterDuffXPFactory::Create(*src, *dst);
698 return true; 697 return true;
699 } else if (xfermode->asMode(&mode) && mode <= kLastCoeffMode) { 698 } else if (xfermode->asMode(&mode) && mode <= kLastCoeffMode) {
700 *xpf = GrPorterDuffXPFactory::Create(mode); 699 *xpf = GrPorterDuffXPFactory::Create(mode);
701 // TODO: This Line will be removed in follow up cl that handles blending and thus we won't
702 // have to set coeffs here.
703 SkAssertResult(ModeAsCoeff(mode, src, dst));
704 return true; 700 return true;
705 } else if (xfermode->asCoeff(src, dst)) { 701 } else if (xfermode->asCoeff(&src, &dst)) {
706 *xpf = GrPorterDuffXPFactory::Create(*src, *dst); 702 *xpf = GrPorterDuffXPFactory::Create(src, dst);
707 return true; 703 return true;
708 } else if (xfermode->asXPFactory(xpf)) { 704 } else if (xfermode->asXPFactory(xpf)) {
709 *src = SkXfermode::kOne_Coeff;
710 *dst = SkXfermode::kZero_Coeff;
711 return true; 705 return true;
712 } else { 706 } else {
713 return xfermode->asFragmentProcessor(fp); 707 return xfermode->asFragmentProcessor(fp);
714 } 708 }
715 } 709 }
716 #else 710 #else
717 bool SkXfermode::AsFragmentProcessorOrXPFactory(SkXfermode* xfermode, 711 bool SkXfermode::AsFragmentProcessorOrXPFactory(SkXfermode* xfermode,
718 GrFragmentProcessor** fp, 712 GrFragmentProcessor** fp,
719 GrXPFactory** xpf, 713 GrXPFactory** xpf,
720 Coeff* src, Coeff* dst) { 714 Coeff* src, Coeff* dst) {
(...skipping 1234 matching lines...) Expand 10 before | Expand all | Expand 10 after
1955 } else { 1949 } else {
1956 proc16 = rec.fProc16_General; 1950 proc16 = rec.fProc16_General;
1957 } 1951 }
1958 } 1952 }
1959 return proc16; 1953 return proc16;
1960 } 1954 }
1961 1955
1962 SK_DEFINE_FLATTENABLE_REGISTRAR_GROUP_START(SkXfermode) 1956 SK_DEFINE_FLATTENABLE_REGISTRAR_GROUP_START(SkXfermode)
1963 SK_DEFINE_FLATTENABLE_REGISTRAR_ENTRY(SkProcCoeffXfermode) 1957 SK_DEFINE_FLATTENABLE_REGISTRAR_ENTRY(SkProcCoeffXfermode)
1964 SK_DEFINE_FLATTENABLE_REGISTRAR_GROUP_END 1958 SK_DEFINE_FLATTENABLE_REGISTRAR_GROUP_END
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698