- 05 Nov, 2019 40 commits
-
-
Xianzhu Wang authored
Now we can use the names defined in web_tests/FlagSpecificConfig as the flag-specific test expectations files and baseline directories. Renamed: enable-blink-features=BackForwardCache -> back-forward-cache enable-blilk-features=CompositeAfterPaint -> composite-after-paint disable-blink-features=LayoutNG -> disable-layout-ng enable-blink-features=LayoutNGFragmentItem -> layout-ng-fragment-item enable-blink-features=NewSystemColors -> new-system-colors Removed: disable-blink-features=MediaCastOverlayButton enable-blink-features=HeapIncrementalMarkingStress disable-features=NetworkService enable-features=NetworkService enable-features=OutOfBlinkCors disable-perfetto enable-blink-features=HeapUnifiedGarbageCollection enable-features=OverflowIconsForMediaControls Unchanged: disable-site-isolation-trials: short enough enable-gpu-rasterization, use-gl=*, use-vulkan=*, enable-features-UseSkiaRenderer: need more follow-up for gpu rasterization configurations for different renderers. Bug: 1019501 Change-Id: I73aa8996a61d67c5852fd74cec7289c2491c4250 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895928Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Tommy Steimel <steimel@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#712461}
-
Yuki Shiino authored
PathManager provides paths to Blink headers and output files. Also supports cross-components output. Bug: 839389 Change-Id: If9514f421ac8dd2d54c68cd9048a5ad5d1eb0536 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897577 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#712460}
-
Yuki Shiino authored
Make LooseFormatter support implicit ({}) and explicit ({0}, {1}, ...) positional argument specifiers. Bug: 839389 Change-Id: I8a820cba90a2be580e63c15d35c194c4ba31c702 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897575 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by: -
Xianzhu Wang authored
It didn't work by itself. It was just a redundant information of whether the scrollbar theme supports solid color scrollbars. Change-Id: I32c61c8337bb8546a05841723833905463e71457 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1896675Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#712458}
-
Julie Jeongeun Kim authored
This CL cleans up old mojo types for mojom::VideoEncodeAcceleratorClient using SelfOwnedReceiver, PendingReceiver, Receiver, PendingRemote, and Remote. Bug: 955171 Change-Id: Id743778fd2bd4edd72ebff10f33510ad74e6ed42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1896494 Commit-Queue: Julie Kim <jkim@igalia.com> Reviewed-by:
Sam McNally <sammc@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#712457}
-
Peter Kotwicz authored
This CL: - Makes WebappActivity use CustomTabToolbarCoordinator - Splits out of CustomTabToolbarCoordinator: - CustomTabToolbarColorController for setting the toolbar color (this behaviour will be merged separately) - CustomTabUmaRecorder for recording UMA metrics BUG=1016650 Change-Id: I952fc265cd1f65366fa0e105fde5444b6a685f7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1887738 Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:Peter Conn <peconn@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#712456}
-
Benjamin Beaudry authored
This is a reland of cc4f1e5f This change was reverted due to "Windows Deterministic" bot failing. Apparently, these bots seems to have a problem with any changes to ppapi/. Currently, the only way we know is to try again. This is the exact same CL as the one that was reverted. Original change's description: > Pdf a11y: Populate AXNode with new text style info > > This change fills the font-family, font-size, font-weight, italic, bold > and color attributes of the AXNode with exposed values from the PDFium > APIs. > > I've also added a PDF accessibility tree dump test using the filter > properties to dump those attributes in the expected file. The test file > is "text-style.pdf". > > Somehow, I added in a previous change text-format-expected-blink.txt. > This file should have never been committed. I removed it in this CL. > > Bug: 985604 > Change-Id: I70a4729269aea69e184e61710d7977425b431704 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1755412 > Reviewed-by: Lei Zhang <thestig@chromium.org> > Reviewed-by: Tom Sepez <tsepez@chromium.org> > Reviewed-by: Bill Budge <bbudge@chromium.org> > Reviewed-by: Ian Prest <iapres@microsoft.com> > Commit-Queue: Lei Zhang <thestig@chromium.org> > Cr-Commit-Position: refs/heads/master@{#711942} TBR=raymes@chromium.org,bbudge@chromium.org,iapres@microsoft.com,tsepez@chromium.org Bug: 985604 Change-Id: I4030926c8aff39c05e45b8cdc3a3262d94158c8a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898162 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#712455}
-
rbpotter authored
Autogenerating v3 versions of: - search_highlight_utils - list_property_update_behavior - search_highlight_style_css These will be used by Web UIs as they are migrated to Polymer 3, initially by Print Preview. Bug: 1012574 Change-Id: I26c69411aa284241bc6d7a7d7bceb077e3c0414f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897318Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#712454}
-
Julie Jeongeun Kim authored
This CL cleans up old mojo types for mojom::DisplayClient using Receiver and PendingRemote. Bug: 955171 Change-Id: I18ebf4160c0c350b76689693ee7176d1e7185b14 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895029Reviewed-by:
Ken Rockot <rockot@google.com> Reviewed-by:
Eric Karl <ericrk@chromium.org> Reviewed-by:
enne <enne@chromium.org> Commit-Queue: Julie Kim <jkim@igalia.com> Cr-Commit-Position: refs/heads/master@{#712453}
-
Leon Han authored
This CL adds support for reading sub records of a NDEFRecord, i.e. NDEFRecord#toRecords(). Note: No support for writing yet. The spec changes: https://github.com/w3c/web-nfc/pull/333 https://github.com/w3c/web-nfc/pull/334 https://github.com/w3c/web-nfc/pull/359 BUG=520391 Change-Id: Ic7ac74416afa8e43f5293e4ca85e538fc700f3c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1824395 Commit-Queue: Leon Han <leon.han@intel.com> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#712452}
-
Luciano Pacheco authored
Enable gzip compression for resources on the following apps: Files, Gallery, Video and Audio Player. For reducing the disk usage on the ChromeOS root partition. Change ExecuteCodeFunction::LoadFile() to use ResourceBundle::LoadDataResourceString() to load resources that may be compressed or not. Test: ui_base_unittests --gtest_filter='ResourceBundleImageTest.*' Bug: 715912, 1003433 Change-Id: I307d16e075c157784a3daad9f2a4c7faf4736cc2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1873827 Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: I5ef88e4cdcbaecccd729598adffc871ec64d8979 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898869Reviewed-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@{#712450}
-
Sara Kato authored
Arc application zoom will be using finch feature (go/finch). It's first necessary to add this to chrome://flags Test: AboutFlagsHistogramTest.CheckHistograms passes Test: AboutFlagsHistogramTest.EveryFlagHasMetadata passes Bug: b/143584033 Change-Id: I8d846981932bdde0214685f8279729fe51f98563 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888869Reviewed-by:
Stefan Kuhne <skuhne@chromium.org> Reviewed-by:
Sara Kato <sarakato@chromium.org> Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Commit-Queue: Sara Kato <sarakato@chromium.org> Auto-Submit: Sara Kato <sarakato@chromium.org> Cr-Commit-Position: refs/heads/master@{#712449}
-
Julie Jeongeun Kim authored
This CL cleans up old Mojo types for mojom::Remoter, mojom::RemotingSource, and mojom::RemoterFactory using Remote, Receiver, PendingReceiver. Bug: 955171 Change-Id: I3b06bf38ceae5e357d7c6de615616302dad7bfce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892990 Commit-Queue: Julie Kim <jkim@igalia.com> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Reviewed-by:
-
Koji Ishii authored
This patch adds a test to check the atomic inline+NBSP quirk[1] should not apply if `nowrap`. This issue was originally introduced in r694788 crrev.com/c/1792465, but was accidentally fixed in r700578 crrev.com/c/1827097. The fix did not have a test because it wasn't intended to fix this issue. [1] https://www.w3.org/TR/css-text-3/#line-break-details TBR=chrishtr@chromium.org, ikilpatrick@chromium.org Bug: 1017465, 1018309 Change-Id: Id505b894c02d6521766e2cf354e7c9c8c4a114ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888230Reviewed-by:
-
Donn Denman authored
Automatically adjusts the length of the Context and the Caption text views as the panel expands open. This fixes some issues with Definitions which can have long captions and uses the Context view to show the pronunciation. The Caption was supposed to auto-adjust, but was not calling super when expanding. The Context just needed to pass the adjustment parameters. BUG=1016973 Change-Id: Ie331cde2cc08a059197ea1e7178099c865e4dbc9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898722 Auto-Submit: Donn Denman <donnd@chromium.org> Reviewed-by:
Jinsuk Kim <jinsukkim@chromium.org> Commit-Queue: Donn Denman <donnd@chromium.org> Cr-Commit-Position: refs/heads/master@{#712446}
-
Chromium WPT Sync authored
Using wpt-import in Chromium d8488bcd. With Chromium commits locally applied on WPT: 1d5b803f "Add WebVTT support for inline styling - Web Platform Tests" 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: guidou@chromium.org, armax@chromium.org: external/wpt/mediacapture-streams mkwst@chromium.org, andypaicu@chromium.org: external/wpt/content-security-policy NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: I9916e575b6c06c705390b0a3a7cd56898a30c3cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898177Reviewed-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@{#712445}
-
Ian Kilpatrick authored
Previously we were only using NGBoxFragmentPainter for blocks with inline children (those specifically with a PaintFragment()). As the NGBoxFragmentPainter removed its dependency on NGPaintFragment we can now use this for block-level fragments as well. Bug: 988015 Change-Id: Id839cb368454bf4c087676dbda2a3ead04e26bd3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1869076Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#712444}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/8a0420a05d13..3f35966470fb Created with: gclient setdep -r src-internal@3f35966470fb 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 jbudorick@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.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: Ib5ce9883e711239cf9f277cdef22331c8977162b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1896958Reviewed-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@{#712443}
-
Julie Jeongeun Kim authored
This CL converts URLLoaderRequest in //android_webview, //chrome, //chromecast, //components, //extensions, //srvices, and //storage to new Mojo types using PendingReceiver and Receiver. Bug: 955171 Change-Id: I089a195aa52fd74111c8ac819ee96c6cdbb87b9d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886283 Commit-Queue: Julie Kim <jkim@igalia.com> Reviewed-by:
Maksim Orlovich <morlovich@chromium.org> Reviewed-by:
Luke Halliwell (slow) <halliwell@chromium.org> Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Reviewed-by:
Tim Volodine <timvolodine@chromium.org> Cr-Commit-Position: refs/heads/master@{#712442}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/71b0f0b588d6..c898ec1a5e2f git log 71b0f0b588d6..c898ec1a5e2f --date=short --no-merges --format='%ad %ae %s' 2019-11-05 jmadill@chromium.org Add EGL GGP extensions. Created with: gclient setdep -r src/third_party/angle@c898ec1a5e2f 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 jonahr@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_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=jonahr@google.com Bug: None Change-Id: I244baf4fe4a4c531511f21964b6acb5e9a71cc82 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898815Reviewed-by:
-
Ahmed Fakhry authored
My previous CL crrev.com/c/1842624 regressed determining if a browser window belongs to the current active Virtual Desk on Windows. This CL fixes this regression. BUG=1021337 Change-Id: Ia134df934530e590219de1a02747c350bb01cf81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898768Reviewed-by:
-
Dominick Ng authored
This CL flushes the App Service's mojo pipes immediately following login to deflake this test. This ensures that apps which are installed at login correctly percolate through the async callbacks. BUG=1006522 Change-Id: I76e1b24d530f206a55584e513f461f5cd1543c7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1817876 Commit-Queue: Dominick Ng <dominickn@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#712439}
-
Ian Kilpatrick authored
wangxianzhu@ reopened bugs which these tests referenced. They were either marked as flakey, and/or just needed to be rebaselined. They were marked as flakey quite a few years ago, so lets try enabling them again. Bug: 453002, 605059, 663858 Change-Id: I6795d58efa741b83e8f3ced1ead603309371e5b7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898439 Auto-Submit: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
-
Hayato Ito authored
I'm no longer responsible for core/dom/events code base. It'd be better to remove OWNERS file there to reflect the current status. Change-Id: Icf7411a5418b5f8e05e2ee23b349ca98ffb260cd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898890Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Hayato Ito <hayato@chromium.org> Cr-Commit-Position: refs/heads/master@{#712437}
-
Vladimir Levin authored
This patch clean-up some unnecessary TODOs, and updates the code to remove an extra call to ComputeTrackSizes, which was introduced by the intrinsic-size patch earlier. R=rego@igalia.com, chrishtr@chromium.org Change-Id: Ie6c521ec0ca8f53b97fb84a0c62d7f4cfbdc0e5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897907Reviewed-by:
Manuel Rego <rego@igalia.com> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#712436}
-
Gabriel Charette authored
Instead use base::DeleteSoon which enforces usage of TaskTraits. This CL is a no-op as-is. It was recently discovered however that some callers did BrowserThread::DeleteSoon() with pending tasks running on different task queues (different traits -- e.g. TaskTraits to make this more obvious. Please review whether calls in this CL can be migrated as-is or need additional traits to match potentially pending tasks. Split from https://chromium-review.googlesource.com/c/chromium/src/+/1894109 for cursory review. This CL was uploaded by git cl split. R=thestig@chromium.org TaskPriority: :BEST_EFFORT) which can result in out-of-order deletion... BrowserThread: :DeleteSoon() is being migrated to base::DeleteSoon() w/ Bug: 1019767 Change-Id: Icabe0c2a24ccb44d27e6575b527baabe1824460e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894883 Auto-Submit: Gabriel Charette <gab@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/4a4f34b76f3a..f3d4109a793b git log 4a4f34b76f3a..f3d4109a793b --date=short --no-merges --format='%ad %ae %s' 2019-11-05 mtklein@google.com add (x,y) params to shader program() Created with: gclient setdep -r src/third_party/skia@f3d4109a793b 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 ethannicholas@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 TBR=ethannicholas@google.com Bug: None Change-Id: I98b7940bee73a02e8eedc5aa6f726dbbc500b2d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898866Reviewed-by:
-
Julie Jeongeun Kim authored
This CL cleans up old mojo types for mojom::SubresourceFilterAgent using AssociatedReceiver and PendingAssociatedReceiver. Bug: 955171 Change-Id: Iedd96df27c643c1bd307980d77c35231650631c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897396Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Reviewed-by:
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=abdulsyed@chromium.org Change-Id: Iaf90f1afcc14e22a8695b22f00b9b2e6b1c5d2a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897807Reviewed-by:
Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#712432}
-
Gabriel Charette authored
Instead use base::DeleteSoon which enforces usage of TaskTraits. This CL is a no-op as-is. It was recently discovered however that some callers did BrowserThread::DeleteSoon() with pending tasks running on different task queues (different traits -- e.g. TaskTraits to make this more obvious. Please review whether calls in this CL can be migrated as-is or need additional traits to match potentially pending tasks. Split from https://chromium-review.googlesource.com/c/chromium/src/+/1894109 for cursory review. This CL was uploaded by git cl split. R=wjmaclean@chromium.org TaskPriority: :BEST_EFFORT) which can result in out-of-order deletion... BrowserThread: :DeleteSoon() is being migrated to base::DeleteSoon() w/ Bug: 1019767 Change-Id: I08632c1a3a1b69fec27fa626b6d35b61da6bfe2c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895641Reviewed-by:
James MacLean <wjmaclean@chromium.org> Commit-Queue: Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#712431}
-
dpapad authored
Bug: None Change-Id: I7923e5168b2f0e8a1df4b2e9af7db2b10d332f3b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895077 Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#712430}
-
Antonio Gomes authored
The following files were moved to third_party/blink/renderer/platform/mediastream from third_party/blink/public/platform/modules/mediastream: - media_stream_audio_level_calculator.h - media_stream_audio_processor_options.h - media_stream_audio_source.h - webrtc_uma_histograms.h Whereas the following files were moved to third_party/blink/renderer/platform/mediastream from third_party/blink/renderer/platform/export/mediastream - media_stream_audio_level_calculator.cc - media_stream_audio_processor_options.cc - media_stream_audio_processor_options_test.cc - media_stream_audio_source.cc - webrtc_uma_histograms.cc - webrtc_uma_histograms_test.cc Note that there are still headers left over deliberately in the public directory: (included in content/renderer): - media_stream_audio_track.h - web_media_element_source_utils.h - web_media_stream_audio_sink.h - (included from other public headers): - media_stream_audio_deliverer.h - media_stream_types.h - secure_display_link_tracker.h - web_media_stream_audio_renderer.h - web_media_stream_sink.h - web_media_stream_video_renderer.h - web_platform_media_stream_source.h - web_platform_media_stream_track.h web_media_stream_audio_renderer.h web_media_stream_audio_sink.h BUG=919392, 787254, 704136 R=guidou@chromium.org, haraken@chromium.org Change-Id: I40c0268fed8dafb2aa40620a5e4781324d4342a8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1891072 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#712429}
-
Andrew Grieve authored
This was always the intention. Change-Id: I7f49676d6cb682999dc0a3dcb1878f831d6a5f4c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898947 Auto-Submit: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Eric Stevenson <estevenson@chromium.org> Commit-Queue: Eric Stevenson <estevenson@chromium.org> Cr-Commit-Position: refs/heads/master@{#712428}
-
Sergey Ulanov authored
Bug: 999736, b/141714552 Change-Id: I9abf0f01c2bbed6df322b6e822993960ff7bffdf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898134 Auto-Submit: Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
David Dorwin <ddorwin@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#712427}
-
Jinsuk Kim authored
The only callsite outside chrome.browser.tab package now caches its own TWCDA to avoid getting the reference through the Tab API. The API can then be a package private method. Moved TabTestUtils to target chrome_java_test_support. It now contains a helper method for tests to use to get TWCDA. Bug: 995903 Change-Id: I4614cc917710cd21f8be2e426dfcb29f0a4ba8da Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1881004Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Cr-Commit-Position: refs/heads/master@{#712426}
-
Wez authored
Content is typically rendered into a surface by the GPU process, and displayed into a window owned by the Browser process. In the Scenic Ozone platform this was implemented using import/export Nodes connecting the GPU & Browser. Content rendered to a ScenicSurface by the GPU process is now exported for display by a ScenicWindow via a nested View. This is a revert of https://chromium-review.googlesource.com/c/chromium/src/+/1661389 with additional changes to restrict what the embedded View can do with input. Bug: fuchsia:29945, 974826 Change-Id: Ib674ea1eab8f013e29606432300b368cdc582137 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895745Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#712425}
-
Mingjing Zhang authored
This CL adds a trace string that represents frame reporting events and will be output as part of the DCHECK message when the DCHECK fails. Each letter in the trace string represent one reporting function call. The length of the trace string is capped at 2,000 characters. Bug: 1017291 Change-Id: I377737730b1f78a7ca87e53db6acba591ee00090 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895379 Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#712424}
-
Matt Menke authored
Nothing populates it yet, other than a test this CL adds. Bug: 997049 Change-Id: I19012744ce7333cf88a7a82e79fc4a4cc99b6cfa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895859Reviewed-by:
Eric Orth <ericorth@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#712423}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/1776a0ea..d3f9d9a2 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,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Id39f9d10783280bf091e69369d3a5ba90af7fb8b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898176Reviewed-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@{#712422}
-
