add support for skfiddle width/height options

add support for skfiddle width/height options

BUG=skia:
R=jcgregorio@google.com

Committed: https://skia.googlesource.com/skia/+/6c07907eaa464ba98b0144ff20ea6989238b6253

[humper, 2014-10-13 15:32:24]

humper@google.com changed reviewers:
+ jcgregorio@google.com, tfarina@chromium.org

[humper, 2014-10-13 15:33:40]

PTAL -- this works on my local instance; will need to update the Cloud SQL
database schema before we can deploy.

Will also require deleting any local webtry.db instances (or manually updating
their schema) in order to continue to develop / test locally.

[tfarina, 2014-10-13 15:50:51]

just minor comments below and one question.

We may need to implement a migration step for the database?

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
File experimental/webtry/templates/content.html (right):

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
experimental/webtry/templates/content.html:1: 
while here, lets remove this blank line.

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
experimental/webtry/templates/content.html:5: <button id="" class=source><img
width=64 height=64 src=''></button>
lets indent 2 spaces here two?

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
File experimental/webtry/templates/workspace.html (right):

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
experimental/webtry/templates/workspace.html:34: <form action='.' method='POST'>
can we use " " instead of ' ' throughout?

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/webtry.go
File experimental/webtry/webtry.go (right):

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/webtry.g...
experimental/webtry/webtry.go:258: width     			INTEGER   DEFAULT 256           
    NOT NULL,
looks like we need a solution like the versioned db Stephan did for perf/golden?
The 'migratedb' tool.

[humper, 2014-10-13 16:44:06]

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
File experimental/webtry/templates/content.html (right):

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
experimental/webtry/templates/content.html:1: 
On 2014/10/13 15:50:50, tfarina wrote:
> while here, lets remove this blank line.

Done.

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
experimental/webtry/templates/content.html:5: <button id="" class=source><img
width=64 height=64 src=''></button>
On 2014/10/13 15:50:50, tfarina wrote:
> lets indent 2 spaces here two?

Done.

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
File experimental/webtry/templates/workspace.html (right):

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/template...
experimental/webtry/templates/workspace.html:34: <form action='.' method='POST'>
On 2014/10/13 15:50:50, tfarina wrote:
> can we use " " instead of ' ' throughout?

Will do -- I've been cleaning up some HTML stuff as I go.

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/webtry.go
File experimental/webtry/webtry.go (right):

https://codereview.chromium.org/656463002/diff/1/experimental/webtry/webtry.g...
experimental/webtry/webtry.go:258: width     			INTEGER   DEFAULT 256           
    NOT NULL,
On 2014/10/13 15:50:50, tfarina wrote:
> looks like we need a solution like the versioned db Stephan did for
perf/golden?
> The 'migratedb' tool.

Will consult with joe/stephan about this

[jcgregorio, 2014-10-13 17:48:23]

LGTM

[humper, 2014-10-13 17:58:13]

Committed patchset #3 (id:230001) manually as
6c07907eaa464ba98b0144ff20ea6989238b6253.