aboutsummaryrefslogtreecommitdiffhomepage
path: root/experimental
Commit message (Collapse)AuthorAge
* 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
* Revert of Revert of Fix SkTextBlob offset semantics. (patchset #1 id:1 of ↵Gravatar fmalita2014-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/609223003/) Reason for revert: Re-landing: Chromium-side fix to be landed with the roll (https://codereview.chromium.org/607853003/) Original issue's description: > Revert of Fix SkTextBlob offset semantics. (patchset #2 id:20001 of https://codereview.chromium.org/605533002/) > > Reason for revert: > Breaking the Chrome builds with the error: > > [14:54:14.317833] ../../skia/ext/pixel_ref_utils.cc:221:16: error: 'drawPosText' marked 'override' but does not override any member functions > [14:54:14.318022] virtual void drawPosText(const SkDraw& draw, > [14:54:14.318082] ^ > > Original issue's description: > > Fix SkTextBlob offset semantics. > > > > Implement proper x/y drawTextBlob() handling by plumbing a > > drawPosText() offset parameter (to act as an additional glyph pos > > translation) throughout the device layer. > > > > The new offset superceeds the existing constY, with a minor semantic > > tweak: whereas previous implementations were ignoring constY in 2D > > positioning mode (scalarsPerGlyph == 2), now the offset is always > > observed, in all positioning modes. We can do this because existing > > drawPosText() clients always pass constY == 0 for full positioning mode. > > > > R=reed@google.com, jvanverth@google.com, robertphillips@google.com > > > > Committed: https://skia.googlesource.com/skia/+/c13bc571d3e61a43b87eb97f0719abd304cafaf2 > > TBR=jvanverth@google.com,reed@google.com,bsalomon@google.com,fmalita@chromium.org > NOTREECHECKS=true > NOTRY=true > > Committed: https://skia.googlesource.com/skia/+/d46b8d2bab7cfba8458432248e1568ac377429e9 R=jvanverth@google.com, reed@google.com, bsalomon@google.com, robertphillips@google.com TBR=bsalomon@google.com, jvanverth@google.com, reed@google.com, robertphillips@google.com NOTREECHECKS=true NOTRY=true Author: fmalita@chromium.org Review URL: https://codereview.chromium.org/607413003
* Revert of Fix SkTextBlob offset semantics. (patchset #2 id:20001 of ↵Gravatar robertphillips2014-09-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://codereview.chromium.org/605533002/) Reason for revert: Breaking the Chrome builds with the error: [14:54:14.317833] ../../skia/ext/pixel_ref_utils.cc:221:16: error: 'drawPosText' marked 'override' but does not override any member functions [14:54:14.318022] virtual void drawPosText(const SkDraw& draw, [14:54:14.318082] ^ Original issue's description: > Fix SkTextBlob offset semantics. > > Implement proper x/y drawTextBlob() handling by plumbing a > drawPosText() offset parameter (to act as an additional glyph pos > translation) throughout the device layer. > > The new offset superceeds the existing constY, with a minor semantic > tweak: whereas previous implementations were ignoring constY in 2D > positioning mode (scalarsPerGlyph == 2), now the offset is always > observed, in all positioning modes. We can do this because existing > drawPosText() clients always pass constY == 0 for full positioning mode. > > R=reed@google.com, jvanverth@google.com, robertphillips@google.com > > Committed: https://skia.googlesource.com/skia/+/c13bc571d3e61a43b87eb97f0719abd304cafaf2 R=jvanverth@google.com, reed@google.com, bsalomon@google.com, fmalita@chromium.org TBR=bsalomon@google.com, fmalita@chromium.org, jvanverth@google.com, reed@google.com NOTREECHECKS=true NOTRY=true Author: robertphillips@google.com Review URL: https://codereview.chromium.org/609223003
* Fix SkTextBlob offset semantics.Gravatar Florin Malita2014-09-26
| | | | | | | | | | | | | | | | Implement proper x/y drawTextBlob() handling by plumbing a drawPosText() offset parameter (to act as an additional glyph pos translation) throughout the device layer. The new offset superceeds the existing constY, with a minor semantic tweak: whereas previous implementations were ignoring constY in 2D positioning mode (scalarsPerGlyph == 2), now the offset is always observed, in all positioning modes. We can do this because existing drawPosText() clients always pass constY == 0 for full positioning mode. R=reed@google.com, jvanverth@google.com, robertphillips@google.com Review URL: https://codereview.chromium.org/605533002
* Reinitialize monit to changes take effect.Gravatar tfarina2014-09-26
| | | | | | | | | | | | | Otherwise changes made to the configuration won't take effect when they are synced. BUG=None TEST=None R=jcgregorio@google.com Author: tfarina@chromium.org Review URL: https://codereview.chromium.org/602293002
* Fix typo in experimental/webtry/DESIGN.md.Gravatar jcgregorio2014-09-22
| | | | | | | | R=tfarina@chromium.org Author: jcgregorio@google.com Review URL: https://codereview.chromium.org/593723003
* fixes to linux build for webtryGravatar humper2014-09-18
| | | | | | | | | BUG=skia: R=jcgregorio@google.com Author: humper@google.com Review URL: https://codereview.chromium.org/578013003
* hide deviceproperties, prepare the way for surfacepropsGravatar reed2014-09-17
| | | | | | | | | | BUG=skia: NOTRY=True R=bungeman@google.com Author: reed@google.com Review URL: https://codereview.chromium.org/577023002
* gyp build of skfiddle apps, take 2Gravatar humper2014-09-17
| | | | | | | | | BUG=skia: R=jcgregorio@google.com Author: humper@google.com Review URL: https://codereview.chromium.org/553333004
* Revert "Revert "Move SkGpuDevice.h to src/gpu""Gravatar bsalomon2014-09-17
| | | | | | | | | | | This reverts commit b0a35f7c5d2c4bfeb601e3ac43f412d202a25292. R=borenet@google.com TBR=borenet@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/575783003
* Revert "Move SkGpuDevice.h to src/gpu"Gravatar borenet2014-09-17
| | | | | | | | | | | | | | | This reverts commit d99bbb61e58e8bd34db3954147ce1c9166fe4637. Causing Chrome canary failures as well as failures of Chrome trybots due to not cleaning up properly after failed DEPS roll attempts. BUG=skia: R=bsalomon@google.com, reed@google.com TBR=bsalomon, reed Author: borenet@google.com Review URL: https://codereview.chromium.org/579733003
* Move SkGpuDevice.h to src/gpuGravatar bsalomon2014-09-16
| | | | | | | | R=reed@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/545193006
* disable webtry seccmp on non-linux buildGravatar humper2014-09-09
| | | | | | | | | BUG=skia: R=jcgregorio@google.com Author: humper@google.com Review URL: https://codereview.chromium.org/555993002
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-01 07:35:32 +0000