Skip to content

Commit

Permalink
Bug 1276438 part 4. Move the .body getter from HTMLDocument to Docume…
Browse files Browse the repository at this point in the history
…nt. r=mystor

The "body" part of responsexml-document-properties.htm is not really per current
spec text, and fails in every non-Firefox browser, and in Firefox after this
change.  web-platform-tests/wpt#2668 tracks this issue
to some extent, but if all browsers are going to align here anyway, we should
just adjust the test and move on.

MozReview-Commit-ID: HTLfggvi5LL

UltraBlame original commit: d6b890b102b7197415317b0966e6a5c636fd5f57
  • Loading branch information
marco-c committed Oct 2, 2019
1 parent bc16ccc commit c452f00
Show file tree
Hide file tree
Showing 5 changed files with 48 additions and 306 deletions.
7 changes: 6 additions & 1 deletion dom/bindings/test/test_bug560072.html
Expand Up @@ -219,7 +219,7 @@
.
getOwnPropertyDescriptor
(
HTMLDocument
Document
.
prototype
"
Expand Down Expand Up @@ -257,8 +257,13 @@
.
getPrototypeOf
(
Object
.
getPrototypeOf
(
document
)
)
"
body
"
Expand Down
71 changes: 41 additions & 30 deletions dom/webidl/Document.webidl
Expand Up @@ -91,31 +91,46 @@ MPL
.
*
*
http
https
:
/
/
dxr
dom
.
mozilla
spec
.
whatwg
.
org
/
mozilla
#
interface
-
central
/
source
document
*
https
:
/
dom
/
interfaces
html
.
spec
.
whatwg
.
org
/
core
multipage
/
nsIDOMDocument
dom
.
idl
html
#
the
-
document
-
object
*
/
interface
Expand Down Expand Up @@ -180,7 +195,7 @@ pseudo
;
/
*
http
https
:
/
/
Expand Down Expand Up @@ -686,26 +701,23 @@ allowUnsafeHTML
;
/
/
http
https
:
/
/
www
html
.
spec
.
whatwg
.
org
/
specs
/
web
-
apps
/
current
-
work
multipage
/
dom
.
html
#
the
-
Expand Down Expand Up @@ -799,12 +811,11 @@ attribute
DOMString
dir
;
/
/
(
HTML
only
)
[
CEReactions
Pure
SetterThrows
]
attribute
HTMLElement
?
Expand Down
10 changes: 0 additions & 10 deletions dom/webidl/HTMLDocument.webidl
Expand Up @@ -130,16 +130,6 @@ name
)
;
[
CEReactions
Pure
SetterThrows
]
attribute
HTMLElement
?
body
;
[
Pure
]
readonly
Expand Down

0 comments on commit c452f00

Please sign in to comment.