- 21 May, 2020 40 commits
-
-
Mike Wittman authored
This reverts commit 01b9624c. Reason for revert: first of two reverts to fix https://ci.chromium.org/p/chromium/builders/try/android_compile_x64_dbg/1504 Original change's description: > [Sampling profiler] Provide unwinder in stack_unwinder DFM > > Implements the Android native/Java stack unwinder within the stack_unwinder > dynamic feature module. > > Adds visibility/assert_no_deps constraints to //base/BUILD.gn to avoid > leaking module implementation types into general Chrome code, which > would defeat the purpose of using the module to avoid Chrome binary size > increase. > > Adds required stub declarations/implementation for MemoryRegionsMap. > > Bug: 1083530, 1004855 > Change-Id: If9d31c4e1b9d214bb0baabe3afa8475ba7adef4a > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209297 > Commit-Queue: kylechar <kylechar@chromium.org> > Reviewed-by: kylechar <kylechar@chromium.org> > Reviewed-by: Etienne Pierre-Doray <etiennep@chromium.org> > Auto-Submit: Mike Wittman <wittman@chromium.org> > Cr-Commit-Position: refs/heads/master@{#771018} TBR=wittman@chromium.org,kylechar@chromium.org,etiennep@chromium.org Change-Id: I992dcf2c36b13ef51d323a44fef840e1e04bdd7b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1083530, 1004855 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211927Reviewed-by:
Mike Wittman <wittman@chromium.org> Commit-Queue: Mike Wittman <wittman@chromium.org> Cr-Commit-Position: refs/heads/master@{#771046}
-
Miguel Casas-Sanchez authored
crrev.com/c/2195334 just landed PowerObserver::OnThermalStateChange, this CL listens to it from MediaStreamManager and logs it in the WebRTC log. The utility function PowerMonitorSource::DeviceThermalStateToString gets #if DCHECK_IS_ON() removed to be used from the new code, which runs also in non-dcheck_always_on builds. Bug: 1071431 Change-Id: I5641ae5a96e4ca9504498821ffb3ea91adbec18a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207410Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#771045}
-
Nick Diego Yamane authored
Until now, both code that deals with the Wayland DND protocol and logic for handling regular drag and drop operations has been living in the same component, WaylandDataDevice (which also has code for handling Clipboard operations). As a means of supporting alternative DND use cases, mainly Window/Tab dragging, this patch introduces DragDelegate interfaces into DataDevice and DataSource classes, and shifts most of the use-case specific responsibility to them. WaylandDataDragController implements these brand new interfaces for the usual "data" drag and drop operations, basically containing most of the code previously in WaylandDataDevice. A second implementation for them will be introduced in follow up CLs, aimed at supporting tab dragging. Bug: 896640 Change-Id: I07ff12722230c1f5c065de857e77b086c75c18e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2205881Reviewed-by:
Maksim Sisov <msisov@igalia.com> Commit-Queue: Nick Yamane <nickdiego@igalia.com> Cr-Commit-Position: refs/heads/master@{#771044}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4c5b1b3a9278..4d042c5649f6 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 vasilii@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: vasilii@google.com Change-Id: I44e7c5d6cb46b6bc0de040967501c46d6bb3d3cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212344Reviewed-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@{#771043}
-
Jeremy Roman authored
This is already available as content::WebContents::GetNativeView. Fixed: 1085304 Change-Id: I15152e073db6750e8b82ca9d94467fff68a9ff1c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208815Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Commit-Queue: Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#771042}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/f045c636f9b4..50cdbe3343ad 2020-05-21 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: I7a5df33715ba7fc2ed354b015b69bb43b6cd0802 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212347Reviewed-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@{#771041}
-
Lukasz Anforowicz authored
Consider the example fixed by this CL in tests/basics-original.cc: struct MyStruct { SomeClass *ptr_field1, *ptr_field2; }; Before this CL, the rewriter would generate a separate replacement for |ptr_field1| and another one for |ptr_field2|. Such replacements would cover overlapping ranges of the source file, leading to butchered, incorrect edits. Note that in the example above |git cl format| places the "*" character above next to the field names (e.g. |*ptr_field1|) rather than next to the type name (e.g. |SomeClass*|). For a single field |git cl format| places the "*" character next to the type name, but the CL adds a single-field test to also cover the non-standard placement. After this CL: *) apply_edits.py detects overlapping replacements and reports them as an error. Simple unit tests have been added to apply_edits_test.py *) RewriteRawPtrFields.cpp defines and uses a custom |hasUniqueTypeLoc| matcher that can be used to ignore fields with a non-unique type location *) RewriteRawPtrFields.cpp tweaks slightly the |replacement_range| and |replacement_text| so that non-standard placement of the "*" character is handled correctly. Bug: 1069567 Change-Id: I5e0a90de6d8c7dc5d16ef68012509508c84b6ccf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2157992 Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:Bartek Nowierski <bartekn@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#771040}
-
Gavin Williams authored
This should have been created in camel case because this is a JSON variable. Bug: 1059607 Change-Id: Id434236900ce5e234dcc2cc3a6381035c995c500 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2203136 Commit-Queue: Gavin Williams <gavinwill@chromium.org> Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Cr-Commit-Position: refs/heads/master@{#771039}
-
Ramin Halavati authored
OffTheRecord profile use in LoadingPredictorConfig unittest is updated and a test is added for non-primary OffTheRecord profiles. Bug: 1060940 Change-Id: Ieba3efeb75e7af40b356d9e656b8d8246b4d7a80 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212161 Auto-Submit: Ramin Halavati <rhalavati@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#771038}
-
Cheng Zhao authored
In this code: base::PostTaskAndReplyWithResult( task_runner_.get(), FROM_HERE, base::BindOnce(&InitializeDictionaryLocation, language_), base::BindOnce(&InitializeDictionaryLocationComplete, weak_ptr_factory_.GetWeakPtr())); When |SpellcheckHunspellDictionary| is destroyed before |InitializeDictionaryLocation| is finished, the |InitializeDictionaryLocationComplete| method would not be called, and the returned |DictionaryFile| will be destroyed on UI thread, with its |base::File| member. It would then result in a DCHECK like this: [904:0518/140618.257:FATAL:thread_restrictions.cc(78)] Check failed: !g_blocking_disallowed.Get().Get(). Function marked as blocking was called from a scope that disallows blocking! If this task is running inside the ThreadPool, it needs to have MayBlock() in its TaskTraits. Otherwise, consider making this blocking work asynchronous or, as a last resort, you may use ScopedAllowBlocking (see its documentation for best practices). g_blocking_disallowed currently set to true by base::debug::CollectStackTrace [0x00007FF6805435C2+18] (o:\base\debug\stack_trace_win.cc:284) base::debug::StackTrace::StackTrace [0x00007FF68049BCC2+18] (o:\base\debug\stack_trace.cc:203) logging::LogMessage::~LogMessage [0x00007FF6804AF447+215] (o:\base\logging.cc:605) logging::LogMessage::~LogMessage [0x00007FF6804B0190+16] (o:\base\logging.cc:598) base::internal::AssertBlockingAllowed [0x00007FF680514B33+163] (o:\base\threading\thread_restrictions.cc:85) base::ScopedBlockingCall::ScopedBlockingCall [0x00007FF6805103EB+155] (o:\base\threading\scoped_blocking_call.cc:40) base::File::Close [0x00007FF680549E4B+139] (o:\base\files\file_win.cc:42) SpellcheckHunspellDictionary::DictionaryFile::~DictionaryFile [0x00007FF680071DDE+194] (o:\chrome\browser\spellchecker\spellcheck_hunspell_dictionary.cc:102) base::OnceCallback<void (SpellcheckHunspellDictionary::DictionaryFile)>::Run [0x00007FF6800749BF+75] (o:\base\callback.h:100) To avoid have |base::File| destroyed on UI thread, we should make sure the file is closed in the destructor of |DictionaryFile|. Bug: 1085227 Change-Id: I1dba3b36661ec7968b22cf6d7835e41a81a313cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2206199Reviewed-by:Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Guillaume Jenkins <gujen@google.com> Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#771037}
-
Thomas Lukaszewicz authored
Removed the use of ViewHierarchyChanged() to update native view properties when added to a view hierarchy. Changed to use AddedToWidget() instead. Currently the logic in ViewHierarchyChanged() requires the view be in a Widget hierarchy. This should not be assumed in ViewHierarchyChanged(). Instead, having the logic in AddedToWidget() helps to make the assumptions more clear. Bug: 1083822 Change-Id: Ia393fd0141a863212e5c9e30e364fbc4d2b2d7d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2206215Reviewed-by:
Robert Flack <flackr@chromium.org> Commit-Queue: Thomas Lukaszewicz <tluk@chromium.org> Cr-Commit-Position: refs/heads/master@{#771036}
-
Lukasz Anforowicz authored
Bug: 1080832 Change-Id: I95435ef51b4231a2f08368af722a6bff6b8c4c6e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208542Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/fe49ce29c3e1..b37105ea6cca 2020-05-21 egdaniel@google.com Remove support for manual memory management in GrVkMemory. 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 lovisolo@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-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: lovisolo@google.com Change-Id: I0c07af9a8a62b4eb48bd1faa07a2c8725a23addf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212348Reviewed-by:
-
Andrew Grieve authored
Changes it into an analysis step rather that runs at the same time as errorprone. Removes ability to output .jar files from bytecode_processor. It now outputs a stamp file. TBR=agrieve # Renamed build variable. Bug: 1080670 Change-Id: If98eb4e3673ccdfa47c151370b3f98aa23488a15 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2210873 Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#771033}
-
Adrienne Walker authored
I'm in the process of documenting what we need to move cache storage to storage service. Some of these dependencies are easier to clean up than to document, hence this patch. Most of these includes seem vestigal. I tried to be careful to not remove things that were IWYU and transitively included elsewhere. Change-Id: Ib9e599a4336f65949d1aeaaf15a9a827e36be218 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211229 Auto-Submit: enne <enne@chromium.org> Commit-Queue: Ben Kelly <wanderview@chromium.org> Reviewed-by:
Ben Kelly <wanderview@chromium.org> Cr-Commit-Position: refs/heads/master@{#771032}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/66bd749aed4d..c4e15df5d5da 2020-05-21 ilkos@google.com Merge "Revisit fixing RSS captured after heap dump was taken" 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: Ied361e72daf7b405ead5450445fa913ac14841a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212346Reviewed-by:
-
Sam Maier authored
* Fixes optimization bug (see linked bug) * Introduces enum simplification * Adds command-line support for L8 optimizations Bug: 1077307 Change-Id: I85e4bba61253c9056d4544cbd849ec0d9bcd97a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2210792Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: Sam Maier <smaier@chromium.org> Cr-Commit-Position: refs/heads/master@{#771030}
-
Rouslan Solomakhin authored
Before this patch, the desktop logic for checking whether the payment app is waiting for a price update from the merchant was defined in the desktop controller, which prevented from sharing this logic across platforms and in different types of payment apps. This patch moves PaymentHandlerHost.is_waiting() check from payment_request.cc to service_worker_payment_app.cc. After this patch, the logic for checking whether the payment app is waiting for a price update is defined in the potentially cross-platform payment app interface. Design: https://bit.ly/cross-platform-pay-app-factory Bug: 1083242 Change-Id: Idde2f1c9396641835f154be19ac7d2901c1c1d14 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207973 Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Liquan (Max) Gu <maxlg@chromium.org> Cr-Commit-Position: refs/heads/master@{#771029}
-
Chromium WPT Sync authored
Using wpt-import in Chromium d9f75a3a. With Chromium commits locally applied on WPT: f6984fd9 "De-flaky pointerlock/pointerevent_pointermove_in_pointerlock.html" 45436d4d "CookieStore: WPT to verify subscribe/unsubscribe is idempotent" 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: kojii@chromium.org: external/wpt/css/css-scoping NOAUTOREVERT=true TBR=raphael.kubo.da.costa No-Export: true Change-Id: Ia0550b3a6c37f9a4783c2ca9b1f26cb5188b08c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212321Reviewed-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@{#771028}
-
Tim van der Lippe authored
These tests were disabled during the ES modules migration. Since that migration has now concluded, we can reenable them. R=aerotwist@chromium.org AUTOSUBMIT=1 Bug: 1006759 Change-Id: I28eabe38d8da7859cd8cad739961ba3e198dd9ff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212045 Commit-Queue: Paul Lewis <aerotwist@chromium.org> Reviewed-by:
Paul Lewis <aerotwist@chromium.org> Auto-Submit: Tim van der Lippe <tvanderlippe@chromium.org> Cr-Commit-Position: refs/heads/master@{#771027}
-
Mugdha Lakhani authored
Link to Site settings UI from Page Info. Bug: 1049683, 1052375 Change-Id: Iab1e5825fc4b6a8413163c1aecc41ab46003fc5e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2193691 Commit-Queue: Mugdha Lakhani <nator@chromium.org> Reviewed-by:
Robbie McElrath <rmcelrath@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#771026}
-
Theresa Wellington authored
Move //chrome/browser/profiles/android/DEPS to //chrome/browser/profiles/android/java/DEPS, add missing entry, and remove C++ exclusions. BUG=1084827 Change-Id: Icb291315651af6cb41b076dd2b4d63dfbc74cb36 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211231Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#771025}
-
Chris Hamilton authored
This exposes an additional parameter to the timeout calculation logic, and adds a bunch of tests. BUG=1082920 Change-Id: I6058210207cbb695fe33ac8cfa3c1d4ef26d1c5a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2210662Reviewed-by:
Sigurður Ásgeirsson <siggi@chromium.org> Commit-Queue: Chris Hamilton <chrisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#771024}
-
Ramin Halavati authored
ReportScheduler and ExtensionRequestObserverFactory use IsIncognitoProfile() to prevent OffTheRecord profiles in two contexts. IsIncognitoProfile() currently covers all non-guest off-the-record profiles, but this will be soon changed to only incognito profiles. So to keep the functionality as it is, IsIncognitoProfile() is replaced with IsOffTheRecord(). This CL does not create any behavioral change now, and is to prevent change when IsIncognitoProfile() is updated. Bug: 1074201 Change-Id: I71dba6f52195df7b12a2ae7df984ce5dc374ed3c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211343 Commit-Queue: Owen Min <zmin@chromium.org> Auto-Submit: Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#771023}
-
Koji Ishii authored
Code added to generate |NGFragmentItem| for culled inline[1] were mostly removed when we supported culled inline[2], but some are still left unused. This patch removes them. [1]: https://chromium-review.googlesource.com/c/chromium/src/+/1774425 [2]: https://chromium-review.googlesource.com/c/chromium/src/+/2160671 Bug: 982194 Change-Id: I7fab33c278e2048a1a449dee3ff2f49aead997ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2210296Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#771022}
-
Ramin Halavati authored
ProfileDestroyer is updated for better readability. Bug: 1033903 Change-Id: Id2d114ca28288aced86b3d8aa61fc36fc48f11a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2134019 Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#771021}
-
Liviu Tinta authored
A bunch of tests synthesize input events during testing, and this ranges from unit-tests, interactive-ui-tests, telemetry tests etc. During such tests, it is possible that chrome also receives unexpected input events from the native system (e.g. if the window happens to show up under the mouse cursor), causing unexpected failures (e.g. the event-stream becomes invalid, or the tooltip shows up, etc.). Ignoring all input events from the system during these tests should make these tests more deterministic. Bug: 766450, 883710 Change-Id: Ib7fccaf7a0495207cc9b097bfe25862cad302dc4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2176697Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Commit-Queue: Liviu Tinta <liviutinta@chromium.org> Cr-Commit-Position: refs/heads/master@{#771020}
-
Yi Gu authored
This test has been flaky under virtual/threaded test suite. https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=blink_web_tests%20(with%20patch)&tests=external%2Fwpt%2Fscroll-animations%2Felement-based-offset.html Bug: 1080609 Change-Id: I2fce270915b921028411e144c0fef7326ad14e99 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212361Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#771019}
-
Mike Wittman authored
Implements the Android native/Java stack unwinder within the stack_unwinder dynamic feature module. Adds visibility/assert_no_deps constraints to //base/BUILD.gn to avoid leaking module implementation types into general Chrome code, which would defeat the purpose of using the module to avoid Chrome binary size increase. Adds required stub declarations/implementation for MemoryRegionsMap. Bug: 1083530, 1004855 Change-Id: If9d31c4e1b9d214bb0baabe3afa8475ba7adef4a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209297 Commit-Queue: kylechar <kylechar@chromium.org> Reviewed-by:
kylechar <kylechar@chromium.org> Reviewed-by:
Etienne Pierre-Doray <etiennep@chromium.org> Auto-Submit: Mike Wittman <wittman@chromium.org> Cr-Commit-Position: refs/heads/master@{#771018}
-
François Degros authored
BUG=None TEST=out/release/browser_tests --gtest_filter="*FilesApp*" --gtest_list_tests TEST=testing/xvfb.py out/release/browser_tests --gtest_filter="*FilesApp*" --test-launcher-jobs=10 Change-Id: Id60e003aa889f6f1886f2c1332e50fb5f6080b65 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2210041Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: François Degros <fdegros@chromium.org> Cr-Commit-Position: refs/heads/master@{#771017}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/b4efb6ecad4e..f045c636f9b4 2020-05-21 tvanderlippe@chromium.org Use terser rollup plugin 2020-05-21 yangguo@chromium.org Revert "Only enable custom formatters for the current session." 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: chromium:1016755 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Icdfa4cd4da001dfc13795f3af1a42a7bfa4a269c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212342Reviewed-by:
-
Gayane Petrosyan authored
Bug: 1084787 Change-Id: Iba082a91843becfb0080fc22029fa8a518e3d73e Tbr: tiborg@chromium.org for lint suppression Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211065 Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Reviewed-by:
Tanya Gupta <tgupta@chromium.org> Cr-Commit-Position: refs/heads/master@{#771015}
-
Internal Frameworks Autoroller authored
TBR=bling-team@google.com Change-Id: I2fa3f7ae8e9254c3df63a5dca80314ee27004d29 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211902Reviewed-by:
Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#771014}
-
Jiawei Shao authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Cross.git/+log/d638d2df9c8c..287e93ff80ee $ git log d638d2df9..287e93ff8 --date=short --no-merges --format='%ad %ae %s' 2020-05-20 dsinclair Roll dependencies 2020-05-20 post GLSL: Add more test shaders for hit attribute types. 2020-05-20 post GLSL: Support ray payloads and hit attributes declared as Block. 2020-05-20 post GLSL: Add some more focused RT test shaders. 2020-05-19 post HLSL: Implement image queries for UAV images. Created with: roll-dep src/third_party/spirv-cross/spirv-cross TBR=cwallez@chromium.org Bug: dawn:267 Change-Id: Id9d3c0124a78ab6eff9f6c02ea55fabe5ef31af3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211923Reviewed-by:
Jiawei Shao <jiawei.shao@intel.com> Reviewed-by:
Corentin Wallez <cwallez@chromium.org> Commit-Queue: Jiawei Shao <jiawei.shao@intel.com> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Cr-Commit-Position: refs/heads/master@{#771013}
-
David Black authored
Bug: b:129411551 Change-Id: If61cb8184b9a2de77d8051a03d6ad5f593577c50 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209280 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#771012}
-
Adithya Srinivasan authored
This CL makes sure the indicator (which is a notification on android) is shown if a portal is capturing media. Currently the only way this is possible is if a page that is capturing media is adopted into a portal, but this will change in the future. Bug: 1080740 Change-Id: Iec2d5e41368e2233e86a4aba308f104dbd7ec520 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2170988Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Reviewed-by:
Lucas Gadani <lfg@chromium.org> Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Cr-Commit-Position: refs/heads/master@{#771011}
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1590047700-f78bd2d622c61ab01f15ac85a9b78063006149d8.profdata to chrome-mac-master-1590061793-cada8ca96f7665a04948a973b6cd8e2e12d92609.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 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: jeffyoon@google.com,liaoyuke@google.com Change-Id: Ic0a563bec6d828ad8f07ee7f390752e65f96626c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2212345Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/e8b8b8164520..ce285f432c39 2020-05-21 angle-autoroll@skia-public.iam.gserviceaccount.com Roll SPIRV-Tools from 63fa9114a931 to 3c47dac28208 (17 revisions) 2020-05-21 angle-autoroll@skia-public.iam.gserviceaccount.com Roll SwiftShader from b6e8c3f0f483 to cc5cda0f997d (1 revision) 2020-05-21 angle-autoroll@skia-public.iam.gserviceaccount.com Roll Vulkan-ValidationLayers from 4683991f7675 to a6d833327f7e (6 revisions) 2020-05-21 angle-autoroll@skia-public.iam.gserviceaccount.com Roll Vulkan-Tools from ca8958de1238 to 0c4ea014bf28 (2 revisions) 2020-05-21 angle-autoroll@skia-public.iam.gserviceaccount.com Roll glslang from 0ab78114a9e1 to 6f723ebbe3b0 (5 revisions) 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 geofflang@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: geofflang@google.com Change-Id: If277ae672b4143682307087ef8f6776c163b8941 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211897Reviewed-by:
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/529df8d63450..66bd749aed4d 2020-05-21 hjd@google.com trace_processor: Add missing header 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: I67e6bd9c60017c98f4303a57f99f58c9f4ac1022 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2211898Reviewed-by:
-
Gabriel Charette authored
This reverts commit e1334196. Reason for revert: it was the wrong fix. See crbug.com/1057995#c11 for details. (keeps comment tweak on the EXPECT_CHECK_DEATH) Original change's description: > Support DCHECK-death-tests in official builds too > > This is required by win-asan which has the following GN config: > dcheck_is_configurable = true > is_chrome_branded = true > is_debug = false > is_official_build = true > target_cpu = "x86" > > R=thakis@chromium.org > > Bug: 1057995 > Change-Id: I337e214f2194d3533ea329c22fd240dddae50e5a > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090136 > Commit-Queue: Gabriel Charette <gab@chromium.org> > Auto-Submit: Gabriel Charette <gab@chromium.org> > Reviewed-by: Wez <wez@chromium.org> > Cr-Commit-Position: refs/heads/master@{#748849} R=wez@chromium.org Bug: 1057995 Change-Id: I549dba1babb9fdfc996b34775bd1d102ed26ba6b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2210489 Auto-Submit: Gabriel Charette <gab@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#771007}
-
