Repository: wicket
Updated Branches:
  refs/heads/WICKET-5819_Improvements bf881a3eb -> 7343727ce


WICKET-5819 Javadoc for Video, Audio, MediaComponent, Source, Track

Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/7343727c
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/7343727c
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/7343727c

Branch: refs/heads/WICKET-5819_Improvements
Commit: 7343727cee68ee60390339ca5a907f73e86ee5f6
Parents: bf881a3
Author: Tobias Soloschenko <[email protected]>
Authored: Mon May 25 06:58:17 2015 +0200
Committer: Tobias Soloschenko <[email protected]>
Committed: Mon May 25 06:58:17 2015 +0200

----------------------------------------------------------------------
 .../apache/wicket/markup/html/image/Image.java  |  50 ++++++--
 .../apache/wicket/markup/html/image/Source.java |  45 ++++++-
 .../markup/html/media/MediaComponent.java       | 122 ++++++++++++++++---
 .../apache/wicket/markup/html/media/Source.java |  83 ++++++++++++-
 .../apache/wicket/markup/html/media/Track.java  | 102 +++++++++++++++-
 .../wicket/markup/html/media/audio/Audio.java   | 100 ++++++++++++++-
 .../wicket/markup/html/media/video/Video.java   | 100 ++++++++++++++-
 7 files changed, 552 insertions(+), 50 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/7343727c/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Image.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Image.java 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Image.java
