AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
14,923 Commits 1 Branch 0 Tags 544 MiB
a06a953121
Commit Graph

14923 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
mtklein
a06a953121 Prune SkRTree 
- Propagate a bunch of constant parameters through.
  - Delete code that's not used when bulk loading.
  - Allocate all Nodes together.
  - Stay in SkRect.

Doing a single malloc for the nodes can't not have improved memory usage.

Looks like this might improve record performance ~5%, probably mostly from
staying in SkRects.  This finally dethrones building the BBH as the hot spot.
(Now it's mapping user bounds back to device bounds and adjusting for paints.)

Recording time changes from my MBP:
    desk_rectangletransition.skp	11.5us -> 11.7us	1x
             desk_forecastio.skp	 115us ->  114us	0.98x
                desk_booking.skp	 550us ->  541us	0.98x
            tabl_mercurynews.skp	 176us ->  173us	0.98x
                   tabl_hsfi.skp	 294us ->  287us	0.98x
              desk_wordpress.skp	 351us ->  343us	0.98x
           tabl_worldjournal.skp	 439us ->  426us	0.97x
                  tabl_gmail.skp	20.3us -> 19.7us	0.97x
         desk_youtubetvvideo.skp	10.8us -> 10.4us	0.97x
             desk_googleplus.skp	 1.1ms -> 1.07ms	0.97x
               tabl_slashdot.skp	 106us ->  103us	0.97x
         desk_jsfiddlebigcar.skp	26.7us -> 25.7us	0.96x
               tabl_techmeme.skp	95.4us -> 91.7us	0.96x
             tabl_deviantart.skp	 133us ->  127us	0.96x
              desk_pinterest.skp	40.6us -> 38.9us	0.96x
                 desk_carsvg.skp	 195us ->  187us	0.96x
               tabl_engadget.skp	 376us ->  359us	0.96x
                tabl_sahadan.skp	60.5us -> 57.5us	0.95x
      tabl_culturalsolutions.skp	 255us ->  242us	0.95x
                  tabl_gspro.skp	58.3us -> 55.5us	0.95x
               desk_linkedin.skp	 146us ->  138us	0.94x
                   desk_ebay.skp	 192us ->  181us	0.94x
                    tabl_cnn.skp	 467us ->  440us	0.94x
     desk_jsfiddlehumperclip.skp	29.9us -> 28.1us	0.94x
               desk_tigersvg.skp	43.2us -> 40.5us	0.94x
           desk_yahooanswers.skp	 131us ->  123us	0.94x
desk_googlespreadsheetdashed.skp	1.18ms -> 1.11ms	0.94x
                desk_blogger.skp	 193us ->  181us	0.94x
                tabl_mozilla.skp	1.82ms ->  1.7ms	0.94x
                    tabl_mlb.skp	 145us ->  136us	0.93x
              mobi_wikipedia.skp	 577us ->  539us	0.93x
               tabl_frantzen.skp	54.1us -> 50.4us	0.93x
                  desk_baidu.skp	87.9us -> 81.9us	0.93x
             desk_techcrunch.skp	 224us ->  209us	0.93x
                 desk_sfgate.skp	 206us ->  192us	0.93x
                  tabl_ukwsj.skp	 269us ->  250us	0.93x
               desk_facebook.skp	 316us ->  293us	0.93x
            desk_gmailthread.skp	 205us ->  190us	0.93x
         tabl_googlecalendar.skp	 158us ->  147us	0.93x
                   tabl_digg.skp	 382us ->  354us	0.93x
                 desk_amazon.skp	 106us -> 98.5us	0.93x
          tabl_androidpolice.skp	 693us ->  642us	0.93x
                tabl_nytimes.skp	 206us ->  191us	0.92x
                    desk_gws.skp	 124us ->  114us	0.92x
                desk_youtube.skp	 255us ->  235us	0.92x
           tabl_cuteoverload.skp	 583us ->  537us	0.92x
            desk_oldinboxapp.skp	  18us -> 16.6us	0.92x
             desk_mobilenews.skp	 297us ->  273us	0.92x
                 tabl_pravda.skp	 168us ->  154us	0.92x
              tabl_vnexpress.skp	 236us ->  217us	0.92x
          desk_css3gradients.skp	 202us ->  185us	0.92x
            tabl_gamedeksiam.skp	 508us ->  464us	0.91x
                desk_wowwiki.skp	1.02ms ->  929us	0.91x
                   desk_espn.skp	 209us ->  191us	0.91x
             desk_chalkboard.skp	 315us ->  284us	0.9x
                 desk_mapsvg.skp	 607us ->  543us	0.89x
            desk_pokemonwiki.skp	5.18ms -> 4.62ms	0.89x
               desk_samoasvg.skp	 335us ->  298us	0.89x
        desk_youtubetvbrowse.skp	10.1us -> 8.59us	0.85x
BUG=skia:3085, skia:2834

Review URL: https://codereview.chromium.org/734723002
 2014-11-18 09:27:49 -08:00
mtklein
52b7822fa6 Revert of SkVarAlloc::approxBytesAllocated (patchset #5 id:80001 of https://codereview.chromium.org/730193003/) 
Reason for revert:
Android needs dlmalloc_usable_size().

Original issue's description:
> SkVarAlloc::approxBytesAllocated()
>
> This is what I was getting at on the other CL.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/f27f1bcce50c8f95aea8469684a70b70c9baee09
>
> CQ_EXTRA_TRYBOTS=Test-Android-Nexus5-Adreno330-Arm7-Release-Trybot

TBR=reed@google.com,mtklein@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Review URL: https://codereview.chromium.org/741443002
 2014-11-18 08:15:23 -08:00
bsalomon
6d467ecb81 minor cleanup in GrGpu.h 
Review URL: https://codereview.chromium.org/731073003
 2014-11-18 07:36:19 -08:00
reed
611f3bb4cf rebaseline perspective gm after matrix dont-renormalize-cl 
BUG=skia:
TBR=
NOTRY=True

Review URL: https://codereview.chromium.org/741433002
 2014-11-18 07:32:26 -08:00
mtklein
f27f1bcce5 SkVarAlloc::approxBytesAllocated() 
This is what I was getting at on the other CL.

BUG=skia:

Review URL: https://codereview.chromium.org/730193003
 2014-11-18 07:24:14 -08:00
reed
9b1dd15d6d don't normalize after perspective concat 
BUG=skia:

Review URL: https://codereview.chromium.org/734513004
 2014-11-18 06:04:37 -08:00
kkinnunen
542ecbb2bd Remove unused NVPR related GL tokens 
Remove unused NVPR related GL tokens.

Also replace GR_GL_PATH_INITIAL_END_CAP and GR_GL_PATH_TERMINAL_END_CAP
with a single call setting GR_GL_PATH_END_CAPS. Skia does not and
probably will not have different initial and terminal caps. This came up
in the review of command buffer implementation of the extension.

If more NVPR features will be used, the respective tokens can be added
back per implemented feature.

Review URL: https://codereview.chromium.org/723453002
 2014-11-18 05:13:48 -08:00
kkinnunen
b33402bc87 Do not calculate many sierpinski fractals for each nanobench run unless needed 
Removes work done by the constructors of picture_nesting benches,
and moves the work to the Benchmark::onPreDraw override.

This avoids PictureNesting::sierpinsky showing up in profile traces
when profiling other benches.

Review URL: https://codereview.chromium.org/725523002
 2014-11-18 04:50:50 -08:00
skia.buildbots
a8f38238d6 Update SKP version 
Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/733323002
 2014-11-17 23:03:57 -08:00
reed
a43da3ebfe more baselines for blur2rects 
BUG=skia:
TBR=
NOTRY=True

Review URL: https://codereview.chromium.org/715223003
 2014-11-17 18:24:34 -08:00
joshualitt
fc63f66a31 one more rebase on k1 
TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/734243002
 2014-11-17 17:59:49 -08:00
joshualitt
72f7e09ece rebase nvpr after ds on stack 
TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/726103004
 2014-11-17 16:48:13 -08:00
joshualitt
9853ccef19 Drawstate on stack 
BUG=skia:

Review URL: https://codereview.chromium.org/732693002
 2014-11-17 14:22:49 -08:00
reed
b079ec682d baselines for blur2rects 
BUG=skia:
TBR=
NOTRY=True

Review URL: https://codereview.chromium.org/733233002
 2014-11-17 13:53:15 -08:00
bsalomon
b436ed6754 Reduce the amount of validation in GrResourceCache2 
BUG=chromium:433711

Review URL: https://codereview.chromium.org/737533002
 2014-11-17 12:15:57 -08:00
robertphillips
f52a063dfd Fix yet another convexicator issue 
This CL fixes the case where a bad initial vector (i.e., nearly zero) managed to short circuit all of the convexicator's logic. The initial bad vector would become the last vector and then never get displaced.

The history of this is:

https://codereview.chromium.org/298973004/
Switched the convexicator to not advance the last vector when the cross product wasn't significant

https://codereview.chromium.org/573763002/
Fixed a bug (crbug.com/412640) wherein a zero area path was being incorrectly categorized as convex b.c. opposite but equal vectors were not signaling concavity.

BUG=433683

Review URL: https://codereview.chromium.org/727283003
 2014-11-17 12:11:42 -08:00
scroggo
a18cc77c33 Really enable deprecation warnings for Android framework. 
Left out of a previous CL to reenable.

(We had the define added in two places for some reason...)

Review URL: https://codereview.chromium.org/728503002
 2014-11-17 10:29:18 -08:00
borenet
0f881c64a3 Remove bug_chomper 
It has been moved into buildbot repo.

BUG=skia:

Review URL: https://codereview.chromium.org/731123003
 2014-11-17 09:59:53 -08:00
bsalomon
84c8e62fad Allow GPU resources to not be counted against the cache budget. 
BUG=skia:2889

Review URL: https://codereview.chromium.org/721353002
 2014-11-17 09:33:27 -08:00
bsalomon
bd27e514e2 Rebase GM on win7 after res cache change. 
TBR=

Review URL: https://codereview.chromium.org/733133003
 2014-11-17 08:54:04 -08:00
robertphillips
8910879098 Make BBH parameter to SkRecordComputeLayers and SkRecordFillBounds optional 
Chromium creates a picture to contain their picture pile in order to use MultiPictureDraw. They currently do not create a bounding box for that picture but I still need layer information for it. This change allows Chromium to continue without a BBH but still have layer information.

In the future, the brute force BBH might be suitable for their use case.

Please see gpu_raster_worker_pool.cc in (Add flag to beginRecording to request saveLayer information - https://codereview.chromium.org/721883002/) for where this is happening in Chromium.

Review URL: https://codereview.chromium.org/733963004
 2014-11-17 08:16:15 -08:00
egdaniel
912b3d24a0 Use DrawStates ProcOptInfo when creating the Optimized Draw State. 
BUG=skia:

Review URL: https://codereview.chromium.org/726693002
 2014-11-17 07:45:53 -08:00
reed
468b6b4eaf tweak gm to show fractional offsets 
BUG=skia:
TBR=

Review URL: https://codereview.chromium.org/728403002
 2014-11-17 07:41:35 -08:00
bsalomon
dace19ec17 Correct accounting for wrapped resources 
BUG=skia:2889

Review URL: https://codereview.chromium.org/720033004
 2014-11-17 07:34:06 -08:00
mtklein
c6ad9eefa0 More SkPicture cleanup 
- move field declarations together and pack them a little tighter
 - get rid of fData
 - remove dead code in debugger, including unused SkPicturePlayback subclass

There are now no more long-lived SkPictureData!  (Really, there never were,
but now we don't pretend to support them.)

BUG=skia:

No API changes.
TBR=reed@google.com

Review URL: https://codereview.chromium.org/725143002
 2014-11-17 06:45:18 -08:00
Greg Humphreys
21c771b3fc Revert "move SkPDFD*.h from include to src" 
This reverts commit 693024300f.

was breaking canary build

BUG=skia:

Review URL: https://codereview.chromium.org/731173002
 2014-11-17 09:42:09 -05:00
halcanary
693024300f move SkPDFD*.h from include to src 
BUG=278148

Review URL: https://codereview.chromium.org/727343002
 2014-11-17 06:26:21 -08:00
reed
73c2501120 separate c headers 
BUG=skia:

Review URL: https://codereview.chromium.org/728323002
 2014-11-17 06:15:42 -08:00
sugoi
1379b87ae4 Added missing validation 
BUG=433359

Review URL: https://codereview.chromium.org/729583002
 2014-11-17 05:45:55 -08:00
skia.buildbots
0a81f1ee0e Update SKP version 
Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/731003004
 2014-11-16 22:31:01 -08:00
halcanary
a27ef58a50 Move SkDocument_PDF.cpp into pdf.gypi - now standard PDF API 
TBR=reed@google.com

Review URL: https://codereview.chromium.org/714223002
 2014-11-16 08:26:54 -08:00
skia.buildbots
9f61eff52f Update SKP version 
Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/734493002
 2014-11-15 22:52:04 -08:00
halcanary
21041e0cc4 http://skbug.com/3130 step 1 (to fix chromium roll) 
BUG=skia:3130
TBR=djsollen@google.com, bsalomon@google.com

Review URL: https://codereview.chromium.org/729173002
 2014-11-15 06:11:18 -08:00
skia.buildbots
f1102e46ff Update SKP version 
Automatic commit by the RecreateSKPs bot.

TBR=

Review URL: https://codereview.chromium.org/730833002
 2014-11-14 22:56:34 -08:00
halcanary
6dd03c899f Fix Chromium roll broken by http://crrev.com/728823002 
Review URL: https://codereview.chromium.org/727783004
 2014-11-14 17:22:05 -08:00
joshualitt
5478d427c6 Push creation of default GP to the caller 
TBR=
BUG=skia:

Review URL: https://codereview.chromium.org/715903002
 2014-11-14 16:00:38 -08:00
scroggo
28562a0199 Fix Android framework build. 
When we define SK_BUILD_FOR_ANDROID, we don't define it to anything,
so

    #if SK_BUILD_FOR_ANDROID

is not a meaningful check. Instead, use #ifdef.

TBR=caryclark@google.com, djsollen@google.com

Review URL: https://codereview.chromium.org/728013002
 2014-11-14 14:49:34 -08:00
reed
64c0488270 disable gammatext gm for a while, looking for other flakes 
BUG=skia:

Review URL: https://codereview.chromium.org/725303002
 2014-11-14 14:39:29 -08:00
bsalomon
82b1d62013 Guard high water cache stats vars by GR_CACHE_STATS 
TBR=halcanary@google.com
NOTRY=true

Review URL: https://codereview.chromium.org/729753003
 2014-11-14 13:59:57 -08:00
bsalomon
12299ab7a1 Make GrResourceCache2 responsible for calling release, abandon, and ~. 
BUG=skia:2889

TBR=robertphillips@google.com

NOTRY=true

Review URL: https://codereview.chromium.org/729683002
 2014-11-14 13:33:09 -08:00
halcanary
85588344c3 SkDocument SK_API 
Review URL: https://codereview.chromium.org/726073002
 2014-11-14 13:24:18 -08:00
bsalomon
71cb0c241e Replace GrResourceCache with GrResourceCache2. 
BUG=skia:2889

Committed: https://skia.googlesource.com/skia/+/66a450f21a3da174b7eed89a1d5fc8591e8b6ee6

Committed: https://skia.googlesource.com/skia/+/407aa584d183c1bf314f5defd1cf0202e8a96c89

Review URL: https://codereview.chromium.org/716143004
 2014-11-14 12:10:15 -08:00
reed
ae760d579c update gm to place inner rect on fractional boundary 
BUG=skia:
TBR=

Review URL: https://codereview.chromium.org/730673002
 2014-11-14 11:59:10 -08:00
bsalomon
191bcc008e Fix ref-cnting bug in GrContext::readRenderTargetPixels 
TBR=egdaniel@google.com
NOTRY=true

Review URL: https://codereview.chromium.org/732713002
 2014-11-14 11:31:13 -08:00
djsollen
c87dd2ce96 Enable unused param checking for public includes. 
This CL cleans up the existing violations and enables the
build time check to ensure that we don't regress.

The motiviation behind this change is to allow clients who include
our headers to be able to build with this warning enabled.

Review URL: https://codereview.chromium.org/726923002
 2014-11-14 11:11:46 -08:00
djsollen
33068c19f1 Enforce pure virtual behavior for onMatchFamilyStyleCharacter 
All impls that previously did not define the functions have been
updated to return NULL.

NOTRY=true

Review URL: https://codereview.chromium.org/728873002
 2014-11-14 10:52:53 -08:00
mtklein
d4897591fd Don't pass null to memcmp. 
This will allow us to add nonnull-attribute to the UBSAN bot.

We are in fact hitting a case where one of the arguments is null and the other
not, which seems dicey.  I think the scenario is comparing the empty pathref
with another path ref that's just been COWed, without any verbs or points yet.

BUG=skia:

Review URL: https://codereview.chromium.org/732643002
 2014-11-14 09:22:40 -08:00
reed
36e6e266b8 add gm for 2-rects and blur 
BUG=skia:
TBR=

Review URL: https://codereview.chromium.org/728863002
 2014-11-14 08:27:40 -08:00
qiankun.miao
f04713d9c8 Optimize SkAlphaMulQ_SSE2 
These two mask clear are useless, because _mm_srli_epi16 fills high byte
of each word with 0.

BUG=skia:

Review URL: https://codereview.chromium.org/724333003
 2014-11-14 08:23:53 -08:00
mtklein
c7e55e410c Steal ASAN skimage expectations for UBSAN. 
NOTRY=true

BUG=skia:3096

Review URL: https://codereview.chromium.org/731553002
 2014-11-14 07:50:51 -08:00