- 30 Jan, 2020 40 commits
-
-
Peng Huang authored
With AMD GPU, vulkan cannot submit a command buffer with waiting on a semaphore which is signaled by GL. It is a driver bug reported at https://gitlab.freedesktop.org/mesa/mesa/issues/2433 . So disable GL_EXT_memory_object_fd and GL_EXT_semaphore_fd extensions. Without those extensions, chrome will fallback to copy pixels from vkimage and gl texture instead of sharing the same backing memory object. Bug: 1046462 Change-Id: I231af9a626d692dbde89d9cd2f8a4bdc5ddf6eae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2027889Reviewed-by:
Jonathan Backer <backer@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Auto-Submit: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#736887}
-
Charlie Hu authored
Previously if a http response has both Document-Policy and Require-Document-Policy header and they are incompatible, the document will be blocked, which is not the correct behavior. This CL corrects the behavior of document policy under such condition. Bug: 993790 Change-Id: I5fc6b5c213fa7d56d473a8ea7e10235dd418ba42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015443 Commit-Queue: Charlie Hu <chenleihu@google.com> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/heads/master@{#736886}
-
Benoît Lizé authored
Reason: failing, see linked bug. Bug: 599650 TBR: mcasas Change-Id: Iec795ad764a0a2d78dfb77e3b5f752b857da6bd1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030171 Commit-Queue: Benoit L <lizeb@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Cr-Commit-Position: refs/heads/master@{#736885}
-
Omer Katz authored
This is part of an ongoing effort to simplify heap collections and limit the types they can support (specifically, this CL removes the only existing non-member usage of HeapListHashSet). Bug: 1047147 Change-Id: I1c5de9f3a97588c22946eed03048159fd720b502 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2024896Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#736884}
-
Guido Urdaneta authored
Bug: 1045381 Change-Id: I285a44d32d47522786b1676b708b0723f336295b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2031144Reviewed-by:
Marina Ciocea <marinaciocea@chromium.org> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#736883}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/d69935c114aa..5dca3f1336a6 git log d69935c114aa..5dca3f1336a6 --date=short --first-parent --format='%ad %ae %s' 2020-01-30 peah@webrtc.org Add floating point support for writing and reading wav files 2020-01-30 philipel@webrtc.org Remove unused NextFrame function from FrameBuffer. Created with: gclient setdep -r src/third_party/webrtc@5dca3f1336a6 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@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/+/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: Iae37805d25be988d08bcebd60fc954b7d7dd2e7c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2031125Reviewed-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@{#736882}
-
Colin Blundell authored
Follows the componentization of the production class. Bug: 1030692 Change-Id: Ie64cabaa924d06502cd9eb7ad84097c579c554c1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2029510Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#736881}
-
Bruce Long authored
Remove unused locale_set from method LanguageSettingsPrivateGetLanguageListFunction::Run Bug: 1046955 Change-Id: Ie913d5700ea9f4bf71d3824b14f8fbb0f754f942 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028933Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Bruce Long <brlong@microsoft.com> Cr-Commit-Position: refs/heads/master@{#736880}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/google/shaderc/+log/b8c0752438c0..9f95ca93ba65 git log b8c0752438c0..9f95ca93ba65 --date=short --first-parent --format='%ad %ae %s' 2020-01-30 dneto@google.com Better help for glslc -S (#978) Created with: gclient setdep -r src/third_party/shaderc/src@9f95ca93ba65 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/shaderc-chromium-autoroll Please CC radial-bots+chrome-roll@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/+/master/autoroll/README.md Bug: None Tbr: radial-bots+chrome-roll@google.com Change-Id: I6c1ecae8412a2ed811a0fb2b4182a1d42334f562 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030591Reviewed-by:
-
Elly Fong-Jones authored
This change: 1) Removes ConfirmBubbleModel::GetButtons(), because every user had both buttons. Design-wise this makes sense - a ConfirmBubbleModel represents a yes/no choice, so having one or both of the buttons absent doesn't make much sense. 2) Removes ConfirmBubbleModel::BubbleButton, which duplicates ui::DialogButton 3) Removes ConfirmBubbleViews::IsDialogButtonEnabled() - because of point (1), this always returned true. Bug: 1011446 Change-Id: I04bf65b7145ce00d70887feef1d0a6d218255697 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028217 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#736878}
-
Hidehiko Abe authored
BUG=None TEST=Build locally. Change-Id: Ibf130dcb59c094cedb6fc94ed62f74d97a299616 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2029406 Commit-Queue: Hidehiko Abe <hidehiko@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#736877}
-
Guido Urdaneta authored
Bug: 1043828 Change-Id: I41db446ceec894c06b15c40a673f290f8ba2e533 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030886 Commit-Queue: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Armando Miraglia <armax@chromium.org> Cr-Commit-Position: refs/heads/master@{#736876}
-
Fredrik Söderquist authored
The original is (flakily) timing out on Mac - most likely because of the many readbacks. Split the test in smaller, reasonably logical, chunks to hopefully avoid timing out. Bug: 1041846 Change-Id: I43a543cd2dbc58fd9a1f412f9f77e2fb0eadc921 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030765 Commit-Queue: Fernando Serboncini <fserb@chromium.org> Auto-Submit: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Fernando Serboncini <fserb@chromium.org> Cr-Commit-Position: refs/heads/master@{#736875}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/3941f82d7c3f..096422a54f2c git log 3941f82d7c3f..096422a54f2c --date=short --first-parent --format='%ad %ae %s' 2020-01-30 szuend@chromium.org Prepare NetworkPersistenceManager for source map asyncification Created with: gclient setdep -r src/third_party/devtools-frontend/src@096422a54f2c 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/+/master/autoroll/README.md Bug: chromium:1032016 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I72a503d0580be46e7f4b942cca8ed0fcb913a802 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030593Reviewed-by:
-
Mohamed Heikal authored
We currently use aapt(1) to generate an R.txt file so that we can compile our resources and java targets concurrently. This cl replaces aapt with a custom res/ parser that extracts resource java constant names to output a dummy R.txt file. This new implementation has a significant speedup over aapt (60% on average). Bug: 820460 Change-Id: I7c912390a7789385a84a4102aee6bdb70011b248 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015609Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Mohamed Heikal <mheikal@chromium.org> Cr-Commit-Position: refs/heads/master@{#736873}
-
Guido Urdaneta authored
Bug: 1004546 Change-Id: Id95038ced72c34c5a46d71dde8183d34e02798d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030965 Commit-Queue: Guido Urdaneta <guidou@chromium.org> Commit-Queue: Marina Ciocea <marinaciocea@chromium.org> Auto-Submit: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
-
Ian Clelland authored
There was a case missing from the feature policy boolean value 'less-than' comparator which resulted in an a < b comparison sometimes returning the wrong value. Tests added for all cases for bools and doubles. Change-Id: If873ca2b173b5a3b60a3091c030c1482b6b6bcbc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028628Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Ian Clelland <iclelland@chromium.org> Cr-Commit-Position: refs/heads/master@{#736871}
-
John Chen authored
Bug: 1028242 Change-Id: I5a9eeb935c71a83bbafe1a0bdea9374473dd8366 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028738Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#736870}
-
Egor Pasko authored
The main purpose is to replace the default CacheType argument with a normal one. The callers are already passing this parameter explicitly in the vast majority of cases, as it is really helpful to know whether the operation will be persisted across platform instances. For ClearCachedMetadata(): all callers are passing the parameter explicitly. We will continue to need both modes (clearing on disk for privacy reasons, clearing from memory to free up some memory). Making it non-default is trivial. For SetCachedMetadata(): the documentation says that is should always persist, hence it seems that this was the intention initially. The kCacheLocally is used only in tests. Making a test-only carveout with DisableSendToPlatformForTesting(). The reason for not persisting in those tests is not clear to me, perhaps this makes the tests run faster? Increasing the size of the class is slightly unfortunate, I could not find an elegant solution to overcome this. Though the size of these objects is small compared to all of the serialized data, so it feels OK. Bug: 1045052 Change-Id: Ifec42fbd66498c6860b32d86a25d3e857a97b4c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2019246Reviewed-by:
Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Leszek Swirski <leszeks@chromium.org> Reviewed-by:
-
Olya Kalitova authored
Adds message in transparency panel when CrostiniAnsiblePlaybook policy is set for user. If CrostiniAnsiblePlaybook policy is set and Crostini reporting is enabled the new message is set, when only reporting is enabled then old message for Crostini reporting is set. TEST: deployed to DUT and tested using YAPS Bug: 1018775 Change-Id: Ia4df50c3acf69fcc613a8888752706c1e0e6ed0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2000624Reviewed-by:
Thiemo Nagel <tnagel@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Commit-Queue: Olya Kalitova <okalitova@chromium.org> Cr-Commit-Position: refs/heads/master@{#736868}
-
Xida Chen authored
This CL marks virtual/omt-service-worker-startup/http/tests/serviceworker/update-served-from-cache.html and its non-virtual tests flaky. TBR=bashi@chromium.org NOTRY=true Bug: 1047208 Change-Id: Ic630ad8f072858653e3f199cef41a4a41730da33 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2031184 Commit-Queue: Xida Chen <xidachen@chromium.org> Reviewed-by:
Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#736867}
-
Mikel Astiz authored
BookmarkModel is expected to have a unique GUID per bookmark. BookmarkModel does not currently have [D]CHECKs to enforce this, but it does get verified upon loading of bookmarks from storage. Upon sync's initial merge, the presence of duplicate GUIDs locally is specially problematic, in particular if GUID-based merging logic is enabled (currently disabled by default). This is because local nodes may end up untracked, which violates the invariant for bookmark sync to work (and may later run into [D]CHECKs). Change-Id: I76f6139ae341fe1ed4e1f37cc37d96704be0910d Bug: 978430 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030764 Auto-Submit: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#736866}
-
Takuto Ikuta authored
python client is being deprecated. Bug: 984869 Change-Id: Ia0613c0bec75ab752554900eff3928115cd69988 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030384 Commit-Queue: John Budorick <jbudorick@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Auto-Submit: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#736865}
-
Elly Fong-Jones authored
Replaced with direct use of the setters. Bug: 1011446 Change-Id: Ia054462bbe3e1585430b3fe4102c694366a78827 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028197Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#736864}
-
Geoff Lang authored
This reverts commit 248bfd1f. Reason for revert: Broke some Android WebGL video tests BUG=1047201 Original change's description: > Don't create manager objects in ContextGroup when using the passthrough decoder > > These objects are not used when the passthrough command decoder is in > use. This saves several GPU memory allocations in TextureManager for > default and black textures. > > Saves ~2 MB per context. > > BUG=1046613 > BUG=1030835 > > Change-Id: I66c6acf28580742bc313d8cf79de80f7f7d98305 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028291 > Commit-Queue: Geoff Lang <geofflang@chromium.org> > Reviewed-by: Zhenyao Mo <zmo@chromium.org> > Reviewed-by: Dan Sanders <sandersd@chromium.org> > Cr-Commit-Position: refs/heads/master@{#736631} TBR=zmo@chromium.org,geofflang@chromium.org,sandersd@chromium.org Change-Id: I82ca36bd0b449e743ef819fbfe6ad161b219a8d0 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1046613, 1030835 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2031164Reviewed-by:
Geoff Lang <geofflang@chromium.org> Commit-Queue: Geoff Lang <geofflang@chromium.org> Cr-Commit-Position: refs/heads/master@{#736863}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/81c7a60961cc..d69935c114aa git log 81c7a60961cc..d69935c114aa --date=short --first-parent --format='%ad %ae %s' 2020-01-30 ivoc@webrtc.org Remove function that takes command-line arguments directly Created with: gclient setdep -r src/third_party/webrtc@d69935c114aa If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@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/+/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: Ief1fab6ba758cd46ed08c95b798a79d337352d75 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030589Reviewed-by:
-
Hugo Benichi authored
chromeos::NetworkTypePattern requires a non-empty network "type", however ArcNetHostImpl does not care to check if the NetworkState is valid before checking for network of type "Tether". The NOTREACHED triggered in that case has been observed while debugging b:148431888. BUG=145960788 TEST=Compiled Change-Id: I79140461d0d628260fbf873274e73906fcfea646 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030387Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Commit-Queue: Hugo Benichi <hugobenichi@google.com> Cr-Commit-Position: refs/heads/master@{#736861}
-
Peter Beverloo authored
Change-Id: Id0b2b359beebb3f97750be12853bec83042c956a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030489Reviewed-by:
Peter Conn <peconn@chromium.org> Commit-Queue: Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#736860}
-
Gyuyoung Kim authored
This CL migrates the legacy OverridePreference IPC message to the new Mojo in WebTestClient interface. To make WebTestClient interface include |WebPreference| native type, this CL needs to extract [Native] struct WebPreference from native_types.mojom to web_preference.mojom. Then, WebTestClient interface refers to |WebPreference| type through web_preferences_mojom target. This CL should not change any behavior. Bug: 1039247 Change-Id: I707c071169015939b84377ecca8da26cc1aa4395 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2011591Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/38ebf200171d..3941f82d7c3f git log 38ebf200171d..3941f82d7c3f --date=short --first-parent --format='%ad %ae %s' 2020-01-30 szuend@chromium.org Prepare test runners for source map asyncification Created with: gclient setdep -r src/third_party/devtools-frontend/src@3941f82d7c3f 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/+/master/autoroll/README.md Bug: chromium:1032016 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: If5aec86db3d373028169417cb288506209e68914 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030588Reviewed-by:
-
Erik Staab authored
Change-Id: I2438dd60e6991bddd94aa0c77b089547d8758cf4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2029217Reviewed-by:
Peter Wen <wnwen@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#736857}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/8f44c3504ca6..0ec7b8485455 git log 8f44c3504ca6..0ec7b8485455 --date=short --first-parent --format='%ad %ae %s' 2020-01-30 borenet@google.com [infra] Update go.sum Created with: gclient setdep -r src/third_party/skia@0ec7b8485455 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC reed@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/+/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: reed@google.com Change-Id: I8bacc46c2f999451cf1227f4b8cb4b675b1ec18c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030587Reviewed-by:
-
Hugo Benichi authored
BUG=b:145960788 TEST=Compiled Change-Id: I43b58efe4a32b431f8b88dd792ab10af7dec137a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030386Reviewed-by:
-
Yoav Weiss authored
Due to performance regressions related to the field trial, it seems like the best course of action is to (temporarily) remove it, so that it won't impact stable. This CL removes the field trial configuration. Bug: 1036347 Change-Id: I12fbd9e16137d8f5e5803d8502ce80e0b907cae1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2023497Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Commit-Queue: Yoav Weiss <yoavweiss@chromium.org> Cr-Commit-Position: refs/heads/master@{#736854}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 64dcd1dc. 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 NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: I2f22c6a0a0a6b195b764e5ddedf2b7cc4c547790 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028476Reviewed-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@{#736853}
-
Rushan Suleymanov authored
Extend ModelTypeProcessor interface to pass full commit failures. In the next patch commit failure will be passed to sharing message bridge. Bug: 1047123 Change-Id: Ieb59afe7a56e0cc23b4a0a09ff8ca7053ecbf85c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2030405 Commit-Queue: Rushan Suleymanov <rushans@google.com> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#736852}
-
arthursonzogni authored
This correspond to step 1.c from the "Unify CSP" document: https://docs.google.com/document/d/1v5mJnXJ5dSVXE_rgvJnNM9bzH0ni0YzdhPQ7GLqyhao This merges: - content::CSPDirective (removed) - network::mojom::CSPDirective (kept) Only the mojo one remains: network::mojom::CSPDirective; Future follow-up will apply do the same for ContentSecurityPolicy. The goal at the end is to have an unique mojo data structure for representing Content-Security-Policy. TBR=clamy@chromium.org Bug: 1021462 Change-Id: Ic04f491992fadb6a9a97ba6ab4327b2d7b8ad1cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2027339Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
-
Yuki Shiino authored
It turned out that gclient_paths.py seems not working as expected on canary build (could be a conflict with Google-internal src). Implements the similar (but simplified) logic without depending on unknown factors. Bug: 839389 Change-Id: I70032d60098c018f139bdceb78ea73996393ddfc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2029405Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#736850}
-
Mia Glaese authored
Bug: 1041475 Change-Id: I7fcf625aa0d037ebf655fa09170d0681cef8bb98 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2022925Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Patrick Noland <pnoland@chromium.org> Commit-Queue: Mia Glaese <glamia@chromium.org> Cr-Commit-Position: refs/heads/master@{#736849}
-
Mikel Astiz authored
Local sync metadata belongs to one user account. Due to the distributed nature of the locally persisted sync metadata, the cache GUID is used as "epoch" to detect mismatched in edge cases like the browser crashing during shutdown and before I/O gets flushed. However, prior to this patch, the cache GUID itself has no safety mechanism to detect it maps to the intended account ID. In this patch, a new SyncPref is introduced to achieve that, in away that both prefs (cache GUID and account ID) are stored atomically. Because the pref is newly-introduced, migration logic is introduced to populate it for the first time if initially empty. Change-Id: I2cdd9f997077c4acd16e9283df8c025f51d40546 Bug: 1046237,1021527 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2023528Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#736848}
-
