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

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>196843</bug_id>
          
          <creation_ts>2019-04-11 18:10:51 -0700</creation_ts>
          <short_desc>Implement IDREF element reflection</short_desc>
          <delta_ts>2022-10-10 12:16:05 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>DOM</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>239852</dependson>
    
    <dependson>239853</dependson>
    
    <dependson>240563</dependson>
    
    <dependson>244972</dependson>
    
    <dependson>245229</dependson>
    
    <dependson>245301</dependson>
    
    <dependson>245429</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="James Craig">jcraig</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>cdumez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>rego</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1526578</commentid>
    <comment_count>0</comment_count>
    <who name="James Craig">jcraig</who>
    <bug_when>2019-04-11 18:10:51 -0700</bug_when>
    <thetext>## IDREF element reflection

- IDREF element reflection pull request approved by WhatWG, but is awaiting implementations to merge: https://github.com/whatwg/html/pull/3917

Not specific to Accessibility, but there is also an ARIA follow-up.
- The ARIA pull request has been verbally approved; Alice is looking into WPT tests
- pull request: https://github.com/w3c/aria/pull/920
- tests for previous IDL change (generated): https://github.com/web-platform-tests/wpt/commit/4caae418cffa4f803f2fe283a4d3fd7cfaae7461</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1526579</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2019-04-11 18:11:31 -0700</bug_when>
    <thetext>&lt;rdar://problem/49836210&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1868820</commentid>
    <comment_count>2</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2022-05-13 11:54:04 -0700</bug_when>
    <thetext>Manuel, what work remains to be done here?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1869175</commentid>
    <comment_count>3</comment_count>
    <who name="Manuel Rego Casasnovas">rego</who>
    <bug_when>2022-05-16 02:33:42 -0700</bug_when>
    <thetext>(In reply to Chris Dumez from comment #2)
&gt; Manuel, what work remains to be done here?

The implementation work should be ready, unless someone changes on the spec side.
The only remaining bit would be enabling AriaReflectionForElementReferencesEnabled flag by default, but I guess that would need to wait for some spec work.

From the spec side, these properties need to be re-added to ARIA spec.
They were there for a while, but got removed until the HTML spec gets updated:
https://github.com/w3c/aria/pull/1260

The HTML PR is: https://github.com/whatwg/html/pull/3917
I&apos;ve posted a comment today with the updates to see if it can get merged and unblock the situation.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1869933</commentid>
    <comment_count>4</comment_count>
    <who name="Manuel Rego Casasnovas">rego</who>
    <bug_when>2022-05-18 00:20:13 -0700</bug_when>
    <thetext>(In reply to Manuel Rego Casasnovas from comment #3)
&gt; (In reply to Chris Dumez from comment #2)
&gt; &gt; Manuel, what work remains to be done here?
&gt; 
&gt; The implementation work should be ready, unless something changes on the spec
&gt; side.

It looks like there&apos;s a missing bit from the spec that we are not implementing yet. I&apos;ve reported a separated bug about that, see bug #240563.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1875249</commentid>
    <comment_count>5</comment_count>
    <who name="Manuel Rego Casasnovas">rego</who>
    <bug_when>2022-06-10 09:35:33 -0700</bug_when>
    <thetext>(In reply to Chris Dumez from comment #2)
&gt; Manuel, what work remains to be done here?

An update on the status.

WebKit implementation is feature complete and passes all the tests.

The HTML PR has been merged: https://github.com/whatwg/html/pull/7934

I&apos;ve sent a PR for ARIA to add these attributes back to the IDL: https://github.com/w3c/aria/pull/1755

So I guess it&apos;s up to you to decide when you want to enable this feature by default. I can prepare the patch for that, if you ask me to do it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1880967</commentid>
    <comment_count>6</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2022-07-06 08:26:01 -0700</bug_when>
    <thetext>(In reply to Manuel Rego Casasnovas from comment #5)
&gt; (In reply to Chris Dumez from comment #2)
&gt; &gt; Manuel, what work remains to be done here?
&gt; 
&gt; An update on the status.
&gt; 
&gt; WebKit implementation is feature complete and passes all the tests.
&gt; 
&gt; The HTML PR has been merged: https://github.com/whatwg/html/pull/7934
&gt; 
&gt; I&apos;ve sent a PR for ARIA to add these attributes back to the IDL:
&gt; https://github.com/w3c/aria/pull/1755
&gt; 
&gt; So I guess it&apos;s up to you to decide when you want to enable this feature by
&gt; default. I can prepare the patch for that, if you ask me to do it.

I have no opposition to enabling by default.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1880978</commentid>
    <comment_count>7</comment_count>
    <who name="James Craig">jcraig</who>
    <bug_when>2022-07-06 09:20:20 -0700</bug_when>
    <thetext>&gt; I have no opposition to enabling by default.

Same here. Thanks Rego.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1881226</commentid>
    <comment_count>8</comment_count>
    <who name="Manuel Rego Casasnovas">rego</who>
    <bug_when>2022-07-06 21:49:14 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/2151</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1885016</commentid>
    <comment_count>9</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2022-07-18 22:27:00 -0700</bug_when>
    <thetext>Committed 252590@main (dcb5be262b06): &lt;https://commits.webkit.org/252590@main&gt;

Reviewed commits have been landed. Closing PR #2151 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1899831</commentid>
    <comment_count>10</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2022-09-20 09:06:24 -0700</bug_when>
    <thetext>Re-opened since this is blocked by bug 245429</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>