aboutsummaryrefslogtreecommitdiffhomepage
path: root/experimental
Commit message (Collapse)AuthorAge
...
* fix issues with simultaneous configs and security jail on skfiddleGravatar Greg Humphreys2014-10-29
| | | | | | | BUG=skia: TBR=jcgregorio Review URL: https://codereview.chromium.org/688003002
* rework GUI, scripts, and fiddle main for multiple simultaneous configsGravatar Greg Humphreys2014-10-29
| | | | | | | | | | | 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
* SkV8Sample: Now with Path2D and Path2DBuilder.Gravatar jcgregorio2014-10-29
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/661033005
* remove the GPU checkbox from the hash / databaseGravatar Greg Humphreys2014-10-28
| | | | | | | | | | | | | | | 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
* Start moving to the new canvas structure.Gravatar jcgregorio2014-10-27
| | | | | | | | | | | | | 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
* Fixup references to the js files from the HTML files in SkV8Example.Gravatar jcgregorio2014-10-27
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/680053002
* Move .js files to their own directoryGravatar robertphillips2014-10-24
| | | | Review URL: https://codereview.chromium.org/677133002
* Doing the "using namespace" penance by adding in v8:: everywhere, like I ↵Gravatar jcgregorio2014-10-24
| | | | | | | | | | | | | | | 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
* move c api into includeGravatar reed2014-10-24
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/665203004
* fix some bit-rot in the ios port of sampleappGravatar reed2014-10-22
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/637263004
* webtry: Improve error handling/message when creating sqlite tables.Gravatar tfarina2014-10-22
| | | | | | | | | | | 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
* Move init() to Init() so that flag.Parse() can be called first.Gravatar jcgregorio2014-10-20
| | | | | | | | | Otherwise the --log_dir flag doesn't affect where glog output goes to. TBR=humper BUG=skia: Review URL: https://codereview.chromium.org/667493002
* webtry: Add logserver to serve /tmp/glog.Gravatar tfarina2014-10-17
| | | | | | | | | | | | | 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
* fix display of fiddles that DON'T have compile errorsGravatar Greg Humphreys2014-10-16
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/644253003
* Properly parse compiler errors for fiddles and mark them in the editor.Gravatar humper2014-10-16
| | | | | | | | | 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
* webtry: Do not forget the close the rows.Gravatar tfarina2014-10-16
| | | | | | | | | | | | 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
* webtry: Forgot onr log.Fatal.Gravatar Thiago Farina2014-10-15
| | | | | | | | | | 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
* webtry: Switch from Go's log package to golang/glog.Gravatar tfarina2014-10-15
| | | | | | | | BUG=skia:2980 TEST=go get -d && go build webtry.go R=jcgregorio@google.com Review URL: https://codereview.chromium.org/655323002
* webtry: Check the result of os.Chdir().Gravatar tfarina2014-10-15
| | | | | | | | | | | | | | 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
* webtry: Simplify template construction.Gravatar tfarina2014-10-14
| | | | | | | | | | | | | | | | 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
* Re-land ganesh checkbox. I needed to increase the size of the addressGravatar Greg Humphreys2014-10-13
| | | | | | | | | 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
* Revert of add ganesh checkbox to skfiddle interface (patchset #6 id:270001 ↵Gravatar humper2014-10-13
| | | | | | | | | | | | | | | | | | | | | | 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
* add ganesh checkbox to skfiddle interfaceGravatar Greg Humphreys2014-10-13
| | | | | | | BUG=skia: R=mtklein@google.com, tfarina@chromium.org Review URL: https://codereview.chromium.org/649313002
* Fix webtry setup once instructions in README doc.Gravatar tfarina2014-10-13
| | | | | | | | | BUG=None TEST=NOne R=humper@google.com TBR=humper@google.com Review URL: https://codereview.chromium.org/648563003
* add support for skfiddle width/height optionsGravatar Greg Humphreys2014-10-13
| | | | | | | BUG=skia: R=jcgregorio@google.com Review URL: https://codereview.chromium.org/656463002
* webtry: Simply move sys/ directory into setup/.Gravatar tfarina2014-10-08
| | | | | | | | | | | | | | 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
* pre-build the resources library for the fiddle jailGravatar Greg Humphreys2014-10-08
| | | | | | | BUG=skia: TBR=jcgregorio Review URL: https://codereview.chromium.org/637353003
* repair the webtry setup processGravatar Greg Humphreys2014-10-08
| | | | | | | | | | | | | | | | 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
* allow fiddles to use the futex system call; fixes fonts finallyGravatar Greg Humphreys2014-10-08
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/639013002
* In the generated fiddle, set the portable fonts flag first.Gravatar Greg Humphreys2014-10-08
| | | | | | | BUG=skia: TBR=jcgregorio Review URL: https://codereview.chromium.org/635373002
* preliminary support for fiddle font useGravatar Greg Humphreys2014-10-08
| | | | | | | BUG=skia:2998 TBR=jcgregorio, tfarina Review URL: https://codereview.chromium.org/639833003
* Implement C path functionsGravatar robertphillips2014-10-08
| | | | | | Mainly just to familiarize myself with this code. Review URL: https://codereview.chromium.org/633183002
* fix missing webtry output capture displayGravatar Greg Humphreys2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/628353002
* rework webtry css with compass and bootstrapGravatar Greg Humphreys2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/623173004
* implement some so we can test itGravatar reed2014-10-06
| | | | | | | NOTRY=True TBR= Review URL: https://codereview.chromium.org/629903004
* a few more tweaks to the webtry setup docGravatar humper2014-10-06
| | | | | | | BUG=skia: TBR=tfarina Review URL: https://codereview.chromium.org/626623002
* add environment variable to webtry local run documentationGravatar Greg Humphreys2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/631693003
* Automatically resize the webtry text editor to fit the contentGravatar Greg Humphreys2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/626033004
* fix webtry local run documentationGravatar Greg Humphreys2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/630093002
* experimental c apiGravatar reed2014-10-06
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/624973004
* Restart webtry to pick up changes.Gravatar tfarina2014-10-03
| | | | | | | | | | 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
* Update webtry setup docs to reflect new installation procedureGravatar humper2014-10-02
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/626443004
* fix sudo redirection in setup scriptGravatar humper2014-10-02
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/618013004
* complete overhaul of webtry work partition between jail and hostGravatar humper2014-10-01
| | | | | | BUG=skia: Review URL: https://codereview.chromium.org/619943002
* webtry: More README doc cleanups.Gravatar tfarina2014-10-01
| | | | | | | | | | 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
* webtry: Only create sqlite3 tables if they don't exist yet.Gravatar tfarina2014-09-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* Fix some references to non-existant polyfill directory.Gravatar tfarina2014-09-29
| | | | | | | | | | | | 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
* Mention 'go get' command in the README file.Gravatar tfarina2014-09-29
| | | | | | | | | | | | | 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
* webtry database has more than one table.Gravatar tfarina2014-09-29
| | | | | | | | | | | | | | | | 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
* run the webtry build script instead of using go buildGravatar humper2014-09-29
| | | | | | | | | BUG=skia: R=jcgregorio@google.com Author: humper@google.com Review URL: https://codereview.chromium.org/601203002