René's URL Explorer Experiment


Title: fix($$RAFProvider): prevent a JavaScript error in Firefox by dmuellner · Pull Request #16192 · angular/angular.js · GitHub

Open Graph Title: fix($$RAFProvider): prevent a JavaScript error in Firefox by dmuellner · Pull Request #16192 · angular/angular.js

X Title: fix($$RAFProvider): prevent a JavaScript error in Firefox by dmuellner · Pull Request #16192 · angular/angular.js

Description: What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Bug fix: prevent a JavaScript error in Firefox. What is the current behavior? (You can also link to an open issue here) Firefox raises a Javascript Error "TypeError: 'requestAnimationFrame' called on an object that does not implement interface Window." with animated elements. This is because Window.requestAnimationFrame() is called without binding to a Window instance in the function which is returned from $$RAFProvider(). What is the new behavior (if this is a feature change)? No error message. Does this PR introduce a breaking change? Not that I know of. Please check if the PR fulfills these requirements The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format Tests for the changes have been added (for bug fixes / features) Docs have been added / updated (for bug fixes / features) Other information: I was not able to run local tests.

Open Graph Description: What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Bug fix: prevent a JavaScript error in Firefox. What is the current behavior? (You can also link to an open issue he...

X Description: What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Bug fix: prevent a JavaScript error in Firefox. What is the current behavior? (You can also link to an open issue he...

Opengraph URL: https://github.com/angular/angular.js/pull/16192

X: @github

direct link

Domain: patch-diff.githubusercontent.com

route-pattern/:user_id/:repository/pull/:id/files(.:format)
route-controllerpull_requests
route-actionfiles
fetch-noncev2:8e255d55-d0eb-cf31-4581-8f7bc83dea80
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idAEFC:154085:92DCBBD:C2F0462:696E0967
html-safe-nonce23b292f0fbe41762b25e5b6a748fa5e9286e84ec301c3230a56868240e708dad
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBRUZDOjE1NDA4NTo5MkRDQkJEOkMyRjA0NjI6Njk2RTA5NjciLCJ2aXNpdG9yX2lkIjoiODc5NTM4MjEzMzg3NDg4NzAxNSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmacb4211a583c7432c301aec650f732893a1057bfef53044615b761c5899cb587ea
hovercard-subject-tagpull_request:137152630
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///pull_requests/show/files
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/angular/angular.js/pull/16192/files
twitter:imagehttps://avatars.githubusercontent.com/u/1724712?s=400&v=4
twitter:cardsummary_large_image
og:imagehttps://avatars.githubusercontent.com/u/1724712?s=400&v=4
og:image:altWhat kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Bug fix: prevent a JavaScript error in Firefox. What is the current behavior? (You can also link to an open issue he...
og:site_nameGitHub
og:typeobject
hostnamegithub.com
expected-hostnamegithub.com
None0b1760aa20e1a810eba6245d04e3885be7363dfe08d08b61f605c852359472e0
turbo-cache-controlno-preview
diff-viewunified
go-importgithub.com/angular/angular.js git https://github.com/angular/angular.js.git
octolytics-dimension-user_id139426
octolytics-dimension-user_loginangular
octolytics-dimension-repository_id460078
octolytics-dimension-repository_nwoangular/angular.js
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id460078
octolytics-dimension-repository_network_root_nwoangular/angular.js
turbo-body-classeslogged-out env-production page-responsive full-width
disable-turbotrue
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release84090305cd10c2b140bf307f69f57f0892b4f215
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#start-of-content
https://patch-diff.githubusercontent.com/
Sign in https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fangular%2Fangular.js%2Fpull%2F16192%2Ffiles
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub SparkBuild and deploy intelligent appshttps://github.com/features/spark
GitHub ModelsManage and compare promptshttps://github.com/features/models
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
GitHub SponsorsFund open source developershttps://github.com/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/accelerator
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://patch-diff.githubusercontent.com/login?return_to=https%3A%2F%2Fgithub.com%2Fangular%2Fangular.js%2Fpull%2F16192%2Ffiles
Sign up https://patch-diff.githubusercontent.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fpull_requests%2Fshow%2Ffiles&source=header-repo&source_repo=angular%2Fangular.js
Reloadhttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
Reloadhttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
Reloadhttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
angular https://patch-diff.githubusercontent.com/angular
angular.jshttps://patch-diff.githubusercontent.com/angular/angular.js
Notifications https://patch-diff.githubusercontent.com/login?return_to=%2Fangular%2Fangular.js
Fork 27.3k https://patch-diff.githubusercontent.com/login?return_to=%2Fangular%2Fangular.js
Star 59k https://patch-diff.githubusercontent.com/login?return_to=%2Fangular%2Fangular.js
Code https://patch-diff.githubusercontent.com/angular/angular.js
Issues 389 https://patch-diff.githubusercontent.com/angular/angular.js/issues
Pull requests 72 https://patch-diff.githubusercontent.com/angular/angular.js/pulls
Actions https://patch-diff.githubusercontent.com/angular/angular.js/actions
Projects 0 https://patch-diff.githubusercontent.com/angular/angular.js/projects
Wiki https://patch-diff.githubusercontent.com/angular/angular.js/wiki
Security Uh oh! There was an error while loading. Please reload this page. https://patch-diff.githubusercontent.com/angular/angular.js/security
Please reload this pagehttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
Insights https://patch-diff.githubusercontent.com/angular/angular.js/pulse
Code https://patch-diff.githubusercontent.com/angular/angular.js
Issues https://patch-diff.githubusercontent.com/angular/angular.js/issues
Pull requests https://patch-diff.githubusercontent.com/angular/angular.js/pulls
Actions https://patch-diff.githubusercontent.com/angular/angular.js/actions
Projects https://patch-diff.githubusercontent.com/angular/angular.js/projects
Wiki https://patch-diff.githubusercontent.com/angular/angular.js/wiki
Security https://patch-diff.githubusercontent.com/angular/angular.js/security
Insights https://patch-diff.githubusercontent.com/angular/angular.js/pulse
dmuellnerhttps://patch-diff.githubusercontent.com/dmuellner
angular:masterhttps://patch-diff.githubusercontent.com/angular/angular.js/tree/master
Conversation 16 https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192
Commits 3 https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/commits
Checks 0 https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/checks
Files changed 3 https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
fix($$RAFProvider): prevent a JavaScript error in Firefox https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#top
Show all changes 3 commits https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
f709dec fix($$RAFProvider): prevent a JavaScript error in Firefox dmuellner Aug 23, 2017 https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/commits/f709decd9398c7519fd899ea8d7ca40cd8e4666f
10e8e48 Address reviewer's comments dmuellner Aug 23, 2017 https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/commits/10e8e4886e129902e911b5687c2f8f39869cb9b0
fa79348 Minimal example to showcase bug. dmuellner Aug 28, 2017 https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/commits/fa793489933a9f29aeb523ce062573849a3e989d
Clear filters https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
Please reload this pagehttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
Please reload this pagehttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files
raf.js https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#diff-32b450cd4ce544f4b57c93ae61c2ec784dbe3840b59792a3a9af7b169cb7eeb6
manifest.json https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#diff-06c124969ae697d5c6434c54d32b7d9bdd109a472c61ce7b7ec29fe2310f4a3d
test.js https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#diff-f2bf1a473a862560aa9d8b58f400fe43b09520332fe1192338d6b37757784178
src/ng/raf.jshttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#diff-32b450cd4ce544f4b57c93ae61c2ec784dbe3840b59792a3a9af7b169cb7eeb6
View file https://patch-diff.githubusercontent.com/angular/angular.js/blob/fa793489933a9f29aeb523ce062573849a3e989d/src/ng/raf.js
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/{{ revealButtonHref }}
https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#diff-32b450cd4ce544f4b57c93ae61c2ec784dbe3840b59792a3a9af7b169cb7eeb6
https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#diff-32b450cd4ce544f4b57c93ae61c2ec784dbe3840b59792a3a9af7b169cb7eeb6
test/extension/manifest.jsonhttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#diff-06c124969ae697d5c6434c54d32b7d9bdd109a472c61ce7b7ec29fe2310f4a3d
View file https://patch-diff.githubusercontent.com/angular/angular.js/blob/fa793489933a9f29aeb523ce062573849a3e989d/test/extension/manifest.json
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/{{ revealButtonHref }}
test/extension/test.jshttps://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/files#diff-f2bf1a473a862560aa9d8b58f400fe43b09520332fe1192338d6b37757784178
View file https://patch-diff.githubusercontent.com/angular/angular.js/blob/fa793489933a9f29aeb523ce062573849a3e989d/test/extension/test.js
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://patch-diff.githubusercontent.com/angular/angular.js/pull/16192/{{ revealButtonHref }}
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.