Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add attribution for <frame>, <embed>, <object> #78

Merged
merged 3 commits into from Dec 2, 2019
Merged

Conversation

npm1
Copy link
Contributor

@npm1 npm1 commented Nov 28, 2019

This is basically #65 but doing a new PR because rebasing would've taken me longer.
Fixes #52


Preview | Diff

This is basically #65 but doing a new PR because rebasing would've taken me longer.
Fixes #52
index.bs Outdated Show resolved Hide resolved
@npm1
Copy link
Contributor Author

npm1 commented Nov 28, 2019

Test being added in https://chromium-review.googlesource.com/c/chromium/src/+/1942877. Chrome fails object due to needing data instead of src.

chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Nov 28, 2019
Matches spec change in w3c/longtasks#78

Change-Id: I5641d4a6f1424abe87e1fb9ce000ac776e5e84ed
Copy link
Contributor

@tdresser tdresser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but someone like Domenic would have more context.

index.bs Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
npm1 and others added 2 commits December 2, 2019 15:56
Co-Authored-By: Domenic Denicola <d@domenic.me>
@domenic domenic merged commit c7144c2 into master Dec 2, 2019
@domenic domenic deleted the more-attribution branch December 2, 2019 21:03
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Dec 3, 2019
This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Dec 3, 2019
This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Dec 4, 2019
This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949146
Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721258}
chromium-wpt-export-bot pushed a commit to web-platform-tests/wpt that referenced this pull request Dec 4, 2019
This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949146
Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721258}
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Dec 9, 2019
…nerSrc in attribution, a=testonly

Automatic update from web-platform-tests
[Longtasks] Fix containerType and containerSrc in attribution

This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949146
Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721258}

--

wpt-commits: e92918c7452f2ef99fe95545fdfb41a7787b9b94
wpt-pr: 20595
xeonchen pushed a commit to xeonchen/gecko that referenced this pull request Dec 9, 2019
…nerSrc in attribution, a=testonly

Automatic update from web-platform-tests
[Longtasks] Fix containerType and containerSrc in attribution

This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949146
Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721258}

--

wpt-commits: e92918c7452f2ef99fe95545fdfb41a7787b9b94
wpt-pr: 20595
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Dec 10, 2019
…nerSrc in attribution, a=testonly

Automatic update from web-platform-tests
[Longtasks] Fix containerType and containerSrc in attribution

This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949146
Reviewed-by: Steve Kobes <skobeschromium.org>
Commit-Queue: Nicolás Peña Moreno <npmchromium.org>
Cr-Commit-Position: refs/heads/master{#721258}

--

wpt-commits: e92918c7452f2ef99fe95545fdfb41a7787b9b94
wpt-pr: 20595

UltraBlame original commit: a13898cb3364862002fb9de0e58f039c4c0a5b3e
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Dec 10, 2019
…nerSrc in attribution, a=testonly

Automatic update from web-platform-tests
[Longtasks] Fix containerType and containerSrc in attribution

This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949146
Reviewed-by: Steve Kobes <skobeschromium.org>
Commit-Queue: Nicolás Peña Moreno <npmchromium.org>
Cr-Commit-Position: refs/heads/master{#721258}

--

wpt-commits: e92918c7452f2ef99fe95545fdfb41a7787b9b94
wpt-pr: 20595

UltraBlame original commit: a13898cb3364862002fb9de0e58f039c4c0a5b3e
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Dec 10, 2019
…nerSrc in attribution, a=testonly

Automatic update from web-platform-tests
[Longtasks] Fix containerType and containerSrc in attribution

This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949146
Reviewed-by: Steve Kobes <skobeschromium.org>
Commit-Queue: Nicolás Peña Moreno <npmchromium.org>
Cr-Commit-Position: refs/heads/master{#721258}

--

wpt-commits: e92918c7452f2ef99fe95545fdfb41a7787b9b94
wpt-pr: 20595

UltraBlame original commit: a13898cb3364862002fb9de0e58f039c4c0a5b3e
ns-rsilva pushed a commit to ns-rsilva/chromium that referenced this pull request Apr 25, 2024
This CL brings the longtasks API up to date with the recent changes on
w3c/longtasks#78. It also adds WPT tests for
those changes. The changes are:

* containerType is "window", "iframe", "embed", etc. depending on the
type of the culprit browsing context container.
* containerSrc is now the 'data' attribute when the container is an
<object>.

Change-Id: I06f661968778e3bec433a1bfe758b4604df01557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1949146
Reviewed-by: Steve Kobes <skobes@chromium.org>
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721258}

Former-commit-id: 7598e87a8495404cc97e8db34852714ea35920e0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The processing model only works for iframes, not object/embed/etc.
3 participants