- 08 Nov, 2018 40 commits
-
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/36b85be49443..c248c0542bc1 git log 36b85be49443..c248c0542bc1 --date=short --no-merges --format='%ad %ae %s' 2018-11-07 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-11-07 reed@google.com Revert "Revert "hide nested hinting enum"" 2018-11-07 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-11-07 goderbauer@google.com Comment out debug prints in SkFontMgr_custom_directory 2018-11-07 reed@google.com Revert "hide nested hinting enum" 2018-11-07 mtklein@google.com baby steps to GN check correctness 2018-11-07 reed@google.com hide nested hinting enum Created with: gclient setdep -r src/third_party/skia@c248c0542bc1 The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_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;master.tryserver.blink:linux_trusty_blink_rel TBR=allanmac@chromium.org Change-Id: Ifcc3448042128cd4eaa8e6c70621eb47fa39bb52 Reviewed-on: https://chromium-review.googlesource.com/c/1325129Reviewed-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@{#606347}
-
Andrew Luo authored
Bug: 902958 Change-Id: I1c8a6fbc21c871ced824c2b387196d2eae856728 Reviewed-on: https://chromium-review.googlesource.com/c/1324991Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Andrew Luo <aluo@chromium.org> Cr-Commit-Position: refs/heads/master@{#606346}
-
Yoshifumi Inoue authored
This reverts commit 77e3d644. Reason for revert: he test flakiness is caused by bot flakiness. Original change's description: > Mark DialogClientViewTest.ButtonPosition flaky on Mac > > This patch marks DialogClientViewTest.ButtonPosition flaky on Mac. > > TBR=bsep@chromium.org > > Bug: 903000 > Change-Id: I7d533956e437c143e4d4030dd357553fd4caabf1 > Reviewed-on: https://chromium-review.googlesource.com/c/1325592 > Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> > Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#606338} TBR=yosin@chromium.org Change-Id: Ib253d9870dcbae3ca12ccba6b7db76d0c34afd7e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 903000 Reviewed-on: https://chromium-review.googlesource.com/c/1325279Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#606345}
-
Yoshifumi Inoue authored
This reverts commit 5f5ef388. Reason for revert: he test flakiness is caused by bot flakiness. Original change's description: > Mark ScrollViewTest.ClipHeightToShortContentHeight flaky on Mac > > This patch marks ScrollViewTest.ClipHeightToShortContentHeight flaky on Mac. > > TBR=kylixrd@chromium.org > > Bug: 903001 > Change-Id: Iccc1792d207b44fe05d8abb3f1d86687a8567eec > Reviewed-on: https://chromium-review.googlesource.com/c/1325275 > Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> > Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#606337} TBR=yosin@chromium.org Change-Id: I06d5f420c5fcf8570812aa8b055e259a0861bbe1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 903001 Reviewed-on: https://chromium-review.googlesource.com/c/1325278Reviewed-by:
-
Yoshifumi Inoue authored
This reverts commit 1724744c. Reason for revert: The test flakiness is caused by bot flakiness. Original change's description: > Mark ViewTest.GetIndexOf flaky on Mac > > This patch marks ViewTest.GetIndexOf flaky on Mac. > > TBR=tfarina@chromium.org, sky@chromium.org > > Bug: 903002 > Change-Id: Ic8a82a9f09de1dab8a201c23452b86a3714f54be > Reviewed-on: https://chromium-review.googlesource.com/c/1325571 > Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> > Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#606330} TBR=yosin@chromium.org Change-Id: I2d0fffa6740ab614081632ea6a0bc08e26c6069f No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 903002 Reviewed-on: https://chromium-review.googlesource.com/c/1325593Reviewed-by:
-
Yoshifumi Inoue authored
This reverts commit c1ae026a. Reason for revert: It seems flakiness was caused by bot flakiness. Original change's description: > Mark BubbleFrameViewTest.RemoveFootnoteView flaky on Mac > > This patch marks BubbleFrameViewTest.RemoveFootnoteView flaky on Mac > > TBR=mrefaat@chromium.org > > Bug: 903003 > Change-Id: I7028c038018bf1547b92a5c7fb19b5ddf65c3537 > Reviewed-on: https://chromium-review.googlesource.com/c/1325590 > Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> > Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#606328} TBR=yosin@chromium.org Change-Id: I39fc11331c1c794763a00bb8a6b82c17dc0f20c5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 903003 Reviewed-on: https://chromium-review.googlesource.com/c/1325576Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/75fa8552337b..cf6df902d647 git log 75fa8552337b..cf6df902d647 --date=short --no-merges --format='%ad %ae %s' 2018-11-08 dbeam@chromium.org git cl status: Force description encoding to utf-8 to avoid throwing Created with: gclient setdep -r src/third_party/depot_tools@cf6df902d647 The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. BUG=chromium:none TBR=agable@chromium.org Change-Id: Ie393cafdd79f8633c9e3ef457843100131ea269e Reviewed-on: https://chromium-review.googlesource.com/c/1325334Reviewed-by:
-
Clifford Cheng authored
Bug: 903016 Change-Id: I9800b5fb42cf76982c28be85b557b66167bd3d1d Reviewed-on: https://chromium-review.googlesource.com/c/1325201 Commit-Queue: mark a. foltz <mfoltz@chromium.org> Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Cr-Commit-Position: refs/heads/master@{#606340}
-
yoshiki iguchi authored
If the flag is enabled, when pressing Ctrl-Shift-Alt-W, an ash window tree is dumped to the UI log. This CL adds WindowStateType of each window to that log. Bug: none Test: Press Ctrl-Shift-Alt-W and see the type is correctly dumped in log Change-Id: I813539a02378800fb124e133fadd850853b04059 Reviewed-on: https://chromium-review.googlesource.com/c/1322259 Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#606339}
-
Yoshifumi Inoue authored
This patch marks DialogClientViewTest.ButtonPosition flaky on Mac. TBR=bsep@chromium.org Bug: 903000 Change-Id: I7d533956e437c143e4d4030dd357553fd4caabf1 Reviewed-on: https://chromium-review.googlesource.com/c/1325592Reviewed-by:
-
Yoshifumi Inoue authored
This patch marks ScrollViewTest.ClipHeightToShortContentHeight flaky on Mac. TBR=kylixrd@chromium.org Bug: 903001 Change-Id: Iccc1792d207b44fe05d8abb3f1d86687a8567eec Reviewed-on: https://chromium-review.googlesource.com/c/1325275Reviewed-by:
-
Tatsuhisa Yamaguchi authored
Records metric for the number of clicks to open the help for managed device. Currently only enterprise-managed device shows such button. The variable is made to enum because we are likely to add same button for Family Link managed account. Bug: 885114 Change-Id: Ib508686737c0a17cbf0385a39f4d45b2fef863f7 Reviewed-on: https://chromium-review.googlesource.com/c/1319404Reviewed-by:
Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Cr-Commit-Position: refs/heads/master@{#606336}
-
Yoshifumi Inoue authored
This reverts commit 0b8e0f13. Reason for revert: It seems the flakiness caused by bot flakiness. Original change's description: > Mark BubbleDialogDelegateViewTest.VisibleAnchorChanges flaky > > This patch marks BubbleDialogDelegateViewTest.VisibleAnchorChanges flaky > > TBR=erikchen@chromium.org > > Bug: 903004 > Change-Id: I03f9d1f227dfdb281b04dc3073dbe803518f94e5 > Reviewed-on: https://chromium-review.googlesource.com/c/1325272 > Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> > Commit-Queue: Yoshifumi Inoue <yosin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#606325} TBR=yosin@chromium.org Change-Id: Ibf71df02f951f45ae43cec1f7428cee98f6da838 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 903004 Reviewed-on: https://chromium-review.googlesource.com/c/1325276Reviewed-by:
-
Esmael El-Moslimany authored
Bug: 867065 Change-Id: I05f3b8c4c1f313fa203302a0a27da5b002cedfe0 Reviewed-on: https://chromium-review.googlesource.com/c/1309219Reviewed-by:
calamity <calamity@chromium.org> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#606334}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=kariah@chromium.org Change-Id: I3eb5fb678ab86294180a942b607e92e9da53e5e3 Reviewed-on: https://chromium-review.googlesource.com/c/1325531Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#606333}
-
Dominick Ng authored
This CL: - stops recording site engagement metrics at startup - passes a scoped_refptr to HostContentSettingsMap in a background thread call to query the content settings. This addresses a use-after-free due to the HostContentSettingsMap being freed before the background thread execution completes. BUG=901287 Change-Id: I7a1cd46a4628da490387aa98a7fc383ae5a5adb3 Reviewed-on: https://chromium-review.googlesource.com/c/1322339Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/c9546dc1a494..316dba28ff32 Created with: gclient setdep -r src-internal@316dba28ff32 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=mmoss@chromium.org Change-Id: I38780796512a0e582b3e6a1f266250024b4332d8 Reviewed-on: https://chromium-review.googlesource.com/c/1325330Reviewed-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@{#606331}
-
Yoshifumi Inoue authored
This patch marks ViewTest.GetIndexOf flaky on Mac. TBR=tfarina@chromium.org, sky@chromium.org Bug: 903002 Change-Id: Ic8a82a9f09de1dab8a201c23452b86a3714f54be Reviewed-on: https://chromium-review.googlesource.com/c/1325571Reviewed-by:
-
Keishi Hattori authored
Converts Inspector* class allocations from new to MakeGarbageCollected. Bug: 757708 Change-Id: Ia74983400abed2d0aaeb4152947a5a3e77c6c74f Reviewed-on: https://chromium-review.googlesource.com/c/1322670Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Keishi Hattori <keishi@chromium.org> Cr-Commit-Position: refs/heads/master@{#606329}
-
Yoshifumi Inoue authored
This patch marks BubbleFrameViewTest.RemoveFootnoteView flaky on Mac TBR=mrefaat@chromium.org Bug: 903003 Change-Id: I7028c038018bf1547b92a5c7fb19b5ddf65c3537 Reviewed-on: https://chromium-review.googlesource.com/c/1325590Reviewed-by:
-
Dan Sanders authored
This is a partial revert of commit 44323658. Bug: 902900 Change-Id: I0279d839a83ff67ac37508aad8de632b237fa65e Reviewed-on: https://chromium-review.googlesource.com/c/1325072 Commit-Queue: Tao Wu <wutao@chromium.org> Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Cr-Commit-Position: refs/heads/master@{#606327}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/4a463d57e9aa..e5e0aea5a404 git log 4a463d57e9aa..e5e0aea5a404 --date=short --no-merges --format='%ad %ae %s' 2018-11-08 tsepez@chromium.org Add pdf_fx_date_helpers_fuzzer target. 2018-11-07 tsepez@chromium.org Move another chunk of code to fx_date_helper.cpp. Created with: gclient setdep -r src/third_party/pdfium@e5e0aea5a404 The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: I2685eb6cdac8c1b5cce5289b322348267cdb6eac Reviewed-on: https://chromium-review.googlesource.com/c/1325329Reviewed-by:
-
Yoshifumi Inoue authored
This patch marks BubbleDialogDelegateViewTest.VisibleAnchorChanges flaky TBR=erikchen@chromium.org Bug: 903004 Change-Id: I03f9d1f227dfdb281b04dc3073dbe803518f94e5 Reviewed-on: https://chromium-review.googlesource.com/c/1325272Reviewed-by:
-
Ned Nguyen authored
Caleb is stepping up to work on Telemetry & benchmarking harnesses Charlie & Annie, Emily now work on Chrome speed metrics, sharing review load of the web platform benchmarks peria@'s ownership of oilpan is now reflected in https://cs.chromium.org/chromium/src/tools/perf/contrib/oilpan/OWNERS Update the OWNER files to reflect the new ownership Change-Id: Ia3178e3610d00c14a04c420fdf062d53c9e422e2 NOTRY: true # CQ flake Change-Id: Ia3178e3610d00c14a04c420fdf062d53c9e422e2 Reviewed-on: https://chromium-review.googlesource.com/c/1325079 Commit-Queue: Ned Nguyen <nednguyen@google.com> Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Reviewed-by:
Annie Sullivan <sullivan@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Cr-Commit-Position: refs/heads/master@{#606324}
-
Jun Mukai authored
BUG=883523 TEST=none Change-Id: I92ec1fc1a652deecdbb0c3e1c5c94217cb09801f Reviewed-on: https://chromium-review.googlesource.com/c/1318681 Commit-Queue: Jun Mukai <mukai@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#606323}
-
Noel Gordon authored
These test helpers return an element's getBoundingClientRect width and height only. Include the DOMRect .top and .left as well [1]. [1] Useful for table mode + touch event testing, should be the same as W3C DOMRect .x and .y values. Would prefer those, but closure does not define them [2] whereas the W3C DOM specs do. [2] Filed https://github.com/google/closure-compiler/issues/3140 Bug: 839243 Change-Id: I0cce0e4a121fd7bca49645d09e98d013fbf920f0 Reviewed-on: https://chromium-review.googlesource.com/c/1325270 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#606322}
-
Eugene But authored
SadTabCoordinator will be used instead SadTabLegacyCoordinator for SadTab UI presentation. SadTabLegacyCoordinator does not use view controller and presents SadTab UI with WebState::ShowTransientView, which is misuse of WebState API. Bug: 901563 Change-Id: I7c29c82d0036c05d06e44e8806ff0b82adad34ca Reviewed-on: https://chromium-review.googlesource.com/c/1320593Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#606321}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/e121da84258f..4e6c54b6026d git log e121da84258f..4e6c54b6026d --date=short --no-merges --format='%ad %ae %s' 2018-11-08 stevenperron@google.com Fix up comment for |ParseFlags|. (#2037) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@4e6c54b6026d The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: I92abaeb5692490a12ce54edb9ef2bcb9502a0c17 Reviewed-on: https://chromium-review.googlesource.com/c/1325331Reviewed-by:
-
Azeem Arshad authored
This CL adds logic to set a default-to-persist cookie that will force the messages PWA to default to persisting the pairing. Also fixes issue with PWA installing unexpectedly when disabling features. Bug: 896057,900799 Change-Id: I77354ecbbccf69e1b9a97e1bcff82bccaeeb44b1 Reviewed-on: https://chromium-review.googlesource.com/c/1312305Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Jeremy Klein <jlklein@chromium.org> Commit-Queue: Azeem Arshad <azeemarshad@chromium.org> Cr-Commit-Position: refs/heads/master@{#606319}
-
Andrew Grieve authored
Change-Id: I3c59e8bcdb16fc1fd0275ac053c4a889931bfd3c Reviewed-on: https://chromium-review.googlesource.com/c/1324147Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Commit-Queue: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#606318}
-
Dave Tapuska authored
The feature shipped in M66. BUG=740886 Change-Id: I8a0c0927e80100ba652018c4d476e8797ad0349a Reviewed-on: https://chromium-review.googlesource.com/c/1323895 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Kunihiko Sakamoto <ksakamoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#606317}
-
Findit authored
This reverts commit b43c6fca. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 606292 as the culprit for failures in the build cycles as shown on: https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2I0M2M2ZmNhNDM0OWI3ZjJmZjUzYjQxOTI2ZjVkN2NiNDJlZTRlMGYM Sample Failed Build: https://ci.chromium.org/buildbot/chromium.linux/Cast%20Audio%20Linux/24981 Sample Failed Step: webkit_unit_tests Original change's description: > Make NGOffsetMapping available for non-LayoutNG LayoutBlockFlow > > This patch allows computing NGOffsetMapping for legacy inline > formatting context. > > For an experimental project "Invisible DOM"[1][2] to support > the find-in-page feature, it is needed to compute the text > content with whitespace collapsing applied, along with the > mapping to the DOM offset. This patch adds an API to compute > NGOffsetMapping even when the LayoutBlockFlow is not laid out > by LayoutNG for that purpose. > > Note that the project is still in the early phase. We may > revisit the design as it moves forward. > > [1] https://github.com/rakina/searchable-invisible-dom > [2] https://www.chromestatus.com/feature/5105291213406208 > > Bug: 636993, 873057 > Change-Id: I101b411960813a7b9b5c9c6e2db85d28737af882 > Reviewed-on: https://chromium-review.googlesource.com/c/1322184 > Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org> > Reviewed-by: Emil A Eklund <eae@chromium.org> > Reviewed-by: Yoshifumi Inoue <yosin@chromium.org> > Reviewed-by: Rakina Zata Amni <rakina@chromium.org> > Commit-Queue: Koji Ishii <kojii@chromium.org> > Cr-Commit-Position: refs/heads/master@{#606292} Change-Id: I7942801d9268078959619af61363c3a769fc0a53 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 636993, 873057 Reviewed-on: https://chromium-review.googlesource.com/c/1325350 Cr-Commit-Position: refs/heads/master@{#606316}
-
Etienne Pierre-doray authored
base::ThreadRestrictions::ScopedAllowWait is deprecated in favor of its more explicit counterpart. It should have been replaced by : * base::ScopedAllowBaseSyncPrimitivesForTesting in test files. * base::ScopedAllowSyncPrimitives in non-test files * base::ScopedAllowSyncPrimitivesOutsideBlockingScope when it's used on threads that don't allow blocking The last one is strongly frowned upon but this CL aims to document existing behavior rather than address it. Owners are encouraged to follow-up by fixing unnecessary waits and more particularly unnecessary waits outside-blocking-scope. Note: The non-for-testing versions require friend'ing in thread_restrictions.h but care was taken to add these friends ahead of git cl split (since it wasn't possible to do a line-by-line associated CL split). Refer to the top-level CL if necessary : https://chromium-review.googlesource.com/c/chromium/src/+/1288533 Please CQ if LGTY! This CL was uploaded by git cl split. R=piman@chromium.org Bug: 766678 Change-Id: I6f4e77e4d48a29afc52709cf1c1b82ef08be7f98 Reviewed-on: https://chromium-review.googlesource.com/c/1325229Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#606315}
-
Ben Wells authored
This reverts commit 2dd14e92. Reason for revert: This changes the meaning of existing code. E.g. the default app installer now calls with require_manifest false and always_update true, instead of require_manifest true. Original change's description: > [System PWAs] Make System PWAs update on Chrome launch. > > As a temporary measure, make System PWAs update on every Chrome launch. > This allows developers to see changes when they update manifests. This > will be replaced with a lighter, more robust system before consumer > launch. > > Bug: 836128 > Change-Id: I04e65a4a060a6e91c0f7d32bd44be51492e2fd4e > Reviewed-on: https://chromium-review.googlesource.com/c/1282503 > Commit-Queue: calamity <calamity@chromium.org> > Reviewed-by: Alan Cutter <alancutter@chromium.org> > Cr-Commit-Position: refs/heads/master@{#603800} TBR=calamity@chromium.org,alancutter@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 836128 Change-Id: I997eed1e5154ee1cc4f40eed55c6d2ffdadb2cf1 Reviewed-on: https://chromium-review.googlesource.com/c/1322185Reviewed-by:
Ben Wells <benwells@chromium.org> Commit-Queue: calamity <calamity@chromium.org> Cr-Commit-Position: refs/heads/master@{#606314}
-
Xiaocheng Hu authored
This patch converts the layout test with W3C testharness so that it doesn't rely on text dump comparison. This is also a preparation for using element.innerText as text dump; otherwise, this test generates different innerText dumps on different platforms (crrev.com/c/1318149), which is undesired. Bug: 887148 Change-Id: Ic279b9362ff054eb13f847e111baef37d0ef79e3 Reviewed-on: https://chromium-review.googlesource.com/c/1323890Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
-
Sunny Sachanandani authored
Reenable direct composition surface on devices without hardware overlays since it enables other power improvements like flip mode swap chain and disabling DWM redirection surface. To mitigate the original concerns about direct composition, this change makes DCLayerTree lazy initialize the video context and processor when a video layer is first used. On devices that don't support hardware overlays, the video context and processor will never be initialized. Bug: 894675, 900702 Change-Id: I8c9d5b78718de2620ca177410d4916390e7cb219 Reviewed-on: https://chromium-review.googlesource.com/c/1318808 Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
-
Steven Bennetts authored
This CL: * Introduces ShowKeyboard, HideKeyboard, and IsKeyboardVisible to keyboard_controller.mojom. * Includes HideReason for HideKeyboard (User or System). * Introduces keyboard::mojom::KeyboardEnableFlag::kTemporarilyEnabled for temporarily enabling the keyboard and showing it. * Renames keyboard::mojom::ReloadKeyboard() -> RebuildKeyboardIfEnabled(). * Introduces keyboard::mojom::ReloadKeyboardIfNeeded() which calls KeyboardUI::ReloadKeyboardIfNeeded() which updates the keyboard URL if it has changed and otherwise does nothing. * Replaces direct calls to KeyboardController from most of src/chrome Notable exception: ChromeVirtualKeyboardDelegate which will be transitioned in a follow up CL. Bug: 843332 Change-Id: I2faf000c8233ee0253b799a6e4780020ba3bbe1f Reviewed-on: https://chromium-review.googlesource.com/c/1309215 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Shu Chen <shuchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#606311}
-
Kent Tamura authored
type-change-state.html contains assertions for the following operation in [1]: > ..., and set its selection direction to "none". However it had an issue that the assertions checked if input.selectionDirection was "none". According to [2], input.selectionDirection can be "forward" on specific platforms. > update the element's selection direction to the given direction, unless > the direction is "none" and the platform does not support that direction; > in that case, update the element's selection direction to "forward". [1] https://html.spec.whatwg.org/multipage/input.html#the-input-element:set-the-selection-direction [2] https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#set-the-selection-direction Change-Id: Idc1a97fbb2036e351a788a1f758b3239308d8c71 Reviewed-on: https://chromium-review.googlesource.com/c/1322264Reviewed-by:
Rakina Zata Amni <rakina@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#606310}
-
Daniel Rubery authored
Extensions can download and execute a pyo or pyd with no user gestures, allowing for remote code execution. By marking pyo and pyd as ALLOW_ON_USER_GESTURE and DANGEROUS, respectively, we prevent this behavior. Bug: 902234 Change-Id: I295cdd2906ebe779670afb55196cea7715dc88f4 Reviewed-on: https://chromium-review.googlesource.com/c/1324419 Commit-Queue: Daniel Rubery <drubery@chromium.org> Reviewed-by:
Nathan Parker <nparker@chromium.org> Cr-Commit-Position: refs/heads/master@{#606309}
-
Oystein Eftevaag authored
This reverts commit 7acd0b52. Reason for revert: Re-landing with crashfix Original change's description: > Revert "Perfetto: Optimization to avoid string table lookup for _END events" > > This reverts commit a7afe430. > > Reason for revert: I suspect this may have caused crashes on https://ci.chromium.org/p/chrome/builders/luci.chrome.ci/linux-perf-fyi# > > Original change's description: > > Perfetto: Optimization to avoid string table lookup for _END events > > > > If we're emitting an _END trace event, we know that the string > > table entries for the name and the categories have already been > > emitted and so we don't need to check the string table in this > > case. > > > > R=primiano@chromium.org,skyostil@chromium.org > > > > Change-Id: I9107bffb0b48f7b71d0a42961a78a3861ffcb76f > > Reviewed-on: https://chromium-review.googlesource.com/c/1258043 > > Commit-Queue: oysteine <oysteine@chromium.org> > > Reviewed-by: Sami Kyöstilä <skyostil@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#600183} > > TBR=primiano@chromium.org,oysteine@chromium.org,skyostil@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Change-Id: Ia460750fcaf338c4bf24ee7317d58b7aeede9520 > Reviewed-on: https://chromium-review.googlesource.com/c/1293612 > Reviewed-by: Christian Biesinger <cbiesinger@chromium.org> > Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> > Cr-Commit-Position: refs/heads/master@{#601586} TBR=cbiesinger@chromium.org,primiano@chromium.org,oysteine@chromium.org,skyostil@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I8f3b5c21494d66fe9e6c25363627ec156f5abf12 Reviewed-on: https://chromium-review.googlesource.com/c/1325077 Commit-Queue: oysteine <oysteine@chromium.org> Reviewed-by:
oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#606308}
-
