Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(132)

Issue 181107: Demo tamed flash in the playground (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years, 6 months ago by Jasvir
Modified:
16 years, 5 months ago
Reviewers:
metaweta
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

* Splits out the taming section of the playground * Tamed swfobject * Tamed alert * Added an example on how to embed flash (without script access) * Remember previously typed urls in the playground to ease repeated use

Patch Set 1 #

Total comments: 6

Patch Set 2 : Demo tamed flash in the playground #

Patch Set 3 : Demo tamed flash in the playground #

Unified diffs Side-by-side diffs Delta from patch set Stats (+126 lines, -16 lines) Patch
M build.xml View 1 chunk +1 line, -0 lines 0 comments Download
M src/com/google/caja/demos/playground/client/Playground.java View 1 chunk +1 line, -0 lines 0 comments Download
M src/com/google/caja/demos/playground/client/ui/Example.java View 2 chunks +3 lines, -0 lines 0 comments Download
M src/com/google/caja/demos/playground/client/ui/PlaygroundView.java View 3 chunks +14 lines, -16 lines 0 comments Download
A src/com/google/caja/demos/playground/taming.js View 1 1 chunk +107 lines, -0 lines 0 comments Download

Messages

Total messages: 5
Jasvir
16 years, 6 months ago (2009-12-30 20:25:36 UTC) #1
Jasvir
ping
16 years, 6 months ago (2010-01-04 08:24:18 UTC) #2
metaweta
http://codereview.appspot.com/181107/diff/1/2 File src/com/google/caja/demos/playground/taming.js (right): http://codereview.appspot.com/181107/diff/1/2#newcode38 src/com/google/caja/demos/playground/taming.js:38: // else use whatever variant of version 9 the ...
16 years, 6 months ago (2010-01-04 17:19:56 UTC) #3
Jasvir
Snapshot http://codereview.appspot.com/181107/diff/1/2 File src/com/google/caja/demos/playground/taming.js (right): http://codereview.appspot.com/181107/diff/1/2#newcode38 src/com/google/caja/demos/playground/taming.js:38: // else use whatever variant of version 9 ...
16 years, 6 months ago (2010-01-05 18:02:41 UTC) #4
metaweta
16 years, 6 months ago (2010-01-06 22:34:21 UTC) #5
LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b