From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: Thomas Wood <thomas.wood@intel.com>
Subject: Re: [maintainer-tools PATCH] drm-intel: embed wavedrom engine and skin into the web page
Date: Wed, 16 Dec 2015 11:08:25 +0200 [thread overview]
Message-ID: <87vb7ysquu.fsf@intel.com> (raw)
In-Reply-To: <1449566713-31664-1-git-send-email-jani.nikula@intel.com>
On Tue, 08 Dec 2015, Jani Nikula <jani.nikula@intel.com> wrote:
> The wavedrom timeline will be missing from html pages served over https
> due to "mixed active content" blocking [1], because the wavedrom engine
> and skin are only available over http. Embed the engine and skin into
> the resulting html to avoid the problem.
>
> The rst :url: will fetch and include the scripts at html build time.
>
> [1] https://developer.mozilla.org/en-US/docs/Security/MixedContent
>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Pushed.
> ---
> drm-intel-timeline.rst | 21 +++++++++++++++++----
> drm-intel.rst | 1 -
> 2 files changed, 17 insertions(+), 5 deletions(-)
>
> diff --git a/drm-intel-timeline.rst b/drm-intel-timeline.rst
> index 0d78046736b2..e1766a5df98b 100644
> --- a/drm-intel-timeline.rst
> +++ b/drm-intel-timeline.rst
> @@ -1,10 +1,23 @@
> -.. This is a wrapper intended to both keep the master document clean of the raw
> -.. html script stuff and to keep the wavedrom source pure json.
> +.. raw:: html
> +
> + <script type="text/javascript">
> + /* Embedded WaveDrom skin from http://wavedrom.com/skins/default.js */
> +
> +.. raw:: html
> + :url: http://wavedrom.com/skins/default.js
>
> .. raw:: html
>
> - <script src="http://wavedrom.com/skins/default.js" type="text/javascript"></script>
> - <script src="http://wavedrom.com/WaveDrom.js" type="text/javascript"></script>
> + </script>
> + <script type="text/javascript">
> + /* Embedded WaveDrom engine from http://wavedrom.com/WaveDrom.js */
> +
> +.. raw:: html
> + :url: http://wavedrom.com/WaveDrom.js
> +
> +.. raw:: html
> +
> + </script>
> <script type="WaveDrom">
>
> .. raw:: html
> diff --git a/drm-intel.rst b/drm-intel.rst
> index dc1393bbec21..c6b0800e2dbc 100644
> --- a/drm-intel.rst
> +++ b/drm-intel.rst
> @@ -136,7 +136,6 @@ kernel release cycle. Worth noting is that we're working on two or three kernel
> releases at the same time. Big features take a long time to hit a kernel
> release. There are no fast paths.
>
> -.. Note: This requires JavaScript and will access http://wavedrom.com to render.
> .. include:: drm-intel-timeline.rst
>
> For predictions on the future merge windows and releases, see
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
prev parent reply other threads:[~2015-12-16 9:08 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-08 9:25 [maintainer-tools PATCH] drm-intel: embed wavedrom engine and skin into the web page Jani Nikula
2015-12-16 9:08 ` Jani Nikula [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87vb7ysquu.fsf@intel.com \
--to=jani.nikula@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=thomas.wood@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox