| Commit message (Collapse) | Author | Age |
... | |
|
|
|
|
|
|
| |
BUG=skia:
TBR=jcgregorio
Review URL: https://codereview.chromium.org/688003002
|
|
|
|
|
|
|
|
|
|
|
| |
You can now select Raster, Ganesh, or both (PDF coming soon), and see
all output simultaneously.
BUG=skia:
R=jcgregorio@google.com
TBR=jcgregorio
Review URL: https://codereview.chromium.org/688433002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/661033005
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We decided last week that GPU rendering wouldn't be associated
with the fiddle itself; options in the left bar will be split into
ones that are bound to the fiddle (so appear in the hash and database)
and ones that are run-specific (like the rendering mode).
This will leave us better placed to do things like rendering in multiple
modes at once.
BUG=skia:
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/685543002
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds DrawingMethods with some of the methods it defines. Context is now
an implementation of DrawingMethods.
The sample.js file now shows how the context is used.
Not much new code here, that's mostly in DrawingMethods::DrawPath, most everything else is a code move.
BUG=skia:
Review URL: https://codereview.chromium.org/676423002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/680053002
|
|
|
|
| |
Review URL: https://codereview.chromium.org/677133002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
should have to begin with.
Also:
- SkWindow now has createSurface, not createCanvas.
- Add the platform init code v8 now seems to require.
- Fix library linkage.
- Call isolate->Enter(); because it doesn't look
like v8 starts with a default isolate to begin with.
BUG=skia:
Review URL: https://codereview.chromium.org/673223002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/665203004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/637263004
|
|
|
|
|
|
|
|
|
|
|
| |
In case they fail, lets fail with an ERROR message rather than an INFO,
and lets make the error message a little bit better.
BUG=None
TEST=None
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/669643004
|
|
|
|
|
|
|
|
|
| |
Otherwise the --log_dir flag doesn't affect where glog output goes to.
TBR=humper
BUG=skia:
Review URL: https://codereview.chromium.org/667493002
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This setup and runs a logserver, like the one being used by skiaperf, to
serve the directory /tmp/glog, so we don't need to ssh into the instance
to see the logs of the webtry application.
This is a follow up to suggestion made in codereview.chromium.org/655323002.
BUG=None
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/643903003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/644253003
|
|
|
|
|
|
|
|
|
| |
You can also click on the errors in the parsed output to jump directly to the offending location in the editor.
BUG=skia:
NOTRY=true
Review URL: https://codereview.chromium.org/660573003
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should prevent further enumeration according to the docs.
http://golang.org/pkg/database/sql/#Rows.Close
BUG=None
TEST=None
R=humper@google.com
Review URL: https://codereview.chromium.org/638373003
|
|
|
|
|
|
|
|
|
|
| |
Update it to use glog variant.
BUG=skia:2980
TEST=go build webtry.go
TBR=jcgregorio@google.com
Review URL: https://codereview.chromium.org/659813003
|
|
|
|
|
|
|
|
| |
BUG=skia:2980
TEST=go get -d && go build webtry.go
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/655323002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As mentioned in http://golang.org/doc/effective_go.html#blank_assign,
errors are provided for a reason, so always check them.
The documentation for this function is available at
http://golang.org/pkg/os/#Chdir.
BUG=None
TEST=None
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/651343002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
text/template and html/template both have a helper function called
"Must" that according to the docs in http://golang.org/pkg/text/template/#Must
and http://golang.org/pkg/html/template/#Must, wraps a call to a function
returning (*Template, error) and panics if the error is non-nil.
So we don't need to that ourselves. Which is a good thing and makes code
cleaner.
BUG=None
TEST=None
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/642243004
|
|
|
|
|
|
|
|
|
| |
space available to a fiddle because linking against libGL causes the
amount of memory used by a process to bloat to over 100M. Yikes.
BUG=skia:
Review URL: https://codereview.chromium.org/638003008
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
of https://codereview.chromium.org/649313002/)
Reason for revert:
adding -Dskia_mesa=1 broke production; don't yet know why and don't want to leave things in a broken state.
Original issue's description:
> add ganesh checkbox to skfiddle interface
>
> BUG=skia:
> R=mtklein@google.com, tfarina@chromium.org
>
> Committed: https://skia.googlesource.com/skia/+/ccb54895c614632fd01d22b64ab614921f980eb1
TBR=tfarina@chromium.org,jcgregorio@google.com,mtklein@google.com
NOTREECHECKS=true
NOTRY=true
BUG=skia:
Review URL: https://codereview.chromium.org/655633003
|
|
|
|
|
|
|
| |
BUG=skia:
R=mtklein@google.com, tfarina@chromium.org
Review URL: https://codereview.chromium.org/649313002
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=NOne
R=humper@google.com
TBR=humper@google.com
Review URL: https://codereview.chromium.org/648563003
|
|
|
|
|
|
|
| |
BUG=skia:
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/656463002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
That seems to make things more tied and that seems to be where we have
the same scripts/configs in the perf directory:
https://skia.googlesource.com/buildbot/+/master/perf/setup/sys/
That way we keep all files used/related to setting up a new webtry server in one place.
BUG=None
TEST=None
R=humper@google.com
Review URL: https://codereview.chromium.org/644513003
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=jcgregorio
Review URL: https://codereview.chromium.org/637353003
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The previous version of the script didn't properly update the code for the webtry
user, so only changes to skia would really be pulled, not changes to the actual
webtry server itself.
This version of the setup script is 100% automated and brings everything up to date.
It's also a little more verbose about what it's doing, so it's easier to debug.
BUG=skia:
R=tfarina@chromium.org
TBR=jcgregorio, mtklein, tfarina
Review URL: https://codereview.chromium.org/636173003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/639013002
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=jcgregorio
Review URL: https://codereview.chromium.org/635373002
|
|
|
|
|
|
|
| |
BUG=skia:2998
TBR=jcgregorio, tfarina
Review URL: https://codereview.chromium.org/639833003
|
|
|
|
|
|
| |
Mainly just to familiarize myself with this code.
Review URL: https://codereview.chromium.org/633183002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/628353002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/623173004
|
|
|
|
|
|
|
| |
NOTRY=True
TBR=
Review URL: https://codereview.chromium.org/629903004
|
|
|
|
|
|
|
| |
BUG=skia:
TBR=tfarina
Review URL: https://codereview.chromium.org/626623002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/631693003
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/626033004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/630093002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/624973004
|
|
|
|
|
|
|
|
|
|
| |
Also add webtry.db to gitignore so it does not polute `git status`.
BUG=None
TEST=None
R=humper@google.com
Review URL: https://codereview.chromium.org/626703002
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/626443004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/618013004
|
|
|
|
|
|
| |
BUG=skia:
Review URL: https://codereview.chromium.org/619943002
|
|
|
|
|
|
|
|
|
|
| |
Puts it much more in sync with the README in perf/.
BUG=None
TEST=None
R=jcgregorio@google.com
Review URL: https://codereview.chromium.org/615313002
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Otherwise it will through an error like the following:
2014/09/28 21:22:24 Info: status creating sqlite table for sources:
"table source_images already exists"
2014/09/28 21:22:24 Info: status creating sqlite table for webtry:
"table webtry already exists"
2014/09/28 21:22:24 Info: status creating sqlite table for workspace:
"table workspace already exists"
2014/09/28 21:22:24 Info: status creating sqlite table for workspace
try: "table workspacetry already exists"
To test locally the following was done:
$ ./gyp_skia gyp/webtry.gyp gyp/most.gyp -Dskia_gpu=0
$ ninja -C out/Debug webtry
$ cd experimental/webtry
$ go get -d
$ go build webtry.go
$ ./webtry
$ google-chrome http://localhost:8000
Expected: see no more the above messages.
BUG=None
TEST=see above
R=stephana@google.com, jcgregorio@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/613593002
|
|
|
|
|
|
|
|
|
|
|
|
| |
Looks like the name of the directory is "poly" rather than "polyfill".
BUG=None
TEST=None
R=humper@google.com, jcgregorio@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/610003003
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is necessary to build webtry.go as it imports packages from
github.com
BUG=None
TEST=follow the README instructions
R=humper@google.com, jcgregorio@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/601033004
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix the wording in the DESIGN doc. Currently it says "the only table" as
implying the database has just a single table.
That is not true, the webtry database has four tables: webtry,
workspace, workspacetry and source_images.
BUG=None
TEST=None
R=jcgregorio@google.com
Author: tfarina@chromium.org
Review URL: https://codereview.chromium.org/611763002
|
|
|
|
|
|
|
|
|
| |
BUG=skia:
R=jcgregorio@google.com
Author: humper@google.com
Review URL: https://codereview.chromium.org/601203002
|