permalinks working

Now with permalinks.

Once we get a response from our XHR request update the URL to a permalink to that code.

This CL comes after https://codereview.chromium.org/235373002/

BUG=skia:

Committed: http://code.google.com/p/skia/source/detail?r=14187

[mtklein, 2014-04-11 21:45:06]

https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/DESI...
File experimental/webtry/DESIGN.md (right):

https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/DESI...
experimental/webtry/DESIGN.md:141: // If this gets changed also update the
sqlite create statement in webtry.go.
Maybe add a mode to webtry.go to do the remote DB creation as a one-off for you,
so this all can be written in one place in code?

https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/webt...
File experimental/webtry/webtry.go (right):

https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/webt...
experimental/webtry/webtry.go:119: code      TEXT      DEFAULT ''               
 NOT NULL,
Won't break anything to indent this a few tabs more to the right will it?

[jcgregorio, 2014-04-12 02:37:07]

https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/DESI...
File experimental/webtry/DESIGN.md (right):

https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/DESI...
experimental/webtry/DESIGN.md:141: // If this gets changed also update the
sqlite create statement in webtry.go.
Actually I've the got MySQL webtry user locked down so tightly that it can only
do INSERT, UPDATE and SELECT (no little bobby tables for me), so it wouldn't be
useful to have this as code in webtry for anything but the sqlite case.

At some point though this code is going to grow beyond a single file and I'm
going to have to start breaking things into modules, but that means change up
all the code to follow the GOPATH route:

   http://golang.org/doc/code.html

When that happens it might be worth revisiting this.

On 2014/04/11 21:45:06, mtklein wrote:
> Maybe add a mode to webtry.go to do the remote DB creation as a one-off for
you,
> so this all can be written in one place in code?

https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/webt...
File experimental/webtry/webtry.go (right):

https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/webt...
experimental/webtry/webtry.go:119: code      TEXT      DEFAULT ''               
 NOT NULL,
On 2014/04/11 21:45:06, mtklein wrote:
> Won't break anything to indent this a few tabs more to the right will it?

Done.

[mtklein, 2014-04-12 13:27:15]

On 2014/04/12 02:37:07, jcgregorio wrote:
>
https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/DESI...
> File experimental/webtry/DESIGN.md (right):
> 
>
https://codereview.chromium.org/235323006/diff/20001/experimental/webtry/DESI...
> experimental/webtry/DESIGN.md:141: // If this gets changed also update the
> sqlite create statement in webtry.go.
> Actually I've the got MySQL webtry user locked down so tightly that it can
only
> do INSERT, UPDATE and SELECT (no little bobby tables for me), so it wouldn't
be
> useful to have this as code in webtry for anything but the sqlite case.
> 
> At some point though this code is going to grow beyond a single file and I'm
> going to have to start breaking things into modules, but that means change up
> all the code to follow the GOPATH route:
> 
>    http://golang.org/doc/code.html
> 
> When that happens it might be worth revisiting this.

Ah, right.  lgtm

[jcgregorio, 2014-04-14 17:49:14]

The CQ bit was checked by jcgregorio@google.com

[commit-bot: I haz the power, 2014-04-14 17:49:27]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/jcgregorio@google.com/235323006/30001

[commit-bot: I haz the power, 2014-04-14 18:53:29]

Change committed as 14187