Skip to content

GitLab

Switch branch/tag
  1. 05 Jun, 2014 1 commit
  3. 04 Jun, 2014 3 commits
    • egdaniel's avatar
      Add utils gypi file · ea902d91
      egdaniel authored 
      BUG=skia:
R=bsalomon@google.com, robertphillips@google.com

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/317733003
      ea902d91
    • egdaniel's avatar
      Revert of Move Dashing filterPath to a dashing utils file... · 90b8cafb
      egdaniel authored 
      Revert of Move Dashing filterPath to a dashing utils file (https://codereview.chromium.org/314623004/)

Reason for revert:
Need to create utils.gypi file and update chrome

Original issue's description:
> Move Dashing filterPath to a dashing utils file
>
> From inside GrContext, we have a need to create an SkPath an original path and
> some dashing info. We do not have access to the original path effect so we need
> a way to make the FilterPath function accessible outside of the effect. So I moved
> the core filterPath code (and all need helper functions) out of SkDashPathEffect
> and created a SkDashPath in utils to store these helper functions.
>
> BUG=skia:
>
> Committed: https://skia.googlesource.com/skia/+/576dcdc793a762ec63fbecdbfd5768066b548fe5

R=bsalomon@google.com, reed@google.com, rmistry@google.com
TBR=bsalomon@google.com, reed@google.com, rmistry@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/317663003
      90b8cafb
    • egdaniel's avatar
      Move Dashing filterPath to a dashing utils file · 576dcdc7
      egdaniel authored 
      From inside GrContext, we have a need to create an SkPath an original path and
some dashing info. We do not have access to the original path effect so we need
a way to make the FilterPath function accessible outside of the effect. So I moved
the core filterPath code (and all need helper functions) out of SkDashPathEffect
and created a SkDashPath in utils to store these helper functions.

BUG=skia:
R=bsalomon@google.com, reed@google.com, rmistry@google.com

Author: egdaniel@google.com

Review URL: https://codereview.chromium.org/314623004
      576dcdc7
  5. 22 May, 2014 2 commits
  7. 21 May, 2014 1 commit
  9. 16 May, 2014 2 commits
  11. 08 May, 2014 2 commits
  13. 15 Apr, 2014 1 commit
  15. 21 Mar, 2014 1 commit
  17. 16 Mar, 2014 1 commit
  19. 28 Feb, 2014 1 commit
    • commit-bot@chromium.org's avatar
      DM: make GPU tasks multithreaded again. Big refactor. · ef57b7e6
      commit-bot@chromium.org authored 
      The main meat of things is in SkThreadPool.  We can now give SkThreadPool a
type for each thread to create and destroy on its local stack.  It's TLS
without going through SkTLS.

I've split the DM tasks into CpuTasks that run on threads with no TLS, and
GpuTasks that run on threads with a thread local GrContextFactory.

The old CpuTask and GpuTask have been renamed to CpuGMTask and GpuGMTask.

Upshot: default run of out/Debug/dm goes from ~45 seconds to ~20 seconds.

BUG=skia:
R=bsalomon@google.com, mtklein@google.com, reed@google.com

Author: mtklein@chromium.org

Review URL: https://codereview.chromium.org/179233005

git-svn-id: http://skia.googlecode.com/svn/trunk@13632 2bbb7eff-a529-9590-31e7-b0007b416f81
      ef57b7e6
  21. 31 Jan, 2014 1 commit
    • commit-bot@chromium.org's avatar
      initial import of Chrome's trace_event into skia framework · 6169f2b4
      commit-bot@chromium.org authored 
      This patch includes a modified version of Chrome's trace_event.h, which provides
tracing macros that can easily integrate into the about://tracing framework.

Currently the macros link to a default implementation of the (narrow) tracing
class SkDefaultEventTracer which does nothing; next step will be to have Chrome
subclass the SkEventTracer with a shim that bolts Skia's trace events to its own,
allowing Skia's trace events to show up in about://tracing.

I've verified that this file builds properly, and when I added a simple scoped
TRACE_EVENT0 to SkCanvas::drawRect, along with some debug prints in the NOP
implementation of tracing, I saw what I expected printed to the screen.

BUG=skia:
R=nduca@chromium.org, reed@google.com, mtklein@google.com, bsalomon@google.com

Author: humper@google.com

Review URL: https://codereview.chromium.org/149563004

git-svn-id: http://skia.googlecode.com/svn/trunk@13256 2bbb7eff-a529-9590-31e7-b0007b416f81
      6169f2b4
  23. 23 Jan, 2014 1 commit
  25. 17 Jan, 2014 1 commit
  27. 16 Jan, 2014 1 commit
  29. 13 Jan, 2014 1 commit
  31. 07 Jan, 2014 2 commits
  33. 02 Dec, 2013 1 commit
  35. 13 Nov, 2013 1 commit
  37. 12 Nov, 2013 1 commit
  39. 26 Sep, 2013 1 commit
  41. 18 Sep, 2013 1 commit
  43. 04 Sep, 2013 2 commits
  45. 29 Aug, 2013 3 commits
  47. 26 Jul, 2013 1 commit
  49. 22 Jul, 2013 1 commit
    • bungeman@google.com's avatar
      Update obvious deps. · b8f3655d
      bungeman@google.com authored 
      This is a step toward targets declaring their deps in a sane fashion.
This change resolves cycles by forcing core to the root, then everything
in skia_lib pointing toward core as best possible, then everything
outside skia_lib depending on skia_lib for things in skia_lib. This
prevents double definitions where a symbol is provided by both the
skia_lib shared object and and a statically linked component of skia_lib.

R=djsollen@google.com

Review URL: https://codereview.chromium.org/19823003

git-svn-id: http://skia.googlecode.com/svn/trunk@10231 2bbb7eff-a529-9590-31e7-b0007b416f81
      b8f3655d
  51. 20 Jul, 2013 1 commit
  53. 19 Jul, 2013 1 commit
    • bungeman@google.com's avatar
      Update obvious deps. · 2b0ab9e1
      bungeman@google.com authored 
      This is a step toward targets declaring their deps in a sane fashion.
This change resolves cycles by forcing core to the root,
then opts, ports, and utils depending on core, then everything else.
We will need some other change to resolve the fact that
core, opts, ports, and utils depend on each other and other targets which
depend on them. Outside of these targets, things look ok.

R=djsollen@google.com

Review URL: https://codereview.chromium.org/19823003

git-svn-id: http://skia.googlecode.com/svn/trunk@10217 2bbb7eff-a529-9590-31e7-b0007b416f81
      2b0ab9e1
  55. 12 Jul, 2013 1 commit
  57. 01 Jul, 2013 1 commit
  59. 28 Jun, 2013 1 commit
    • scroggo@google.com's avatar
      Replace SkPicture(SkStream) constructors with a factory. · f1754ec6
      scroggo@google.com authored 
      SkPicture:
Remove the constructors which take an SkStream as an argument. Rather
than having to check a variable for success, the factory will return
NULL on failure.
Add a protected function for determining if an SkStream is an SKP
to share code with SkTimedPicture.
In the factory, check for a NULL SkStream.
Use a default decoder (from BUG:
https://code.google.com/p/skia/issues/detail?id=1325)

SkDebuggerGUI:
Call SkPicture::CreateFromStream when necessary.
Write a factory for creating SkTimedPictures and use it.

Use the factory throughout tools.

Add include/lazy to utils and effects gyp include_dirs so SkPicture.h
can reference SkImageDecoder.h which references SkBitmapFactory.h (in
include/lazy).

Changes code Chromium uses, so this will require a temporary Skia
and then a change to Chromium to use the new Skia code.

TODO: Create a decoder that does nothing to be used by pinspect,
lua pictures, etc, and allow it to not assert in SkOrderedReadBuffer.

R=reed@google.com

Review URL: https://codereview.chromium.org/17113004

git-svn-id: http://skia.googlecode.com/svn/trunk@9822 2bbb7eff-a529-9590-31e7-b0007b416f81
      f1754ec6
  61. 11 Jun, 2013 1 commit