- 29 Aug, 2020 40 commits
-
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1598684367-c0af722967a8d21b37257898f8c14103e1ccc303.profdata to chrome-win32-master-1598689782-79c1a956e58769ff0627fe1929a3fc8c6eba4637.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: chrome/try:win-chrome Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com Change-Id: Iab18071bbd9b752c35f66dc41e71c4ffb2e0c7d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383493Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802905}
-
Owen Min authored
There is no request dialog and request pending dialog for theme request. And the request is confirmed automatically. Enable these dialogs for theme request for better consistency as admin is able to block theme like an extension Bug: 1077349 Change-Id: I96fe2bafc59f97851d089c646294b33098f18492 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2175740Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#802904}
-
Maksim Ivanov authored
Fix use-after-move (potential) bugs found by the "bugprone-use-after-move" clang-tidy check. Bug: 1122844 Change-Id: Ib8f4ae6bb906363dbbab8ed1157a78b3e6a0e4bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383015Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#802903}
-
Maksim Ivanov authored
Fix use-after-move (potential) bugs found by the "bugprone-use-after-move" clang-tidy check. Bug: 1122844 Change-Id: I3f3dab82cb743c4033bb3c562f9a7767341a7e9b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382113Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#802902}
-
Maksim Ivanov authored
Fix use-after-move (potential) bugs found by the "bugprone-use-after-move" clang-tidy check. Bug: 1122844 Change-Id: I3065aa58a68b1f942a87e315906e0f0d2750c2c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382318Reviewed-by:
Adam Langley <agl@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#802901}
-
Chromium WPT Sync authored
Using wpt-import in Chromium fe6f6198. Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: foolip@chromium.org, lpz@chromium.org, robertma@chromium.org: external/wpt/tools NOAUTOREVERT=true TBR=robertma@google.com No-Export: true Change-Id: I6c7ffbe8c34e191c0d8c8daed270b1d47ead9f30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382333Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802900}
-
K. Moon authored
Makes chrome_pdf::UrlLoader abstract, extracting a new PepperUrlLoader implementation from the old UrlLoader. A number of call sites don't need to create a real UrlLoader. These can just use nullptr, as it turns out they never call any methods on the UrlLoader. Bug: 1099022 Change-Id: Ic86fa5a9ccf8f2128f41fb3023c5a07dd682c3bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2381064 Commit-Queue: K. Moon <kmoon@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#802899}
-
Rakina Zata Amni authored
Previously we wanted to do both cross-site process reuse and same-site process reuse for BFCache, but now we want to enable process-reuse on same-site navigations only for BFCache experiment (at least for the initial experiment). This CL adds a case that handles BFCache process reuse separately from ProactivelySwapBrowsingInstance, to ensure we will only do process-reuse on same-site navigations if BFCache is enabled and ProactivelySwapBrowsingInstance is not explicitly enabled (which is what we will do on the experiment). Also consolidates all three process-reuse cases into a more readable flow. Bug: 1122974 Change-Id: Iec6eb683293b9b384c74da238f3161ab149c0845 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2381223 Commit-Queue: Rakina Zata Amni <rakina@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#802898}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1598658096-a64278c47873df00bb292e7f5bde23944317e855.profdata to chrome-win32-master-1598684367-c0af722967a8d21b37257898f8c14103e1ccc303.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: chrome/try:win-chrome Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com Change-Id: I7b5d5f69edb89aa5f85a1a935a67693acf3c9d09 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383491Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802897}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4720732dc405..bb9ebe49ad3b If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: None Tbr: Change-Id: I7d71e49a8222923527c5c8c5257de50e13062aee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383490Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802896}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast Tbr: cr-fuchsia+bot@chromium.org Change-Id: I6fd327e4a714884196186b7a6ee6d6ba9972c322 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383471Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802895}
-
Trevor Perrier authored
This second of four CLs to enable Clank to be in a seperate locale from the system. This CL adds the Android preference key APPLICATION_OVERRIDE_LANGUAGE that will be used to store the override language for Clank. It also adds utility functions and tests for managing the language preference and modifying Configuration locales. Bug: 1068667 Change-Id: Ia8aa6d55c6373696b8af12be43c8741d29bf38d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2355489 Commit-Queue: Trevor Perrier <perrier@chromium.org> Reviewed-by:
Brandon Wylie <wylieb@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#802894}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1598601518-e5c4427f7e0d3b1c20c8bd75b29ccde9f832fa2b.profdata to chrome-win32-master-1598658096-a64278c47873df00bb292e7f5bde23944317e855.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: chrome/try:win-chrome Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com Change-Id: I1725d877da6cdf07bdef6662c4235034c2e1d2c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383034Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802893}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/c6fa19fccc98..fe980adabf55 2020-08-29 thestig@chromium.org Simplify some code in fx_agg_driver.cpp. 2020-08-29 tsepez@chromium.org Make CXFA_ReadyNodeIterator a stack-only type. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pdfium-autoroll Please CC pdfium-deps-rolls@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: pdfium-deps-rolls@chromium.org Change-Id: Ia610a30fdcb371e4664a98288837fbc4b6964485 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383470Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802892}
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1598637591-8ae79944c569ee90562dcd97e5dcdbd0077c5a06.profdata to chrome-mac-master-1598680344-aa469a0f04e08491eb84b2521bf3eecf90475733.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-mac-chromium Please CC sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: chrome/try:mac-chrome Tbr: sebmarchand+pgo_roller@google.com,jeffyoon@google.com,liaoyuke@google.com Change-Id: I6113805762862b431a5e9d31dc84c6564fa71454 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383042Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802891}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/459271df6a8a..f00f89fca0f4 2020-08-29 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update DevTools Chromium DEPS. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Idfa873c36fd0552b5a9ecbf13e2f642e16b0041f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383040Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802890}
-
Ankit Kumar 🌪️ authored
Update PDFiumRange to use gfx::Rect instead of pp::Rect. Update callers to expect gfx::Rect as return value instead of pp::Rect. Noteworthy point: PDFiumRange::GetScreenRects() returns a vector of rects. Instead of converting the returned vector of gfx::Rect to a vector of pp::Rect, the CL modifies recursively the usage of pp::Rect to gfx::Rect of the callers. The reason of updating the callers recursively is to prevent a copy of vector from one structure to another at caller site. Even temporarily, the additional copy of vectors is avoided in this way. Additional points: - In pp::Rect, Union() and Subtract() are utility methods which takes in two rects and returns a rect with the specified operation of input rects. Whereas in gfx::Rect, Union() and Subtract() are member methods which modify the rect in-place. All usage of Union() and Subtract() which are replaced in the CL modify the rect in-place. - In pp::Rect, Intersect() is a utility method which takes in two rects and returns the intersected rect. The counterpart in gfx::Rect is IntersectRects() and not Intersect(). Bug: 1101101 Change-Id: I90dce2c8b33fcced11c4e8a03e2cfb5ae1064abc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2371142Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
K. Moon <kmoon@chromium.org> Commit-Queue: Ankit Kumar
🌪 ️ <ankk@microsoft.com> Cr-Commit-Position: refs/heads/master@{#802889} -
Kuo Jen Wei authored
Replace will createElement('canvas') call with new util.newDrawingCanvas() function with good typed return value. Bug: 1122444 Test: tast run <DUT> "camera.CCAUI*" Change-Id: I7c9f994fdbeec6b6f49b3039be032c578c37099b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2379348 Commit-Queue: Inker Kuo <inker@chromium.org> Auto-Submit: Inker Kuo <inker@chromium.org> Reviewed-by:
Shik Chen <shik@chromium.org> Cr-Commit-Position: refs/heads/master@{#802888}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/5ac807b26749..b4efc051da9d 2020-08-29 jmadill@chromium.org Enable -Wdeprecated-copy. 2020-08-29 jmadill@chromium.org Force impicit fallthrough off in xxhash. 2020-08-29 syoussefi@chromium.org Make copy*Texture parameters consistent for level indices 2020-08-28 cclao@google.com Vulkan: Use context staging buffer for CopyTexSubImage 2020-08-28 timvp@google.com Convert unordered_map to absl::flat_hash_map for select files 2020-08-28 jmadill@chromium.org Enable MSVC warning 4312. 2020-08-28 lehoangq@gmail.com Metal: autogen for EXT_occlusion_query_boolean. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC courtneygo@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: courtneygo@google.com Change-Id: Ibf2fdad9a7df77fe7d955cb8dbe9b1e1e634addc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383038Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802887}
-
Joe DeBlasio authored
This CL is a small tweak to allow us to tune whether or not the keyword matching heuristic applies to the e2LD or not. To maintain current behavior, the default value of the parameter is kept to true. Fixed: 1123151 Change-Id: Ifc85b464776e874a24fe4ec69be6c2aae0960eda Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382404 Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org> Reviewed-by:
Emily Stark <estark@chromium.org> Cr-Commit-Position: refs/heads/master@{#802886}
-
Palak Agarwal authored
Bug: 974149 Change-Id: Ie4753ea473fbfd8448d772c1e07d440e74085ace Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2377718Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Palak Agarwal <agpalak@chromium.org> Cr-Commit-Position: refs/heads/master@{#802885}
-
Darwin Huang authored
Update the comment and explain it a bit better. Bitmaps have been supported in the Native Android clipboard since https://crrev.com/c/2147890. While I'm unsure of the first instance of this comment, an early iteration is active in March 2012(https://crrev.com/9264014), and a recent, more well-documented change that expanded this comment is available here(https://crrev.com/12633009) Change-Id: I192d3abd050577c026f126ed99c9d9313756555c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382881 Auto-Submit: Darwin Huang <huangdarwin@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Commit-Queue: Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#802884}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/f609a00aca46..4720732dc405 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:1098447,chromium:1117068,chromium:1119885,chromium:1123108 Tbr: Change-Id: I61aa66525971132dc34a414afd392f1c28509081 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383037Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802883}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/6ec121d8c3d5..459271df6a8a 2020-08-29 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update DevTools DEPS. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I81192f9c1d1910f3ef06c02f0cb2a37745506075 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383036Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802882}
-
Maksim Ivanov authored
Fix use-after-move (potential) bugs found by the "bugprone-use-after-move" clang-tidy check. Bug: 1122844 Change-Id: I0c067c8b927baae4ce5c81dde91bbd1d45df635b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382813 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#802881}
-
Puthikorn Voravootivat authored
Add proto for power manager to provide thermal hinting to Chrome. Bug: 1081959 Test: See ThermalEvent in chrome://device-log chromeos_unittest --gtest_filter=PowerManagerClientTest.ChangeThermalState Change-Id: Ie7435852da01a5acf3580b5bb569ba117113f658 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2300983 Commit-Queue: Puthikorn Voravootivat <puthik@chromium.org> Auto-Submit: Puthikorn Voravootivat <puthik@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#802880}
-
Kramer Ge authored
create_immed is available since protocol version 2. Exo is at version 2. This method implicitly enables committing a frame of buffers atomically. Change-Id: I46c25af1e257dbef68cc3c810a87ba2dd10b26ca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2381652 Commit-Queue: Kramer Ge <fangzhoug@chromium.org> Reviewed-by:
Maksim Sisov (GMT+3) <msisov@igalia.com> Cr-Commit-Position: refs/heads/master@{#802879}
-
Aaron Leventhal authored
Often, the AX Object Cache will be called with a children changed that is immediately followed by a text change on the same node. Since children changed processing is a superset of text changed processing, the extra text changed processing can be removed in this case. Also, since children changed processing is a superset of text changed processing, reuse the code in AXNodeObject::TextChanged() rather than repeating it in AXNodeObject::ChildrenChanged(). AX-RelNotes: performance improvements Bug: None Change-Id: I82fa983620b36a99f755017f576fddf85d7a5cd9 Cq-Do-Not-Cancel-Tryjobs: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2360319 Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#802878}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/bb235b33d3dd..f609a00aca46 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC mek@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: None Tbr: mek@google.com Change-Id: I7ef36658b5380598760b6eda34cf5a2dddd3899c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383031Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802877}
-
Rulong Chen(陈汝龙) authored
One large favicon might cause an OOM crash when it is converted to a Java-backed Bitmap in Android. Failing to render the favicon still seems preferable than *ever* crash here. R=torne@chromium.org,boliu@chromium.org Bug: 1112794 Change-Id: Idca46d2e1484ce4538b79e8df6910de7b18ef1f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2376929 Commit-Queue: Bo <boliu@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#802876}
-
Dirk Pranke authored
The "generic_wrapper" scripts now being generated during the build need to know whether or not to use Xvfb when running a test; this CL adds that logic. Bug: 816629 Change-Id: Ic2cae1295d7edf583236d6d2640c220c883324ed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2370507 Commit-Queue: Dirk Pranke <dpranke@google.com> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#802875}
-
Maksim Ivanov authored
Enable for Chromium the "bugprone-use-after-move" diagnostic provided by clang-tidy. This check attempts to catch bugs when a C++ variable is used after it was std::move()'ed elsewhere. This diagnostic was discussed at https://groups.google.com/a/chromium.org/d/topic/cxx/IDD4f6Pmnxc/discussion (see also the doc attached to that thread). Bug: 1122844 Change-Id: I9aa1996dcac8307259d86821ce7b4f5cf2175dbe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383010Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
George Burgess <gbiv@chromium.org> Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#802874}
-
zhaoyangli authored
Replicated iOS 13.6 form factors to iOS 14 in the full-configs CQ bot. Moved iOS12 runtimes on full configs bot to ios-simulator-noncq for capacity restrain. Bug: 1123108 Change-Id: I34109ef9278b916de94457b12f4c247eac54d073 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382223Reviewed-by:
Justin Cohen <justincohen@chromium.org> Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org> Cr-Commit-Position: refs/heads/master@{#802873}
-
Richard Knoll authored
This removes all hardcoded strings from the HTML WebUI files and adds proper strings to the grdp file. Also replaces the button to configure the device name externally with an inline edit control. Bug: None Change-Id: Ibd2a17b5b9b8234633399fd6c50fae11fc15e45c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2376857 Commit-Queue: Richard Knoll <knollr@chromium.org> Reviewed-by:
James Vecore <vecore@google.com> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#802872}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/9100465414a9..5ac807b26749 2020-08-28 cclao@google.com Vulkan: Let stageSubresourceUpdateAndGetData use ctx staging buffer 2020-08-28 jmadill@chromium.org Enable -Wreturn-std-move-in-c++11. 2020-08-28 jmadill@chromium.org Enable -Wbad-function-cast. 2020-08-28 timvp@google.com Add third_party/abseil-cpp to ANGLE 2020-08-28 cclao@google.com Vulkan: Fix alignment issue with context staging buffer 2020-08-28 lehoangq@gmail.com Metal: Fix handling of vertex attrib offset not multiple of 4 2020-08-28 jmadill@chromium.org Enable two override suggestion warnings. 2020-08-28 cnorthrop@google.com Capture/Replay: Track groups of strings If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC courtneygo@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: courtneygo@google.com Test: Test: Manhattan MEC works again Change-Id: Ic6832b238622ddf56538b74026ca8185cb59a9df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382699Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802871}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/fb7c6ea5..4b3de99d Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Iad218fcbb8d2f936003619ed6f262dd3841e0620 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382911Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#802870}
-
Peng Huang authored
This is a reland of a37811d1 Original change's description: > SkiaOutputDevice: use buffer format passed to Reshape > > Bug: 1108406 > Change-Id: Iaca79e3b6c1d66e421711c46f75a1cb614b7f961 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2378745 > Reviewed-by: Vasiliy Telezhnikov <vasilyt@chromium.org> > Commit-Queue: Peng Huang <penghuang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#802450} Bug: 1108406 Change-Id: Iec517b1f5b711d9457a09841cedd2db22b520aee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382197Reviewed-by:
Vasiliy Telezhnikov <vasilyt@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#802869}
-
Andrey Kosyakov authored
This reverts commit 82b85893. Reason for revert: let's give this test another change. it seems to reliable pass for me locally and I think the original failure was rather due to a front-end race fixed here: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2242769 Original change's description: > Revert "DevTools: add tests for extensions on DOM UI pages" > > This reverts commit 3c0f2556. > > Reason for revert: test is extremely flaky > > Per https://analysis.chromium.org/p/chromium/flake-portal/flakes/occurrences?key=ag9zfmZpbmRpdC1mb3ItbWVyUgsSBUZsYWtlIkdjaHJvbWl1bUBicm93c2VyX3Rlc3RzQERldlRvb2xzRXh0ZW5zaW9uVGVzdC5UZXN0RXZhbHVhdGVPbkNocm9tZVNjaGVtZQw this flakily fails about 10 times an hour. > > Original change's description: > > DevTools: add tests for extensions on DOM UI pages > > > > This is the chrome-side counterpart of > > https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2128732 > > > > Bug: 1059577, 795595 > > Change-Id: Iec2ee772a42b4c7bc2249627c0839f7506f0cd1d > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129344 > > Reviewed-by: Devlin <rdevlin.cronin@chromium.org> > > Reviewed-by: Dmitry Gozman <dgozman@chromium.org> > > Commit-Queue: Andrey Kosyakov <caseq@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#756375} > > TBR=dgozman@chromium.org,rdevlin.cronin@chromium.org,caseq@chromium.org,bmeurer@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 1059577, 795595 > Change-Id: I2919088167b064086b315d8c3a64df569d95c844 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2188512 > Reviewed-by: Marijn Kruisselbrink <mek@chromium.org> > Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> > Cr-Commit-Position: refs/heads/master@{#766575} TBR=dgozman@chromium.org,mek@chromium.org,rdevlin.cronin@chromium.org,caseq@chromium.org,bmeurer@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1059577 Bug: 795595 Change-Id: Ibd719500160685664de90415d718b88b4621b52e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382487Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#802868}
-
Himanshu Jaju authored
Removes pending TODO and implements the test for registering a send surface when we are already receiving incoming files. Change-Id: I9dd8955b4c423ed6b567ebb2028ed839b9a1f63f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382753 Auto-Submit: Himanshu Jaju <himanshujaju@chromium.org> Commit-Queue: James Vecore <vecore@google.com> Reviewed-by:
James Vecore <vecore@google.com> Cr-Commit-Position: refs/heads/master@{#802867}
-
Austin Eng authored
Mapped GPUBuffers' memory is owned and externally managed by dawn_wire. Using EmptyDeleter tells the backing store this allocation does not hold memory and will not increase GC pressure. Bug: 1061960 Change-Id: Ia44a5dc1be89d7b55540247c90bce614172fa2ae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380086 Commit-Queue: Austin Eng <enga@chromium.org> Commit-Queue: Kai Ninomiya <kainino@chromium.org> Reviewed-by:
Kai Ninomiya <kainino@chromium.org> Cr-Commit-Position: refs/heads/master@{#802866}
-