<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>12391</bug_id>
          
          <creation_ts>2011-03-29 07:02:14 +0000</creation_ts>
          <short_desc>showModalDialog() should be blocked by sandbox attribute</short_desc>
          <delta_ts>2011-08-04 05:33:45 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>HTML WG</product>
          <component>LC1 HTML5 spec</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows NT</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://dev.w3.org/html5/spec/Overview.html#dom-showmodaldialog</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jacob Rossi [MSFT]">jrossi</reporter>
          <assigned_to name="Ian &apos;Hixie&apos; Hickson">ian</assigned_to>
          <cc>ayg</cc>
    
    <cc>ian</cc>
    
    <cc>mike</cc>
    
    <cc>public-html-admin</cc>
    
    <cc>public-html-wg-issue-tracking</cc>
          
          <qa_contact name="HTML WG Bugzilla archive list">public-html-bugzilla</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>47021</commentid>
    <comment_count>0</comment_count>
    <who name="Jacob Rossi [MSFT]">jrossi</who>
    <bug_when>2011-03-29 07:02:14 +0000</bug_when>
    <thetext>From the current spec text, it&apos;s not clear that window.showModalDialog( ) should be blocked inside a sandboxed iframe. It seems that it should follow in the same suit as window.open(). 

showModalDialog() should be added to the sentence &quot;This flag also prevents content from creating new auxiliary browsing contexts, e.g. using the target attribute or the window.open() method.&quot; [1]

Additionally, in the steps for executing showModalDialog [2], the first step should indicate that the UA should abort these steps if the &quot;sandboxed navigation browsing context flag&quot; is set.


[1] http://dev.w3.org/html5/spec/Overview.html#attr-iframe-sandbox
[2] http://dev.w3.org/html5/spec/Overview.html#dom-showmodaldialog</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>49531</commentid>
    <comment_count>1</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2011-06-13 22:27:59 +0000</bug_when>
    <thetext>The sentence &quot;This flag also prevents content from creating new auxiliary browsing contexts, e.g. using the target attribute or the window.open() method.&quot; is non-normative, and is a rather vague description of reality (which is that it&apos;s largely up to the UA what happens). Since you&apos;re an implementor, please ignore the non-normative text. :-)

Blocking showModelDialog() seems reasonable. Should it be always blocked, or should the UA have the option of allowing it anyway, as with window.open()? I&apos;m guessing it should just always block, since it&apos;s more synchronous.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>49617</commentid>
    <comment_count>2</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2011-06-15 06:14:15 +0000</bug_when>
    <thetext>EDITOR&apos;S RESPONSE: This is an Editor&apos;s Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document:
   http://dev.w3.org/html5/decision-policy/decision-policy.html

Status: Accepted
Change Description: see diff given below
Rationale:

Turns out that showModalDialog() was sort of blocked already, in that it would pop up a window but not load anything in the window. Oops. I&apos;ve made it so that it is blocked earlier.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>49621</commentid>
    <comment_count>3</comment_count>
    <who name="">contributor</who>
    <bug_when>2011-06-15 06:28:26 +0000</bug_when>
    <thetext>Checked in as WHATWG revision r6233.
Check-in comment: showModalDialog() inside a sandboxed iframe should be blocked _before_ the window pops up... Also, some minor editorial cleanup in this area.
http://html5.org/tools/web-apps-tracker?from=6232&amp;to=6233</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>53843</commentid>
    <comment_count>4</comment_count>
    <who name="Michael[tm] Smith">mike</who>
    <bug_when>2011-08-04 05:33:45 +0000</bug_when>
    <thetext>mass-move component to LC1</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>