- 10 Nov, 2020 40 commits
-
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/33858054c805..139be0697741 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 hbos@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: hbos@google.com Change-Id: Ifa396027f8d9d66c3b00584b33c31e3c7ce5b7df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528951Reviewed-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@{#825759}
-
Illia Klimov authored
Due to Flash/Plugins deprecation this CL removes UNSANDBOXED_PLUGINS from the SiteSettings page. Bug: 1138156 Change-Id: I443f9aa2626216dc0c1f357e70fb0e3f4e274119 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2517688Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Illia Klimov <elklm@google.com> Auto-Submit: Illia Klimov <elklm@google.com> Cr-Commit-Position: refs/heads/master@{#825758}
-
Honglin Yu authored
NoDestructor should be preferred over Singleton according to the comments in singleton.h and https://bugs.chromium.org/p/chromium/issues/detail?id=925323. And this *may* also relate to https://bugs.chromium.org/p/chromium/issues/detail?id=922327 BUG=chromium:925323, chromium:922327 TEST=cq Change-Id: Icb014d0eae9e0032a2a5ca56dcec59cf72300315 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526643Reviewed-by:
Alexey Baskakov <loyso@chromium.org> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Honglin Yu <honglinyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#825757}
-
David Roger authored
The AppShortcutManager was not correctly unregistering as a ProfileAttributesStorage observer. The reason is that the AppShortcutManager is destroyed as part of the ProfileManager destruction. At this time, g_browser_process()->profile_manager() already returns nullptr. AppShortcutManager was happily skipping the unregistration in that case, because of test-related logic. I did not repro the crash and verified the fix, but this CL should solve the issue. Fixed: 1145906 Change-Id: I2ad4e574ad128ec920f94978446ae33c8dc73a46 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2520776Reviewed-by:
ccameron <ccameron@chromium.org> Commit-Queue: David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#825756}
-
dpapad authored
With this CL, HTML imports inovlving chrome-extension:// URLs are correctly translated to JS imports. For example chrome-extension://path/to/folder/foo.html becomes chrome-extension://path/to/folder/foo.m.js This is necessary to unblock CrOS FileManager migration to JS modules. Fixed: 1145876 Change-Id: Ie2e02b6925108db280f626be9c1ad0c261ffd797 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526501 Commit-Queue: dpapad <dpapad@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#825755}
-
Morten Stenshorne authored
Avoid undefined behavior, such as negative values, when casting from double to int. Bug: 1116832 Change-Id: I49d4d5c6e73a9441d20c8a502c3d9d8cff71b7d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528529Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#825754}
-
Heng-Ruey Hsu authored
In order to support multiple streams for one camera, we need to support multiple clients in camera device context to allocate buffer from corresponding client. Bug: b:151047537 Test: manually Change-Id: I10855309a3dc4a6883af0d852180371e2f807dbe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2351930Reviewed-by:
Dan Sanders <sandersd@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Wei Lee <wtlee@chromium.org> Commit-Queue: Heng-ruey Hsu <henryhsu@chromium.org> Auto-Submit: Heng-ruey Hsu <henryhsu@chromium.org> Cr-Commit-Position: refs/heads/master@{#825753}
-
Camillo Bruni authored
- Use the ParametrizedModuletTest for worklet tests - Split ParametrizedModuleTestBase from ParametrizedModuleTest Bug: 1132793 Change-Id: I28f89c0f1c1a1093d43922eaf9e53949ae4e2bf1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465902 Commit-Queue: Camillo Bruni <cbruni@chromium.org> Reviewed-by:
Robert Flack <flackr@chromium.org> Reviewed-by:
Hongchan Choi <hongchan@chromium.org> Reviewed-by:
Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#825752}
-
Antonio Gomes authored
... instead of ash_constants. It is used in default_frame_header.cc, that will be moved to //chromeos/ui/frame next. BUG=1113900 R=jamescook@chromium.org Change-Id: Ibd4417c516ae912964fbd07179912797257d308b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2522921 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#825751}
-
Ben Reich authored
The structure of the coverage directory is internal details of the DevToolsListener class, extract the structure into a static function |SetupCoverageStore| to avoid duplication of setup code. R=noel@chromium.org --devtools-code-coverage Bug: 1113941 Test: browser_tests --gtest_filter=*FilesApp* --devtools-code-coverage Test: browser_tests --gtest_filter=*DevToolsListener* Change-Id: Ia5d2a0553eeb791af707734bb5e28d8b3ef1c304 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528607 Auto-Submit: Ben Reich <benreich@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#825750}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/eba8b028ddbc..38f3d771bd8e If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-linux-chromium Please CC perfetto-bugs@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 Tbr: perfetto-bugs@google.com Change-Id: I3bfb221dccb7b80e5bb3cd43ca9cdee9fae8a897 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528952Reviewed-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@{#825749}
-
Kim-Anh Tran authored
Disable to allow for new module linear memory inspector module to land. Bug: chromium:1110202 Change-Id: I7d6ea76a1fd9803f22e04af3aab9fac70befa9ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526442Reviewed-by:
Alex Rudenko <alexrudenko@chromium.org> Commit-Queue: Kim-Anh Tran <kimanh@chromium.org> Cr-Commit-Position: refs/heads/master@{#825748}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/1d532cbd0baa..eba8b028ddbc If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-mac-chromium Please CC perfetto-bugs@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 Tbr: perfetto-bugs@google.com Change-Id: I1848bd59a45fb15ad7ffb06a94333d5dacc5649d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528616Reviewed-by:
-
Wei Lee authored
This CL fixes a bug that on SWA the intent with shouldHandleResult=false does not finish right after the app startup. Bug: b/172780736 Test: tast run [DUT] camera.CCAUIIntent.* Change-Id: Ie3069660d6e71d505caf4a9922fa1967deafb405 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526854 Commit-Queue: Wei Lee <wtlee@chromium.org> Commit-Queue: Shik Chen <shik@chromium.org> Auto-Submit: Wei Lee <wtlee@chromium.org> Reviewed-by:
Shik Chen <shik@chromium.org> Cr-Commit-Position: refs/heads/master@{#825746}
-
Danila Kuzmin authored
unbinding Bug: 1142565 Change-Id: I4d201961aba5e78ec1c3d9c0291c8bba6cf4c36b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2517523 Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#825745}
-
Victor-Gabriel Savu authored
Add LTS to the reported feedback in feedback reports. This will allow queries in Listnr to either include or exclude LTS customers. Bug: chromium:1146354 Change-Id: Iee4c724206666bff804991bf8c00610883ac5cb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2521182Reviewed-by:
Achuith Bhandarkar <achuith@chromium.org> Reviewed-by:
Miriam Zimmerman <mutexlox@chromium.org> Commit-Queue: Victor-Gabriel Savu <vsavu@google.com> Cr-Commit-Position: refs/heads/master@{#825744}
-
Christian Dullweber authored
Close settings activity before loading a URL to try to avoid flakiness. Bug: 1139480 Change-Id: I4e9b617941f07fc3baa313ddf10b6c9cb656990c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526341Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Reviewed-by:
Ehimare Okoyomon <eokoyomon@chromium.org> Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Commit-Queue: Christian Dullweber <dullweber@chromium.org> Cr-Commit-Position: refs/heads/master@{#825743}
-
Mikel Astiz authored
Starting with http://crrev.com/c/2434329, which actually landed some time ago now, TwoClientSearchEnginesSyncTest::SetupClients() incorrectly assumed that there always exist two clients. The caveat is, for E2E tests, ResetSyncForPrimaryAccount() sets up exactly one client as a convenient way to reset sync. Change-Id: Ie7967694e13090a5317de4d2422ba2fadecd7aa9 Bug: 1147195 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527203Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#825742}
-
Nohemi Fernandez authored
Previously, we only had the ability to include a title text in an infobar. Following crrev.com/c/2521619 we are able to include both. This patch updates sign-in notification infobar to include a title, "Signed in to Chrome" and the user email as the message. Bug: 1145592 Change-Id: Ibd1b91737d3cfd6d30ea15cdec5fd11f49cba268 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526105Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Commit-Queue: Nohemi Fernandez <fernandex@chromium.org> Cr-Commit-Position: refs/heads/master@{#825741}
-
danielng authored
Using the metrics added in earlier to track information about the start-up flow for borealis. Cq-Depend: 2509072 Bug: b/171339461 Change-Id: If0e0f3c212ab1ff4cfddf43b10104f453a99e0e7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2517347 Commit-Queue: Daniel Ng <danielng@google.com> Reviewed-by:
Nic Hollingum <hollingum@google.com> Cr-Commit-Position: refs/heads/master@{#825740}
-
Leo Zhang authored
This reverts commit 8ce3c94e. Reason for revert: Tested locally with the latest betty image by run `tast run -build=false 127.0.0.1:9222 inputs.VirtualKeyboard*` after patching the CL. Previous failed tests passed this time. Original change's description: > Revert "Reland "[IME] Remove IME UIL flags."" > > This reverts commit b7ab664d. > > Reason for revert: The inputs.VirtualKeyboard are failing again (right after this CL was relanded. > https://ci.chromium.org/p/chrome/builders/ci/chromeos-betty-pi-arc-cfi-thin-lto-chrome/12767 > (see the overview and blamelist tabs) > > Original change's description: > > Reland "[IME] Remove IME UIL flags." > > > > This reverts commit 583caa18. > > > > Reason for revert: The failed tast tests have been fixed in crrev.com/c/2509530. So reland the flags updates now. > > > > Original change's description: > > > Revert "[IME] Remove IME UIL flags." > > > > > > This reverts commit 20fb36e3. > > > > > > Reason for revert: speculative revert for chrome_all_tast_tests > > > failures affecting inputs.VirtualKeyboard* tests, for example > > > https://ci.chromium.org/p/chrome/builders/ci/chromeos-betty-pi-arc-chrome/9751 > > > > > > Original change's description: > > > > [IME] Remove IME UIL flags. > > > > > > > > Simplify the UIL/IME service dev/testing. > > > > > > > > TEST=simple chrome > > > > BUG=b/171846787 > > > > > > > > Change-Id: Ibeef37bc89b848cffd5a4a08d5619d41a5c35b33 > > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2507236 > > > > Reviewed-by: Darren Shen <shend@chromium.org> > > > > Commit-Queue: Leo Zhang <googleo@chromium.org> > > > > Cr-Commit-Position: refs/heads/master@{#822479} > > > > > > TBR=wuyingbing@chromium.org,shend@chromium.org,googleo@chromium.org > > > > > > Change-Id: I2c2a4ccda60320dd7099afcfc28a12122b11e982 > > > No-Presubmit: true > > > No-Tree-Checks: true > > > No-Try: true > > > Bug: b/171846787 > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2509749 > > > Reviewed-by: Mikel Astiz <mastiz@chromium.org> > > > Commit-Queue: Mikel Astiz <mastiz@chromium.org> > > > Cr-Commit-Position: refs/heads/master@{#822582} > > > > TBR=wuyingbing@chromium.org,mastiz@chromium.org,shend@chromium.org,googleo@chromium.org > > > > # Not skipping CQ checks because original CL landed > 1 day ago. > > > > Bug: b/171846787 > > Change-Id: Ia120b5b89fba677cf3f9ecc592ebc1bd5db14804 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2512630 > > Reviewed-by: Leo Zhang <googleo@chromium.org> > > Commit-Queue: Leo Zhang <googleo@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#823077} > > TBR=wuyingbing@chromium.org,mastiz@chromium.org,shend@chromium.org,googleo@chromium.org > > Change-Id: I8f8cb41509b168d5e3dbc0e77f51f48077330d9f > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: b/171846787 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2513647 > Reviewed-by: Michael Ershov <miersh@google.com> > Commit-Queue: Michael Ershov <miersh@google.com> > Cr-Commit-Position: refs/heads/master@{#823097} TBR=wuyingbing@chromium.org,mastiz@chromium.org,shend@chromium.org,googleo@chromium.org,miersh@google.com # Not skipping CQ checks because original CL landed > 1 day ago. Bug: b/171846787 Change-Id: I69244f04c9be5aed8fbf858fc6a1cc319dfcf540 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528878Reviewed-by:
Leo Zhang <googleo@chromium.org> Commit-Queue: Leo Zhang <googleo@chromium.org> Cr-Commit-Position: refs/heads/master@{#825739}
-
Jan Wilken Dörrie authored
This reverts commit 1131840b. Reason for revert: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89381 got fixed and this change interferes with https://crrev.com/c/2494941. That change added a new sorted_unique_t constructor tag to base::flat_tree, but wasn't taking effect for flat_map, since those constructors were not inherited. Original change's description: > flat_map: Readd constructors and assignment operator overloads. > > https://chromium-review.googlesource.com/c/chromium/src/+/705955 ("base::flat_map > was missing defaults for duplication handling") removed them in favor of > inheriting them from flat_tree. > > This can cause problems with GCC due to > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84782, and the build has been > failing with it since > https://chromium-review.googlesource.com/1000393 ("Mojo C++ Bindings: > unordered_map => flat_map") with a message like this: > > gen/mojo/public/interfaces/bindings/tests/test_unions.mojom.cc: In member function ‘void mojo::test::ObjectUnion::set_f_map_int8(const base::flat_map<std::__cxx11::basic_string<char>, signed char>&)’: > gen/mojo/public/interfaces/bindings/tests/test_unions.mojom.cc:455:30: error: use of deleted function ‘base::flat_map<std::__cxx11::basic_string<char>, signed char>::flat_map(const base::flat_map<std::__cxx11::basic_string<char>, signed char>&)’ > std::move(f_map_int8)); > ^ > In file included from ../../mojo/public/cpp/bindings/clone_traits.h:11:0, > from gen/mojo/public/interfaces/bindings/tests/test_unions.mojom.h:22, > from gen/mojo/public/interfaces/bindings/tests/test_unions.mojom.cc:15: > ../../base/containers/flat_map.h:152:7: note: ‘base::flat_map<std::__cxx11::basic_string<char>, signed char>::flat_map(const base::flat_map<std::__cxx11::basic_string<char>, signed char>&)’ is implicitly declared as deleted because ‘base::flat_map<std::__cxx11::basic_string<char>, signed char>’ declares a move constructor or move assignment operator > class flat_map : public ::base::internal::flat_tree< > ^~~~~~~~ > > Work around it by replacing the inheritance statements with actual > constructors and operator=() overloads again. This change does not > completely revert https://chromium-review.googlesource.com/1000393, as the > constructors maintain their new signature (so that they look like > flat_tree's). > > Bug: 819294, 837221 > Change-Id: I8f37a2c8aa269b3c1025cdc009518a7709e95947 > Reviewed-on: https://chromium-review.googlesource.com/1038103 > Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: kylechar <kylechar@chromium.org> > Cr-Commit-Position: refs/heads/master@{#556168} TBR=danakj@chromium.org,dcheng@chromium.org,raphael.kubo.da.costa@intel.com,kylechar@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 819294 Bug: 837221 Change-Id: I72915a49981c114f16eaa866de226e9cd242a348 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526100Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#825738}
-
Vasilii Sukhanov authored
The tests were creating PasswordStoreX early before it was overridden by the testing factory. Now the tests will create TestPasswordStore from the beginning. Fixed: 1146503 Change-Id: I2701d9d44064536acf2d52a0721def71f9e69473 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527382 Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#825737}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/4f7c1ce4..bc5c4781 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: I733458bd3de2139afb22d8544f7dc30949e7e3e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527349Reviewed-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@{#825736}
-
Sigurdur Asgeirsson authored
Replace ScopedObserver with base::ScopedObservation in /content/browser/storage_partition_impl_unittest.cc. ScopedObserver is being deprecated in favor of two new classes: - base::ScopedObservation for observers that only ever observe a single source. - base::ScopedMultiSourceObservation for observers that do or may observe more than a single source. This CL was uploaded by git cl split. R=kinuko@chromium.org Change-Id: I87a97adbc8cd1ad3e9bb6131289df74f9afb691a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526167 Auto-Submit: Sigurður Ásgeirsson <siggi@chromium.org> Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#825735}
-
chromium-autoroll authored
Roll Chrome Win64 PGO profile from chrome-win64-master-1604955437-c91abc52162c185f79a73166dbe14ec0b1c66fea.profdata to chrome-win64-master-1604966213-11b581102b343c7a1c3accf4f99e5d9072414c62.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-win64-chromium Please CC pgo-profile-sheriffs@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:win64-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: If5c9183338acc5596272c275f70db23bb705f517 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527956Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/7f9e30c27f7b..aa1e66743da1 2020-11-10 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: Id3bdcb7e2138ced99ebe8ef3f7cd31f60648165b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528726Reviewed-by:
-
Luciano Pacheco authored
Preparation for JS Modules/Polymer 3, the polymer_modulizer() expects the Polymer element to not be assigned to a variable, so change this for <files-toggle-ripple>. The assignment to variable was used to be able to refer to the type name, however the correct way to refer to type of the Polymer element is to use the `Element` suffix, replace the references to use the right format. When checking this I realized that the externs `files_elements.js` isn't used so removed it too. Bug: 1133186 Change-Id: I09b1929bc8fc9f2efa42cb646a008203f9f46f92 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526892 Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Jeremie Boulic <jboulic@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Jeremie Boulic <jboulic@chromium.org> Cr-Commit-Position: refs/heads/master@{#825732}
-
Alan Cutter authored
These windows should let the title bar bleed into the WebContents below without any separator. Web app windows with a title bar require a separator line to show the outline of any tabs when adjacent colors are too similar. Before: https://bugs.chromium.org/p/chromium/issues/attachment?aid=475568&signed_aid=70iR6Zl6eyaEks4wOpBBZg==&inline=1 After: https://bugs.chromium.org/p/chromium/issues/attachment?aid=475569&signed_aid=MoU4hgqV5NSMUiizHK5tsw==&inline=1 This CL effectively reverts the change made in https://chromium-review.googlesource.com/c/chromium/src/+/2367584/5/chrome/browser/ui/views/frame/browser_non_client_frame_view.cc to only allow strokes for tab strip web apps instead of all web apps. Bug: 1137301 Change-Id: Ie0dd29872765921f7c1c8840c39bd70736423bed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526756Reviewed-by:
Peter Kasting <pkasting@chromium.org> Reviewed-by:
Joel Hockey <joelhockey@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#825731}
-
Nico Weber authored
With a chance of 1/256, we'd read past the end of kValidLiteralValues. In practice, that read would return zeroes, which we'd write to kTestLibData -- but zeroes aren't valid literal values, so that would then make TestShader() crash when it tried to link the program. Not sure if this is related to crbug.com/1051923, but it's in the same area at least. Add a components/metal_util:unit_tests target. Covering code is useful even if the test doesn't verify a lot, because then the ASan bots can find stuff like this for us. (...but in this case, asan doesn't find the out-of-bounds read because we build with `-asan-globals=0` on macOS because of https://crbug.com/352073 . But the test also checks for non-0 and runs the code often enough that the test consistently fails before the fix and consistently passes with it.) Drive-by no-op cleanups while here: - Remove EXPORTs on enums, they don't have any effect - Remove redundant is_ios block in components/BUILD.gn Bug: 1147027,1051923,1085899 Cq-Include-Trybots: luci.chromium.try:mac_chromium_asan_rel_ng Change-Id: Ia7f9735e450c70eb64b826a499dc60690714112e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526161 Auto-Submit: Nico Weber <thakis@chromium.org> Reviewed-by:
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1604966213-a7264b10c906377cb4db39810bd54ed14433c102.profdata to chrome-mac-master-1604987499-7dfc90285fa0600b167fa15e7d7c9c3f744524d3.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 pgo-profile-sheriffs@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:mac-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: I0da505edb9e653dc21ccfeb40f500b6154a2db5d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528725Reviewed-by:
-
Anupam Snigdha authored
Initialized the boundingRect property of the virtualKeyboard object to default values during construction. Bug: 1147243 Change-Id: I2a751d08aaff2c70f3cf06931e4b8d183a29b413 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527564 Commit-Queue: Anupam Snigdha <snianu@microsoft.com> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#825728}
-
Devlin Cronin authored
Add support for a new property on function types, serializableFunction. IDL Example: callback SerializableFunction = void(); dictionary SerializableFunctionObject { [serializableFunction]SerializableFunction func; }; JSON Example: { "type": "object", "id": "SerializableFunctionObject", "properties": { "func": { "name": "func", "serializableFunction": true, "type": "function", "parameters": [] } } } If this trait is present, functions will be serialized and deserialized as strings, rather than as empty base::DictionaryValues. Add testing for the same. Note: this only provides support in the JSON schema compiler (which includes support in our auto-generated strong-typing code); a follow-up will add support in our bindings code for parsing the values passed from extensions. Bug: 1144841 Change-Id: I430fb211166a5ccffaa6b478642e98115fd25565 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2518717 Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Reviewed-by:Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#825727}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/1d532cbd0baa..eba8b028ddbc If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-linux-chromium Please CC perfetto-bugs@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 Tbr: perfetto-bugs@google.com Change-Id: Ib7558e84f081cf3a3b530e38575a45321333e9ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527052Reviewed-by:
-
Devlin Cronin authored
Short-circuit our localization logic to only attempt to localize a file if it may be required. Bug: None Change-Id: I0b418dbad004a6cc890ae02713f9ebeeed9c92fb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2518199Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/ca2031075565..33858054c805 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 ricea@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: ricea@google.com Change-Id: If55f62b089303fcfb835fa3b6556194243058d22 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528724Reviewed-by:
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/eba8b028ddbc..38f3d771bd8e 2020-11-10 treehugger-gerrit@google.com Merge "Use std::any_of to find if any guardrails are used." If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@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:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: Ied3384d6a2c46baf60f67dfe0f13b3f4e2b1a3a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527691Reviewed-by:
-
John Palmer authored
Bug: 1111135 Change-Id: I0ecb9491026851d01331a95d20b29fc1e5caf0b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2521552 Commit-Queue: John Palmer <jopalmer@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Reviewed-by:
My Nguyen <myy@chromium.org> Cr-Commit-Position: refs/heads/master@{#825722}
-
Ryo Hashimoto authored
This CL depends on CL:2517272 BUG=b:171408795 TEST=components_unittests TEST=Launch Play Store Change-Id: Ie79568e6f03b3433b8d9af1b50fd1db817d76794 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2517269 Commit-Queue: Ryo Hashimoto <hashimoto@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Cr-Commit-Position: refs/heads/master@{#825721}
-
Yu-Hsuan Hsu authored
Change stream type to CRAS_CLIENT_TYPE_LACROS for lacros-chrome. The stream type is used while recording UMA logs in CRAS. So we can separate data between ash-chrome and lacros-chrome. BUG=b:167941044 Change-Id: I35a8320bd9b5d372d9f5ac883078d010ed42ee9e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2521473Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Yu-Hsuan Hsu <yuhsuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#825720}
-