index fdf1f76..f9484ed 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Image.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Image.java
@@ -59,17 +59,33 @@ public class Image extends WebComponent implements 
IResourceListener
         * @see {@link #setCrossOrigin(Cors)}
         */
        public enum Cors {
+               /**
+                * no authentication required
+                */
                ANONYMOUS("anonymous"),
+               /**
+                * user credentials required
+                */
                USE_CREDENTIALS("user-credentials"),
+               /**
+                * no cross origin
+                */
                NO_CORS("");
 
                private final String realName;
 
-               private Cors(String realName) {
+               private Cors(String realName)
+               {
                        this.realName = realName;
                }
 
-               public String getRealName() {
+               /**
+                * Gets the real name for the cors option
+                * 
+                * @return the real name
+                */
+               public String getRealName()
+               {
                        return realName;
                }
        }
@@ -247,8 +263,11 @@ public class Image extends WebComponent implements 
IResourceListener
        /**
         * @param resourceReference
         *            The resource reference to set.
+        * @param parameters
+        *            the parameters to be applied to the localized image 
resource
         */
-       public void setImageResourceReference(final ResourceReference 
resourceReference, final PageParameters parameters)
+       public void setImageResourceReference(final ResourceReference 
resourceReference,
+               final PageParameters parameters)
        {
                if (localizedImageResource != null)
                {
@@ -385,7 +404,8 @@ public class Image extends WebComponent implements 
IResourceListener
                buildSizesAttribute(tag);
 
                Cors crossOrigin = getCrossOrigin();
-               if (crossOrigin != null && Cors.NO_CORS != crossOrigin) {
+               if (crossOrigin != null && Cors.NO_CORS != crossOrigin)
+               {
                        tag.put("crossOrigin", crossOrigin.getRealName());
                }
        }
@@ -414,9 +434,8 @@ public class Image extends WebComponent implements 
IResourceListener
                        // If there are xValues set process them in the applied 
order to the srcset attribute.
                        if (xValues != null)
                        {
-                               xValue = xValues.size() > srcSetPosition &&
-                                       xValues.get(srcSetPosition) != null ? " 
" +
-                                       xValues.get(srcSetPosition) : "";
+                               xValue = xValues.size() > srcSetPosition && 
xValues.get(srcSetPosition) != null
+                                       ? " " + xValues.get(srcSetPosition) : 
"";
                        }
                        tag.put("srcset", (srcset != null ? srcset + ", " : "") 
+ tag.getAttribute("src") +
                                xValue);
@@ -561,7 +580,8 @@ public class Image extends WebComponent implements 
IResourceListener
         *
         * @return the cross origins settings
         */
-       public Cors getCrossOrigin() {
+       public Cors getCrossOrigin()
+       {
                return crossOrigin;
        }
 
@@ -569,17 +589,21 @@ public class Image extends WebComponent implements 
IResourceListener
         * Sets the cross origin settings<br>
         * <br>
         *
-        * <b>ANONYMOUS</b>: Cross-origin CORS requests for the element will 
not have the credentials flag set.<br>
+        * <b>ANONYMOUS</b>: Cross-origin CORS requests for the element will 
not have the credentials
+        * flag set.<br>
         * <br>
-        * <b>USE_CREDENTIALS</b>: Cross-origin CORS requests for the element 
will have the credentials flag set.<br>
+        * <b>USE_CREDENTIALS</b>: Cross-origin CORS requests for the element 
will have the credentials
+        * flag set.<br>
         * <br>
-        * <b>no_cores</b>: The empty string is also a valid keyword, and maps 
to the Anonymous state. The attribute's invalid value default is the
-        * Anonymous state. The missing value default, used when the attribute 
is omitted, is the No CORS state
+        * <b>no_cores</b>: The empty string is also a valid keyword, and maps 
to the Anonymous state.
+        * The attribute's invalid value default is the Anonymous state. The 
missing value default, used
+        * when the attribute is omitted, is the No CORS state
         *
         * @param crossOrigin
         *            the cross origins settings to set
         */
-       public void setCrossOrigin(Cors crossOrigin) {
+       public void setCrossOrigin(Cors crossOrigin)
+       {
                this.crossOrigin = crossOrigin;
        }
 

http://git-wip-us.apache.org/repos/asf/wicket/blob/7343727c/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Source.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Source.java 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Source.java
index 76ea0de..713309a 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Source.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/image/Source.java
@@ -40,6 +40,10 @@ public class Source extends Image
        private String media = null;
 
        /**
+        * Creates a source for a picture
+        * 
+        * @param id
+        *            the component id
         * @see org.apache.wicket.markup.html.image.Image
         */
        protected Source(final String id)
@@ -48,6 +52,12 @@ public class Source extends Image
        }
 
        /**
+        * Creates a source for a picture
+        * 
+        * @param id
+        *            the component id
+        * @param resourceReferences
+        *            the resource references applied to the source in the 
given order
         * @see org.apache.wicket.markup.html.image.Image
         */
        public Source(final String id, final ResourceReference... 
resourceReferences)
@@ -56,6 +66,14 @@ public class Source extends Image
        }
 
        /**
+        * Creates a source for a picture
+        * 
+        * @param id
+        *            the component id
+        * @param resourceParameters
+        *            the resource parameters applied to the localized image 
resource
+        * @param resourceReferences
+        *            the resource references applied to the source in the 
given order
         * @see org.apache.wicket.markup.html.image.Image
         */
        public Source(final String id, PageParameters resourceParameters,
@@ -65,6 +83,12 @@ public class Source extends Image
        }
 
        /**
+        * Creates a source for a picture
+        * 
+        * @param id
+        *            the component id
+        * @param imageResources
+        *            the image resources applied to the source in the given 
order
         * @see org.apache.wicket.markup.html.image.Image
         */
        public Source(final String id, final IResource... imageResources)
@@ -73,6 +97,12 @@ public class Source extends Image
        }
 
        /**
+        * Creates a source for a picture
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
         * @see org.apache.wicket.Component#Component(String, IModel)
         */
        public Source(final String id, final IModel<?> model)
@@ -81,6 +111,12 @@ public class Source extends Image
        }
 
        /**
+        * Creates a source for a picture
+        * 
+        * @param id
+        *            the component id
+        * @param string
+        *            the string used as model
         * @see org.apache.wicket.markup.html.image.Image
         */
        public Source(final String id, final String string)
@@ -124,15 +160,18 @@ public class Source extends Image
         * Unsupported for source tag
         */
        @Override
-       public void setCrossOrigin(Cors crossorigin) {
-               throw new UnsupportedOperationException("It is not allowed to 
set the crossorigin attribute for source tag");
+       public void setCrossOrigin(Cors crossorigin)
+       {
+               throw new UnsupportedOperationException(
+                       "It is not allowed to set the crossorigin attribute for 
source tag");
        }
 
        /**
         * Unsupported for source tag
         */
        @Override
-       public final Cors getCrossOrigin() {
+       public final Cors getCrossOrigin()
+       {
                return null;
        }
 }

http://git-wip-us.apache.org/repos/asf/wicket/blob/7343727c/wicket-core/src/main/java/org/apache/wicket/markup/html/media/MediaComponent.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/MediaComponent.java
 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/MediaComponent.java
index a6ef048..96d757b 100755
--- 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/MediaComponent.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/MediaComponent.java
@@ -43,9 +43,19 @@ public abstract class MediaComponent extends 
WebMarkupContainer
         *
         * @see {@link #setCrossOrigin(Cors)}
         */
-       public enum Cors
-       {
-               ANONYMOUS("anonymous"), USER_CREDENTIALS("user-credentials"), 
NO_CORS("");
+       public enum Cors {
+               /**
+                * no authentication required
+                */
+               ANONYMOUS("anonymous"),
+               /**
+                * user credentials required
+                */
+               USER_CREDENTIALS("user-credentials"),
+               /**
+                * no cross origin
+                */
+               NO_CORS("");
 
                private final String realName;
 
@@ -54,6 +64,11 @@ public abstract class MediaComponent extends 
WebMarkupContainer
                        this.realName = realName;
                }
 
+               /**
+                * Gets the real name for the cors option
+                * 
+                * @return the real name
+                */
                public String getRealName()
                {
                        return realName;
@@ -65,17 +80,32 @@ public abstract class MediaComponent extends 
WebMarkupContainer
         *
         * @see {@link #setPreload(Preload)}
         */
-       public enum Preload
-       {
-               NONE("none"), METADATA("metadata"), AUTO("auto");
+       public enum Preload {
+               /**
+                * preloads nothing
+                */
+               NONE("none"),
+               /**
+                * preloads only meta data like first picture, etc.
+                */
+               METADATA("metadata"),
+               /**
+                * auto detection what is going to be preload
+                */
+               AUTO("auto");
 
-               public final String realName;
+               private final String realName;
 
                private Preload(String realname)
                {
                        realName = realname;
                }
 
+               /**
+                * Gets the real name for the preload option
+                * 
+                * @return the real name
+                */
                public String getRealName()
                {
                        return realName;
@@ -131,47 +161,107 @@ public abstract class MediaComponent extends 
WebMarkupContainer
        }
 
        /**
-        * Constructor.
+        * Creates a media component
         *
         * @param id
         *            The component id
         * @param resourceReference
+        *            the package resource reference of the media file
         */
        public MediaComponent(String id, PackageResourceReference 
resourceReference)
        {
                this(id, null, null, null, resourceReference);
        }
 
-       public MediaComponent(String id, IModel<?> model,
-               PackageResourceReference resourceReference)
+       /**
+        * Creates a media component
+        *
+        * @param id
+        *            The component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the package resource reference of the media file
+        */
+       public MediaComponent(String id, IModel<?> model, 
PackageResourceReference resourceReference)
        {
                this(id, model, null, null, resourceReference);
        }
 
-       public MediaComponent(String id,
-               PackageResourceReference resourceReference,
+       /**
+        * Creates a media component
+        *
+        * @param id
+        *            The component id
+        * @param resourceReference
+        *            the package resource reference of the media file
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
media URL
+        */
+       public MediaComponent(String id, PackageResourceReference 
resourceReference,
                PageParameters pageParameters)
        {
                this(id, null, null, pageParameters, resourceReference);
        }
 
-       public MediaComponent(String id, IModel<?> model,
-               PackageResourceReference resourceReference,
+       /**
+        * Creates a media component
+        *
+        * @param id
+        *            The component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the package resource reference of the media file
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
media URL
+        */
+       public MediaComponent(String id, IModel<?> model, 
PackageResourceReference resourceReference,
                PageParameters pageParameters)
        {
                this(id, model, null, pageParameters, resourceReference);
        }
 
+       /**
+        * Creates a media component
+        *
+        * @param id
+        *            The component id
+        * @param url
+        *            an external URL to be used for the media component
+        */
        public MediaComponent(String id, String url)
        {
                this(id, null, url, null, null);
        }
 
+       /**
+        * Creates a media component
+        *
+        * @param id
+        *            The component id
+        * @param model
+        *            the internally used model
+        * @param url
+        *            an external URL to be used for the media component
+        */
        public MediaComponent(String id, IModel<?> model, String url)
        {
                this(id, model, url, null, null);
        }
 
+       /**
+        * Creates a media component
+        *
+        * @param id
+        *            The component id
+        * @param model
+        *            the internally used model
+        * @param url
+        *            an external URL to be used for the media component
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
media URL
+        */
        public MediaComponent(String id, IModel<?> model, String url, 
PageParameters pageParameters)
        {
                this(id, model, url, pageParameters, null);
@@ -201,8 +291,8 @@ public abstract class MediaComponent extends 
WebMarkupContainer
 
                if (resourceReference != null)
                {
-                       CharSequence urlToMediaReference = 
RequestCycle.get().urlFor(
-                                       resourceReference, pageParameters);
+                       CharSequence urlToMediaReference = 
RequestCycle.get().urlFor(resourceReference,
+                               pageParameters);
                        tag.put("src", urlToMediaReference + timeManagement);
                }
                else if (url != null)

http://git-wip-us.apache.org/repos/asf/wicket/blob/7343727c/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Source.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Source.java 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Source.java
index fb2c467..4abdca9 100755
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Source.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Source.java
@@ -48,52 +48,122 @@ public class Source extends WebMarkupContainer
 
        private final String url;
 
+       /**
+        * Creates a source
+        * 
+        * @param id
+        *            the component id
+        */
        public Source(String id)
        {
                this(id, null, null, null, null);
        }
 
+       /**
+        * Creates a source
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        */
        public Source(String id, IModel<?> model)
        {
                this(id, model, null, null, null);
        }
 
+       /**
+        * Creates a source
+        * 
+        * @param id
+        *            the component id
+        * @param resourceReference
+        *            the resource reference to provide the source data
+        */
        public Source(String id, PackageResourceReference resourceReference)
        {
                this(id, null, null, null, resourceReference);
        }
 
-       public Source(String id, IModel<?> model,
-               PackageResourceReference resourceReference)
+       /**
+        * Creates a source
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the resource reference to provide the source data
+        */
+       public Source(String id, IModel<?> model, PackageResourceReference 
resourceReference)
        {
                this(id, model, null, null, resourceReference);
        }
 
+       /**
+        * Creates a source
+        * 
+        * @param id
+        *            the component id
+        * @param pageParameters
+        *            the the page parameters applied to the source URL
+        * @param resourceReference
+        *            the resource reference to provide the source data
+        */
        public Source(String id, PackageResourceReference resourceReference,
                PageParameters pageParameters)
        {
                this(id, null, null, pageParameters, resourceReference);
        }
 
-       public Source(String id, IModel<?> model,
-               PackageResourceReference resourceReference,
+       /**
+        * Creates a source
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the resource reference to provide the source data
+        * @param pageParameters
+        *            the the page parameters applied to the source URL
+        */
+       public Source(String id, IModel<?> model, PackageResourceReference 
resourceReference,
                PageParameters pageParameters)
        {
                this(id, model, null, pageParameters, resourceReference);
        }
 
+       /**
+        * Creates a source
+        * 
+        * @param id
+        *            the component id
+        * @param url
+        *            an external URL to provide the source information
+        */
        public Source(String id, String url)
        {
                this(id, null, url, null, null);
        }
 
+       /**
+        * Creates a source
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param url
+        *            an external URL to provide the source information
+        */
        public Source(String id, IModel<?> model, String url)
        {
                this(id, model, url, null, null);
        }
 
        private Source(String id, IModel<?> model, String url, PageParameters 
pageParameters,
-                      PackageResourceReference resourceReference)
+               PackageResourceReference resourceReference)
        {
                super(id, model);
                this.url = url;
@@ -111,7 +181,8 @@ public class Source extends WebMarkupContainer
                {
                        CharSequence url = 
RequestCycle.get().urlFor(resourceReference, pageParameters);
                        tag.put("src", url);
-               } else if (url != null)
+               }
+               else if (url != null)
                {
                        tag.put("src", url);
                }

http://git-wip-us.apache.org/repos/asf/wicket/blob/7343727c/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Track.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Track.java 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Track.java
index c41217a..9b053bf 100755
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Track.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/Track.java
@@ -39,10 +39,27 @@ public class Track extends WebMarkupContainer
        /**
         * To be used for the kind attribute
         */
-       public enum Kind
-       {
-               SUBTITLES("subtitles"), CAPTIONS("captions"), 
DESCRIPTIONS("descriptions"), CHAPTERS(
-                       "chapters"), METADATA("metadata");
+       public enum Kind {
+               /**
+                * the track is used for subtitles
+                */
+               SUBTITLES("subtitles"),
+               /**
+                * the track is used for captions
+                */
+               CAPTIONS("captions"),
+               /**
+                * the track is used for descriptions
+                */
+               DESCRIPTIONS("descriptions"),
+               /**
+                * the track is used for chapters
+                */
+               CHAPTERS("chapters"),
+               /**
+                * the track is used to provide metadata
+                */
+               METADATA("metadata");
 
                private String realName;
 
@@ -51,6 +68,11 @@ public class Track extends WebMarkupContainer
                        this.realName = realName;
                }
 
+               /**
+                * The real name of the kind
+                * 
+                * @return the real name
+                */
                public String getRealName()
                {
                        return realName;
@@ -71,42 +93,114 @@ public class Track extends WebMarkupContainer
 
        private final PageParameters pageParameters;
 
+       /**
+        * Creates a track
+        * 
+        * @param id
+        *            the component id
+        */
        public Track(String id)
        {
                this(id, null, null, null, null);
        }
 
+       /**
+        * Creates a track
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        */
        public Track(String id, IModel<?> model)
        {
                this(id, model, null, null, null);
        }
 
+       /**
+        * Creates a track
+        * 
+        * @param id
+        *            the component id
+        * @param resourceReference
+        *            the resource reference to provide track information - 
like .vtt
+        */
        public Track(String id, ResourceReference resourceReference)
        {
                this(id, null, null, null, resourceReference);
        }
 
+       /**
+        * Creates a track
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the resource reference to provide track information - 
like .vtt
+        */
        public Track(String id, IModel<?> model, ResourceReference 
resourceReference)
        {
                this(id, model, null, null, resourceReference);
        }
 
+       /**
+        * Creates a track
+        * 
+        * @param id
+        *            the component id
+        * @param resourceReference
+        *            the resource reference to provide track information - 
like .vtt
+        * @param pageParameters
+        *            the page parameters applied to the track URL
+        */
        public Track(String id, ResourceReference resourceReference, 
PageParameters pageParameters)
        {
                this(id, null, null, pageParameters, resourceReference);
        }
 
+       /**
+        * Creates a track
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the resource reference to provide track information - 
like .vtt
+        * @param pageParameters
+        *            the page parameters applied to the track URL
+        */
        public Track(String id, IModel<?> model, ResourceReference 
resourceReference,
                PageParameters pageParameters)
        {
                this(id, model, null, pageParameters, resourceReference);
        }
 
+       /**
+        * Creates a track
+        * 
+        * @param id
+        *            the component id
+        * @param url
+        *            an external URL to provide the track information
+        */
        public Track(String id, String url)
        {
                this(id, null, url, null, null);
        }
 
+       /**
+        * Creates a track
+        * 
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param url
+        *            an external URL to provide the track information
+        */
        public Track(String id, IModel<?> model, String url)
        {
                this(id, model, url, null, null);

http://git-wip-us.apache.org/repos/asf/wicket/blob/7343727c/wicket-core/src/main/java/org/apache/wicket/markup/html/media/audio/Audio.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/audio/Audio.java
 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/audio/Audio.java
index aa6e782..2b7cb61 100755
--- 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/audio/Audio.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/audio/Audio.java
@@ -33,55 +33,147 @@ public class Audio extends MediaComponent
 {
        private static final long serialVersionUID = 1L;
 
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        */
        public Audio(String id)
        {
                super(id);
        }
 
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        */
        public Audio(String id, IModel<?> model)
        {
                super(id, model);
        }
 
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param resourceReference
+        *            the package resource reference of the audio file
+        */
        public Audio(String id, PackageResourceReference resourceReference)
        {
                super(id, resourceReference);
        }
 
-       public Audio(String id, IModel<?> model,
-               PackageResourceReference resourceReference)
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the package resource reference of the audio file
+        */
+       public Audio(String id, IModel<?> model, PackageResourceReference 
resourceReference)
        {
                super(id, model, resourceReference);
        }
 
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param resourceReference
+        *            the package resource reference of the audio file
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
audio URL
+        */
        public Audio(String id, PackageResourceReference resourceReference,
                PageParameters pageParameters)
        {
                super(id, resourceReference, pageParameters);
        }
 
-       public Audio(String id, IModel<?> model,
-               PackageResourceReference resourceReference,
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the package resource reference of the audio file
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
audio URL
+        */
+       public Audio(String id, IModel<?> model, PackageResourceReference 
resourceReference,
                PageParameters pageParameters)
        {
                super(id, model, resourceReference, pageParameters);
        }
 
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param url
+        *            an external URL to be used for the audio component
+        */
        public Audio(String id, String url)
        {
                super(id, url);
        }
 
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param url
+        *            an external URL to be used for the audio component
+        */
        public Audio(String id, IModel<?> model, String url)
        {
                super(id, model, url);
        }
 
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param url
+        *            an external URL to be used for the audio component
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
audio URL
+        */
        public Audio(String id, String url, PageParameters pageParameters)
        {
                super(id, null, url, pageParameters);
        }
 
+       /**
+        * Creates an audio component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param url
+        *            an external URL to be used for the audio component
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
audio URL
+        */
        public Audio(String id, IModel<?> model, String url, PageParameters 
pageParameters)
        {
                super(id, model, url, pageParameters);

http://git-wip-us.apache.org/repos/asf/wicket/blob/7343727c/wicket-core/src/main/java/org/apache/wicket/markup/html/media/video/Video.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/video/Video.java
 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/video/Video.java
index 7e685b9..c792356 100755
--- 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/media/video/Video.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/media/video/Video.java
@@ -43,55 +43,147 @@ public class Video extends MediaComponent
 
        private PageParameters posterPageParameters;
 
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        */
        public Video(String id)
        {
                super(id);
        }
 
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        */
        public Video(String id, IModel<?> model)
        {
                super(id, model);
        }
 
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        * @param resourceReference
+        *            the package resource reference of the video file
+        */
        public Video(String id, PackageResourceReference resourceReference)
        {
                super(id, resourceReference);
        }
 
-       public Video(String id, IModel<?> model,
-               PackageResourceReference resourceReference)
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the package resource reference of the video file
+        */
+       public Video(String id, IModel<?> model, PackageResourceReference 
resourceReference)
        {
                super(id, model, resourceReference);
        }
 
+       /**
+        * Creates a media component
+        *
+        * @param id
+        *            the component id
+        * @param resourceReference
+        *            the package resource reference of the video file
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
video URL
+        */
        public Video(String id, PackageResourceReference resourceReference,
                PageParameters pageParameters)
        {
                super(id, resourceReference, pageParameters);
        }
 
-       public Video(String id, IModel<?> model,
-               PackageResourceReference resourceReference,
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param resourceReference
+        *            the package resource reference of the video file
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
video URL
+        */
+       public Video(String id, IModel<?> model, PackageResourceReference 
resourceReference,
                PageParameters pageParameters)
        {
                super(id, model, resourceReference, pageParameters);
        }
 
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        * @param url
+        *            an external URL to be used for the video component
+        */
        public Video(String id, String url)
        {
                super(id, url);
        }
 
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param url
+        *            an external URL to be used for the video component
+        */
        public Video(String id, IModel<?> model, String url)
        {
                super(id, model, url);
        }
 
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        * @param url
+        *            an external URL to be used for the video component
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
video URL
+        */
        public Video(String id, String url, PageParameters pageParameters)
        {
                super(id, null, url, pageParameters);
        }
 
+       /**
+        * Creates a video component
+        *
+        * @param id
+        *            the component id
+        * @param model
+        *            the internally used model
+        * @param url
+        *            an external URL to be used for the video component
+        * @param pageParameters
+        *            the page parameters to be used to be prepended to the 
video URL
+        */
        public Video(String id, IModel<?> model, String url, PageParameters 
pageParameters)
        {
                super(id, model, url, pageParameters);

Reply via email to