- 10 Oct, 2019 40 commits
-
-
Miguel Casas authored
Some EDID blobs have HDR information in the extended CEA/CTA tags and we want to parse those to support these monitors/screens/panels. This CL extends the EdidParser code to read the Colorimetry Data block and the HDR Static Metadata, also extending the unit tests. ( Sadly clang-format also unrolls the test cases vector ). Test:./display_unittests --gtest_filter=*Edid* Bug: 1012846 Change-Id: I9937b0933d080f36633a8d2d425d3f4eed8fea0a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851284 Commit-Queue: Miguel Casas <mcasas@chromium.org> Reviewed-by:
Andres Calderon Jaramillo <andrescj@chromium.org> Cr-Commit-Position: refs/heads/master@{#704634}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/a441815f4dc3..8e881f105660 git log a441815f4dc3..8e881f105660 --date=short --no-merges --format='%ad %ae %s' 2019-10-10 jrosenth@chromium.org lib/git: open files for SAX parser before calling parse 2019-10-10 jrosenth@chromium.org lib/git: remove __init__ from RemoteRef 2019-10-10 jrosenth@chromium.org Remove contextlib.nested 2019-10-10 jrosenth@chromium.org Remove usage of iter(...).next Created with: gclient setdep -r src/third_party/chromite@8e881f105660 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@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:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: Iad7272edc475637babcb62fb4bb3f6227897a934 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852572Reviewed-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@{#704633}
-
Emircan Uysaler authored
This CL reduces Skia resource usage to address memory issues explained on fxb/36620. Test: Tested cast_runner on device. Bug: 982922, fuchsia:36620 Change-Id: I7af5c5a252c10110abe7b783527b48e933a86807 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851127 Commit-Queue: Emircan Uysaler <emircan@chromium.org> Reviewed-by:
Peng Huang <penghuang@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#704632}
-
Peter K. Lee authored
Default enable the feature "LanguageSettings" for M79. Bug: 1008433 Change-Id: I179da2485bb4b50481c2fea46c2e8216f6e29e7c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1825922Reviewed-by:
Mark Pearson <mpearson@chromium.org> Commit-Queue: Peter Lee <pkl@chromium.org> Cr-Commit-Position: refs/heads/master@{#704631}
-
Dmitry Gozman authored
Previously, we stored it in InternalDocumentStateData and applied in RenderFrameImpl. This change moves the override to DocumentLoader, to consolidate navigation related parameters in Blink. This corresponds to the step 15.d from the doc linked to the bug. Bug: 855189 Change-Id: Icfabe5cecec3deaab385d3617bc1d9f77c8c13b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846795Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#704630}
-
Mehran Mahmoudi authored
This adds support for adaptive icons for legacy web apps that are installed via app_banner_manager.cc. Bug: 977173 Change-Id: I783284b61e29bd9ae1ea24248cb35ed901eafc61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850538Reviewed-by:
Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Mehran Mahmoudi <mahmoudi@chromium.org> Cr-Commit-Position: refs/heads/master@{#704629}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/0fc9b3829b47..69537d798994 git log 0fc9b3829b47..69537d798994 --date=short --no-merges --format='%ad %ae %s' 2019-10-10 kjlubick@google.com [skottie-perf] Remove beetle from list of tests Created with: gclient setdep -r src/third_party/skia@69537d798994 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 kjlubick@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=kjlubick@google.com Bug: None Change-Id: I39563e294d0ba6c558985c3b222d4d4b8451d8c1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852846Reviewed-by:
-
Andrew Grieve authored
Goal is to have devs be aware that relocations are a thing. Change-Id: Ifd0edf08ff58dd5b7e8879c4f6f32f9b61b55096 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846798 Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#704627}
-
Makoto Shimazu authored
This CL converts base::Bind to BindOnce or BindRepeating, and also changes relevant base::Callbacks or base::Closures to appropriate types. This CL is split from https://crrev.com/c/1831621. This CL was uploaded by git cl split. R=boliu@chromium.org Bug: 1007760 Change-Id: I235ac01f30acce3df1ce9f8a59d6d0a9da73449e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849476Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Auto-Submit: Makoto Shimazu <shimazu@chromium.org> Cr-Commit-Position: refs/heads/master@{#704626}
-
Dmitry Gozman authored
Load times extension bindings are using some rare response data like "was alpn negotiated". Currently, these fields are stored in DocumentState, but they should be available on WebURLResponse instead. Moving these fields to WebURLResponse almost gets rid of DocumentState, which opens up more opportunities for cleanup around navigation flow. This corresponds to the step 15.e from the doc linked to the bug. TBR=csharrison@chromium.org Bug: 855189 Change-Id: I9c83b88d5d941798be4a851166b8e9894cc82974 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1844052 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
-
Mike Jackson authored
This change updates the version of axe-core to the latest released version. As part of picking up the latest version, the following additional changes were made: 1) axe-core removed the 'href-no-hash' rule, so that been removed from the test list. 2) Several 'link-in-text-block' tests started failing with axe-core v3.0.2 (and later). These tests do pass in v3.0.1, but are disabled as part of this change. - CrManagementA11yTest.SimpleTest_link_in_text_block - CrManagementA11yTestWithExtension. ExtensionSection_link_in_text_block - CrostiniAccessibilityTest.CROSTINI_link_in_text_block - CrExtensionsA11yTest.NoExtensions_link_in_text_block - MultideviceA11yTest.MULTIDEVICE_link_in_text_block - MultideviceFeaturesA11yTest. MULTIDEVICE_FEATURES_ACCESSIBILITY_link_in_text_block 3) One 'heading-order' test started failing in v3.0.0 (and later). This tests does pass in V2.4.2, but is disabled as part of this change. - CrExtensionsA11yTestWithMultipleExensions. WithExtensions_heading_order 4) One color-contrast test started failing. This test has been disabled since there other tests also disabled for this rule - WelcomeA11y.WelcomeFlow_color_contrast 5) Several DevTools a11y tests were failing. This change either updates the baseline or excludes the rule depending upon the test/rule itself. - http/tests/devtools/a11y-axe-core/basic-a11y-test.js - http/tests/devtools/a11y-axe-core/console-a11y-test.js - http/tests/devtools/a11y-axe-core/memory/heap-profiler-a11y-test.js - http/tests/devtools/a11y-axe-core/performance/performance-pane-a11y-test.js - http/tests/devtools/a11y-axe-core/sources/dom-breakpoints-pane-a11y-test.js I spent some time investigating the test failures, and it looks like axe-core is highlighting legitimate product issues. Bug: 984817 Change-Id: I245af601c4f086d96768fa5677e253095fc53087 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1841834 Commit-Queue: Mike Jackson <mjackson@microsoft.com> Reviewed-by:Yang Guo <yangguo@chromium.org> Reviewed-by:
Lorne Mitchell <lomitch@microsoft.com> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Cr-Commit-Position: refs/heads/master@{#704624}
-
Antonio Gomes authored
This CL corresponds to phase 6.3 on the design document [1]. [1] https://docs.google.com/document/d/1AJKVA5U4nDkyDB9p4ROrggWXadCxyy-grKaE9KS5vOU/ It moves the respective header webrtc_media_stream_track_adapter.h to Blink's public API layer (TODO added). Last, base::Bind instances will be replaced by their corresponding WTF implementations in a follow up. BUG=787254 R=guidou@chromium.org, haraken@chromium.org Change-Id: Ifb34c9e62556d46dd917b78c433e7a1f706bad1e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816778 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#704623}
-
David Roger authored
TBR=fdoray Bug: 1012601 Change-Id: I9540e12715b4b431af0b886c280f609b51851b10 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852429Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#704622}
-
Olya Kalitova authored
Adds Ansible playbook application step to default Crostini container startup if Ansible is successfully installed to default Crostini container. Test: unit_tests --gtest_filter="Crostini*" Bug: 998124 Change-Id: Id892b5316e2971e4a81ab6d155654c30202d4a6f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847343Reviewed-by:
Nic Hollingum <hollingum@google.com> Reviewed-by:
Timothy Loh <timloh@chromium.org> Commit-Queue: Olya Kalitova <okalitova@chromium.org> Cr-Commit-Position: refs/heads/master@{#704621}
-
Kush Sinha authored
MutableProfileOAuth2TokenServiceDelegate takes a boolean constructor parameter to assess whether token revocation is supported. This was required only on Chrome OS, for supervised users. With the introduction of Chrome OS Account Manager, this code path is no longer used. i.e. revoking all tokens from "Chrome" is no longer supported. This was partially cleaned up in https://crrev.com/c/1833609, which set the aforementioned parameter to |true| by default. Clean it up completely by removing the parameter altogether since it is always set to |true| anyways. Bug: 1010532 Change-Id: Idf585b5e6c2096febce83e641443fce394968319 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852427Reviewed-by:
-
Kush Sinha authored
Refactor Chrome OS Mirror header processing to make the cases more obvious. Bug: 1010532 Change-Id: Ieb2ed44152c34d2519e08e4b6684598fd9339aa2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852210 Commit-Queue: Kush Sinha <sinhak@chromium.org> Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#704619}
-
David Roger authored
TBR=fhorschig Bug: 1012221 Change-Id: I3b3f8f7fd9eb4c4a30db55410b9cda4a932c4590 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1851710Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Commit-Queue: David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#704618}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/1e9229f0fcb4..545a481a74a3 git log 1e9229f0fcb4..545a481a74a3 --date=short --no-merges --format='%ad %ae %s' 2019-10-10 wl@gnu.org * src/sfnt/sfwoff2.c (reconstruct_glyf): Check `triplet_size'. Created with: gclient setdep -r src/third_party/freetype/src@545a481a74a3 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/freetype-chromium Please CC bungeman@google.com,drott@google.com,thestig@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:linux_chromium_msan_rel_ng;luci.chromium.try:linux-blink-rel;luci.chromium.try:mac10.12-blink-rel;luci.chromium.try:mac10.13_retina-blink-rel;luci.chromium.try:win10-blink-rel;luci.chromium.try:win7-blink-rel TBR=bungeman@google.com,drott@google.com,thestig@google.com Bug: None Change-Id: I0f36a66a8fa7a1cb871ad188e0cb00dd0c0aed44 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852186Reviewed-by:
Dominik Röttsches <drott@chromium.org> Commit-Queue: Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/master@{#704617}
-
Bill Budge authored
This is a reland of d99ef40d Patchset #8 modifies the original to handle Write and Fetch completions in any order, which should fix the UAF crashes. diff: https://chromium-review.googlesource.com/c/chromium/src/+/1846336/1..8 Original change's description: > [code caching] Handle small data with a single read/write > > - Many small reads and writes are observed, especially with the JavaScript > code cache. Detect these and write all data into the entry's stream 0, > and clear stream 1. When reading, detect the entry size and for small > data, skip the stream 1 read and copy the data from the stream 0 read. > - Changes the stream 0 data (again) to a header with response time and > data size. The data size, while not strictly necessary right now, will > be needed if we implement de-duplication of identical entries stored by > multiple origins. We can add the code hash to this header, and the size > field will disambiguate between small entries and large de-duplicated > entries where we use the hash as a key to the data. > - This change should make small reads faster than before, as synchronous > completion of stream 0 reads is observed. Otherwise, there should be no > change in performance. > - Renames the buffers and completion callbacks to reflect small/large data > distinction. > > Bug: chromium:992991 > Change-Id: I6fb5337ef1e4148dd9f300f0a8c85acb401be62e > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1834562 > Commit-Queue: Bill Budge <bbudge@chromium.org> > Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> > Reviewed-by: Mythri Alle <mythria@chromium.org> > Reviewed-by: Maks Orlovich <morlovich@chromium.org> > Cr-Commit-Position: refs/heads/master@{#702667} Bug: chromium:992991,chromium:1011879 Change-Id: I9d589c45f567d93841536bb327381223be489928 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846336Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Maksim Orlovich <morlovich@chromium.org> Reviewed-by:
Mythri Alle <mythria@chromium.org> Commit-Queue: Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#704616}
-
Dominik Röttsches authored
This reverts commit dcdf3924. Reason for revert: Flaky on linux-chromeos-rel, see https://crbug.com/1013148 Original change's description: > arc: Provide replacement for deprecated PlayStoreShown UMA > > This rolls new set of UMA that replaces derpecated PlayStoreShown UMA > DD: > https://docs.google.com/document/d/1vfJMzoYsXtH-HWYydbb5VJvpT6WeJrfvaO81RHI5v2M/edit > > BUG=b:142251935 > TEST=Locally + unit tests. > > Change-Id: Ie2ec139b41e702dda604d92324b426782ed494ce > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1845935 > Commit-Queue: Yury Khmel <khmel@chromium.org> > Reviewed-by: Ilya Sherman <isherman@chromium.org> > Reviewed-by: Yusuke Sato <yusukes@chromium.org> > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Cr-Commit-Position: refs/heads/master@{#704384} TBR=xiyuan@chromium.org,yusukes@chromium.org,isherman@chromium.org,khmel@chromium.org Change-Id: Ie870309e4d2b879679e79d3b468736e51bab096f No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: b:142251935, 1013148 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852785Reviewed-by:
-
Elly Fong-Jones authored
This is now a property of an instance rather than a delegate method. This change introduces one minor behavior change: on Windows, HungRendererDialogView used to use a custom frame when Aero Glass was enabled; it will now use a custom frame when Aero Glass is enabled *or* the dialog is parented. I think the old behavior was a bug - it is not supported to use a custom frame with no parent window. Some reworking of the DialogTest suite was required, because this suite used to override ShouldUseCustomFrame to return true despite not having a parent widget for the test dialogs. This change introduces a parent widget so that these tests continue to get the custom frame they desire. Bug: 1011446 Change-Id: Ib3a8521368e1d7d779c2346fc74c97633fd13f16 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846317Reviewed-by:
Michael Wasserman <msw@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#704614}
-
Daniel Nicoara authored
Create a message that will be used by JS message channels to pass data back into the Webview client. Bug: b/141864193 Test: Compiled Change-Id: Iada17bc8f3cf1276f3a59813406f67bc43af8247 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850363Reviewed-by:
Alex Sakhartchouk <alexst@chromium.org> Commit-Queue: Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#704613}
-
Lowell Manners authored
This CL addresses a bunch of TODOs to properly set FrameHostMsg_DidCommitProvisionalLoad_Params on BackForwardCache navigations. With this CL, we store the DidCommitProvisionalLoad_Params used for the navigation on RenderFrameHostImpl, so that it can be reused when restoring from the BackForwardCache. This approach should be somewhat more future proof, particlarly when adding new fields to DidCommitProvisionalLoad_Params. In order to prevent any change in memory usage, the parameters are only stored on the RenderFrameHostImpl if BackForwardCache is enabled. Change-Id: Idb72e7366876e59d1dfd046fa83284f6f280422c Bug: 1005718 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819251 Commit-Queue: Lowell Manners <lowell@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#704612}
-
Arthur Hemery authored
Filtering remaining failing tests after increased sharding. Bug: 1007276 Change-Id: I75b67a9a9e60b5aa1741b2a33cdd53c35936f3a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852249Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
-
Canon Mukai authored
There were some differences between CompressionStream and DecompressionStream. So I aligned them. Also, I made CompressionStream accept BufferSource. Bug: 999091 Change-Id: Iaf6c3e54804a10ef22d05c5f4da0d5ae07d0e765 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847605 Commit-Queue: Canon Mukai <canonmukai@google.com> Reviewed-by:
Adam Rice <ricea@chromium.org> Reviewed-by:
-
Daniel McArdle authored
Bug: 1012848 Change-Id: If0b9c22bcf3d8341d826dfbc718fe47ebb6328ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850432 Commit-Queue: Dan McArdle <dmcardle@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#704609}
-
Thomas Tangl authored
- The profile feature buttons are added - Browser tests for the guest profile are re-enabled after crrev.com/c/1848371 - Autofill buttons are not shown in incognito mode - The guest profile is moved to selectable profiles Bug: 995720 Change-Id: I8499848add6fcc06bb984ce79c4bc5f36ac4546a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850155Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium c89b6cab. With Chromium commits locally applied on WPT: 23748e60 "Change text directive to text=" 2f7c5562 "Fetch Metadata: Split `sec-fetch-dest` out from other headers' 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: rouslan@chromium.org, mathp@chromium.org: external/wpt/payment-request NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Icac6ce43c8297b2649fc8f13232ecff7ae18e0f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852666Reviewed-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@{#704607}
-
David Roger authored
Only one test is flaky: EndToEndTestServerPush.ServerPushOverLimitNonBlocking However, it's not possible to disable only this test, because it's in a third party repository. This CL excludes the whole file from the build instead. TBR=rch Bug: 1010206 Change-Id: Id897407c56f59e632159c73a8d82f669e7c748ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852207Reviewed-by:
-
Maksim Sisov authored
No functionality changes here, but just moving the code and implementing missing interfaces in PlatformWindow. Bug: 990756 Change-Id: Ia39b15c580ea831a5e31bb6d76a807b3321c4f4f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1844820 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#704605}
-
Antonio Gomes authored
... in favor of its Blink renderer child class, RTCCertificateGenerator. CL also the alias WebRTCCertificateCallback to RTCCertificateCallback. BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: I05c68fc040cd1842052a639425dcc6654b11c8ca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1814606 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
-
Thomas Tangl authored
- The sync error description is set to either: "Sync is paused" or "Sync isn't working" - The sync button (no error) is changed to "Sync is on" - The bottom heading is changed to "Other people" Bug: 995720 Change-Id: I0f329462773f76c50d44fd178835474c01757b41 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849898 Auto-Submit: Thomas Tangl <tangltom@chromium.org> Reviewed-by:
-
Francois Doray authored
With this CL, PerformanceManager(Impl)::CallOnGraph can safely be called from any sequence, at any time. The callback does not run if the call is made before creation of after destruction of the Performance Manager. Implementation details: CallOnGraph always posts a task to the Performance Manager sequence. When the task runs on the Performance Manager sequence, it checks if a global pointer to the Performance Manager if set. This is not racy because the pointer is only modified from the Performance Manager sequence. If the pointer is set, the task gets the Graph from it and invokes the callback that was passed to CallOnGraph. Otherwise, the callback does not run. Bug: 980533 Change-Id: I3d640e0ae13d012d76bab1483a5d59afb977c1a8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849213 Commit-Queue: François Doray <fdoray@chromium.org> Reviewed-by:
Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#704602}
-
Sylvain Defresne authored
base::Bind/base::Callback are deprecated in favor of either base::BindOnce/base::OnceCallback or base::BindRepeating/ base::RepeatingCallback (depending on whether the callback is invoked once or multiple time). Convert all uses of base::Bind/base::Callback in services/shape_detection to the recommended methods/types. Bug: 1007831 Change-Id: Ie7ff4256ee62271fc7c3fe2b4acb17b791314090 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1839954 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#704601}
-
Tim van der Lippe authored
- Also upgrade the refactor script to automatically add the extensionpoint to the BUILD.gn to save myself some typing. Bug: 1006759 Change-Id: I3f789f96122569141194ce26ac288f01af68f6e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852209Reviewed-by:
-
Jan Krcal authored
This CL fixes a glitch with spacing between image and label when the button is shrinking down its label. Before this CL, the preferred size jumped by this spacing _at the end_ of the animation when GetText() becomes empty. This CL makes LabelButton disregard this spacing at the beginning of the animation. Bug: 1002160 Change-Id: I213d2ba50aefde01dba1b2615ef47b38f9e3b94c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849388 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Auto-Submit: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#704599}
-
David Roger authored
A lot of GCM tests are flaky. Disabling the worst offenders. Tests disabled: GCMChannelStatusSyncerTest.DisableRestartAndEnable CMDriverFunctionalTest.MessageReceived GCMDriverFunctionalTest.MessageWithCollapseKeyReceived GCMDriverInstanceIDTest.GCMClientNotReadyBeforeInstanceIDData CMDriverFunctionalTest.RegisterAgainWithDifferentSenderIDs GCMDriverFunctionalTest.RegisterAgainWithSameSenderIDs GCMDriverFunctionalTest.RegisterError GCMStatsRecorderImplTest.CheckinTest GCMDriverFunctionalTest.RegisterAfterUnfinishedUnregister TBR=peter Bug: 1010462 Change-Id: Ibb6f80366d356cbd28fffb567095f282fbca12c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852445Reviewed-by:
-
David Roger authored
TBR=tommycli Bug: 1012211 Change-Id: I81816b35680dd3be345e2b699779663cc11bdd74 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852784Reviewed-by:
-
Lily Chen authored
This change allows the SameSiteDataRemoverImpl to use the appropriate CookieAccessSemantics value when determining whether a cookie is effectively SameSite=None. Bug: 978172 Change-Id: I214e115b08d9591462f8405d9e7420bda481db48 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1846316 Commit-Queue: Lily Chen <chlily@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Cr-Commit-Position: refs/heads/master@{#704596}
-
Jan Krcal authored
This CL keeps the user email shown also if parent container is focused or hovered. It introduces a notification from the container towards the avatar button when the highlight state changes. In a follow-up CL the same notification will also be used to hide the border of the button. Bug: 967317 Change-Id: I80866957d7098c675ad20d7fd03d1f45ad5aeec9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1826678 Commit-Queue: Jan Krcal <jkrcal@chromium.org> Reviewed-by:
-
