- 24 Nov, 2019 13 commits
-
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/30319dfe..2cab5168 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: Ib8d4f1e517689c90901bcf1c7c81d166b5644098 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932823Reviewed-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@{#718470}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/9259c834e642..5b50ce5a465a Created with: gclient setdep -r src-internal@5b50ce5a465a 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: I03232e0f5c144eeaeedbcd5f1cd4f8fde420a9c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932887Reviewed-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@{#718469}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/c2afc9928ece..9259c834e642 Created with: gclient setdep -r src-internal@9259c834e642 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: Id3d538b8df1eb889bb4860bf57522c5c4fe30861 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932400Reviewed-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@{#718468}
-
Antonio Gomes authored
BUG=787254 R=guidou@chromium.org Change-Id: I5b80b2322b3c9c15aa271dbe02a7f56a360d1e58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1929773 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@{#718467}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=govind@chromium.org Change-Id: Id84f9b7c3ee07002cf77f8ba8f6bd6edf556b9e7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932159Reviewed-by:
Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#718466}
-
Koji Ishii authored
This patch fixes ink-overflow computation for FragmentItem. Currently in the phase 1 of LayoutNG, we compute ink-overflow by traversing |LayoutObject| tree, using data in |NGPhysicalFragment| for objects laid out by LayoutNG. The initial implementation for FragmentItem missed this point and stored all ink-overflow to |NGFragmentItem|. In future, we will want to walk the fragment tree and fragment item lists. Fixes ~100, regresses 5 by revealing accidental passes. Bug: 982194 Change-Id: I0848d95c53baf9b9d9d08be58c87ae4271650361 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1928462Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#718465}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/5843d1e79594..c2afc9928ece Created with: gclient setdep -r src-internal@c2afc9928ece 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: Ic669a0d7399968a611cf52d81faf2b637af0f322 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932398Reviewed-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@{#718464}
-
Jinsuk Kim authored
This CL defines Tab as an interface, and provides a concrete implementation of it (TabImpl). Initially the codebase switches to TabImpl for accessing non-public interface methods. The interface starts with a subset of methods and gradually includes more, and eventually replace all the references of TabImpl with Tab. See https://bit.ly/2WibHVy for the design proposal. Bug: 995903 Change-Id: Iab5b9f703732f2c6b33b1fce8836de37acf21a7e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1883270 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#718463}
-
Maggie Chen authored
GPU.WatchdogThread.Event(kGpuWatchdogStart) was added too early (eg. watchdog creation time) that it couldn't be persistent. This histogram data was lost after crash or browser exit. Now delay the recording of kGpuWatchdogStart until the first OnWatchdogTimeout() to ensure this metric is created in persistent memory. Bug: 988344, 949839 Change-Id: I8829d31209511d34220eb17a4f39017929d7e0a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1929867 Commit-Queue: Maggie Chen <magchen@chromium.org> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Cr-Commit-Position: refs/heads/master@{#718462}
-
Aran Gilman authored
Bug: 1027612 Change-Id: Ic4f4e1e43eb59a9aa24595a15e35d9003c718733 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1929860 Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Auto-Submit: Aran Gilman <gilmanmh@google.com> Cr-Commit-Position: refs/heads/master@{#718461}
-
Tim Schumann authored
This reverts commit 4951143a. Reason for revert: PaintPreviewCompositorCollectionBrowserTest.TestInitializationSuccess fails on linux-chromeos-google-rel CI: https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-google-rel?limit=100 Instance: https://ci.chromium.org/p/chrome/builders/ci/linux-chromeos-google-rel/9587 example logs: [ RUN ] PaintPreviewCompositorCollectionBrowserTest.TestInitializationSuccess [1:1:1123/144534.806495:WARNING:hugepage_text.cc(182)] The ordering seems incorrect, fall back to small page [17651:17651:1123/144534.888293:WARNING:audio_manager_linux.cc(52)] Falling back to ALSA for audio output. PulseAudio is not available or could not be initialized. [17651:17651:1123/144534.982920:INFO:profile_network_context_service.cc(206)] Using built-in cert verifier. [17651:17651:1123/144534.988049:ERROR:proximity_auth_profile_pref_manager.cc(189)] Failed to find local state prefs for current user. [17651:17651:1123/144535.011423:INFO:profile_network_context_service.cc(206)] Using built-in cert verifier. [17651:17651:1123/144535.017906:INFO:profile_network_context_service.cc(206)] Using built-in cert verifier. [17651:17651:1123/144535.019873:INFO:remote_commands_service.cc(40)] Fetching remote commands. [17651:17651:1123/144535.019945:WARNING:remote_commands_service.cc(42)] Client is not registered. [17651:17651:1123/144535.019973:INFO:remote_commands_invalidator.cc(34)] Initialize RemoteCommandsInvalidator. [17651:17651:1123/144535.019993:INFO:remote_commands_invalidator.cc(59)] Starting RemoteCommandsInvalidator. [17651:17651:1123/144535.020014:INFO:remote_commands_invalidator.cc(130)] RemoteCommandsInvalidator ReloadPolicyData. [17651:17651:1123/144535.020034:INFO:remote_commands_invalidator.cc(172)] Unregister RemoteCommandsInvalidator. [17651:17651:1123/144535.022879:WARNING:wallpaper_controller_client.cc(370)] Cannot get wallpaper files id in RemovePolicyWallpaper. This should never happen under normal circumstances. [17651:17651:1123/144535.064656:ERROR:account_manager_migrator.cc(248)] Could not find a refresh token for the Device Account. [17651:17733:1123/144535.158896:WARNING:google_brand_chromeos.cc(39)] Brand code file missing: /opt/oem/etc/BRAND_CODE ../../content/public/test/browser_test_base.cc:605: Failure Failed RunLoop::Run() timed out. Stack trace: #0 0x55f14751ff09 (/b/s/w/ir/out/Release/browser_tests+0x70a8f08) [17651:17651:1123/144605.264003:INFO:remote_commands_invalidator.cc(47)] Shutdown RemoteCommandsInvalidator. [17651:17651:1123/144605.264078:INFO:remote_commands_invalidator.cc(70)] Stopping RemoteCommandsInvalidator. [17651:17651:1123/144605.264095:INFO:remote_commands_invalidator.cc(172)] Unregister RemoteCommandsInvalidator. [17651:17651:1123/144605.271554:WARNING:pref_notifier_impl.cc(40)] Pref observer for native_printing.external_print_servers_whitelist found at shutdown. [17651:17651:1123/144605.307536:WARNING:pref_notifier_impl.cc(40)] Pref observer for media_router.cast_allow_all_ips found at shutdown. [17651:17651:1123/144605.307567:WARNING:pref_notifier_impl.cc(40)] Pref observer for EulaAccepted found at shutdown. [17651:17651:1123/144605.307579:WARNING:pref_notifier_impl.cc(40)] Pref observer for browser.relaunch_heads_up_period found at shutdown. [17651:17651:1123/144605.307587:WARNING:pref_notifier_impl.cc(40)] Pref observer for browser.relaunch_notification_period found at shutdown. [17651:17651:1123/144605.307595:WARNING:pref_notifier_impl.cc(40)] Pref observer for browser.relaunch_notification found at shutdown. [ FAILED ] PaintPreviewCompositorCollectionBrowserTest.TestInitializationSuccess, where TypeParam = and GetParam() = (30599 ms) Original change's description: > [Paint Preview] Add font access for Linux > > Testing on Cluster Telemetry with Linux found an issue where SkFontMgr > wasn't initialized properly resulting in inability to start the > compositor. This is due to the lack of font access since we didn't > call EnsureBlinkInitializedWithSandbox. However, the compositor only > needs a fraction of the features that come with that call. This CL > starts the font_service which enables the SkFontMgr to initialize. Note > that we don't require WebSandboxSupport to be initialized (at least for > Linux) as all the fonts are stored in the provided SkPictures. > > Test coverage for this will be added as part of the end-to-end tests > for the player. That should catch any issues with other platforms. > > Bug: 1023377 > Change-Id: I95a224f6f3759b1b8f8df2989222757ebbd4f12e > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1908062 > Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org> > Reviewed-by: Jochen Eisinger <jochen@chromium.org> > Reviewed-by: Dominik Röttsches <drott@chromium.org> > Reviewed-by: Ian Vollick <vollick@chromium.org> > Cr-Commit-Position: refs/heads/master@{#718149} TBR=vollick@chromium.org,drott@chromium.org,jochen@chromium.org,ckitagawa@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1023377 Change-Id: I0de7093d6612f2836ff8bfce30db5758d06da258 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932797Reviewed-by:
Tim Schumann <tschumann@chromium.org> Commit-Queue: Tim Schumann <tschumann@chromium.org> Cr-Commit-Position: refs/heads/master@{#718460}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/6761e5fe15f3..5843d1e79594 Created with: gclient setdep -r src-internal@5843d1e79594 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: I71a774fa425311a06add10ebada2a48e8d14877f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932705Reviewed-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@{#718459}
-
Yuly Novikov authored
Suppress WebGL2 OpenGL failures due to which testing was paused. Also re-enable angle_perftests claimed to be fixed by this driver. TBR=bsheedy Bug: 1016459, 1026468, 1002313, 1023745 Change-Id: Iaafa8a2ee290bbc56df201f9ac51dea2f8612c71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932884Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#718458}
-
- 23 Nov, 2019 27 commits
-
-
Tim Schumann authored
As tests are failing on linux-chromeos-dbg TBR=yoshiki@chromium.org Change-Id: I4bf298ed79ff4fb20174d493594e61e55c94dfa4 BUG: 1027919 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932657Reviewed-by:
Tim Schumann <tschumann@chromium.org> Commit-Queue: Tim Schumann <tschumann@chromium.org> Cr-Commit-Position: refs/heads/master@{#718457}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/79330b60ac89..6761e5fe15f3 Created with: gclient setdep -r src-internal@6761e5fe15f3 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: I2ba76fe1ae7e6b806ce5f47d33586eab29ad9d9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932757Reviewed-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@{#718456}
-
Tim Schumann authored
Flaky on Windows and Mac platforms. TBR=crouleau@chromium.org BUG: 1027902 Change-Id: I354fd7411f1c7e65748550fcc297f576f0753e46 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932656 Commit-Queue: Tim Schumann <tschumann@chromium.org> Reviewed-by:
Tim Schumann <tschumann@chromium.org> Cr-Commit-Position: refs/heads/master@{#718455}
-
David Bokan authored
This threshold goes all the way back to: https://crrev.com/ee5bc3acd0986e8fa87df7f804c49c120b8c9bc2 and was initially added to avoid superfluous scrolling of form controls on horizontally scrollable pages. If an element horizontally overlaps the scrollport by at least this threshold it's considered "fully visible" as per the scroll into view algorithm: https://drafts.csswg.org/cssom-view/#scroll-an-element-into-view Horizontal scrollable pages and partially visible form controls should be rather uncommon and we should remove magical numbers from platform behavior. If this behaivor is truly required for form controls we can special case those but lets first try removing this altogether. This does have some small impact on web-facing APIs in that scrollIntoView (as well as fragment navigations) will now align an element that's only partially on screen with the left edge. This matches the behavior in FireFox and the spec. I've modified the input-placeholder-paint-order.html and the ShowUnhandledTapUIIfNeeded tests to avoid scrolling and anchor-frames-same-origin.html to expect horizontal scrolling for this reason. [Reland Note] When first landed this caused SitePerProcessHitTestBrowserTest.ScrolledNestedPopupMenuTest/0 to start failing on linux-chromeos-rel with viz disabled. It now passes when relanded on ToT since https://crrev.com/c/1928895 turned Viz on by default on ChromeOS so this CL is a simple reland. TBR=szager@chromium.org Bug: 916631,1027403 Change-Id: Ic830083defa3684b0f2570edc5ae92249a6d9f18 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1931381Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#718454}
-
Chromium WPT Sync authored
Using wpt-import in Chromium a346870b. With Chromium commits locally applied on WPT: e83dd93c "Limit backtracking on regexp called from blink." 500a74f5 "Add a runtime flag to enforce strict MIME type checks for workers." Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: I5206e5aa3180fa00fb999f5918d22e525ed71646 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932676Reviewed-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@{#718453}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/d8ffff17ad6f..79330b60ac89 Created with: gclient setdep -r src-internal@79330b60ac89 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 Bug: chromium:None,chromium:None,chromium:None Tbr: jbudorick@google.com Change-Id: I90e85c758bb3f2a08a107173fa693a605e1158bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932696Reviewed-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@{#718452}
-
Maggie Chen authored
After adding more thread time and 60 seconds to the watchdog timeout, we don't need full 60 seconds for initialization timeout. 30 seconds (kInitFactor=2) should be a good number. We can adjust it later when Timeout histogram data is available. (gpu/OWNERS is added to the owner list of gpu_watchdog_timeout.h. The constants defined in this file are for watchdog internal use and there is no security concern.) Bug: 988344, 949839 Change-Id: I9c5148f961e5db53c1b7123a7d5c7e2b13de6b89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1930143Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Commit-Queue: Maggie Chen <magchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#718451}
-
Vaclav Brozek authored
CSVPasswordSequence is the new CSV parser for importing credentials. Unlike its predecessor, CSVPasswordSequence does not create all of the parsed passwords in memory at once. Instead it parses the CSV row-per-row on demand. This reduces the memory usage (high water mark). This CL replaces the uses of the old CSV parser with CSVPasswordSequence and removes the old parser. It also changes base::Callback to base::OnceCallback for the importing callback to better reflect the flow of the program. Finally, it adds a test for PasswordManagerPorter to cover importing / writing to PasswordStore. Due to introducing the new parser, the csv_reader_fuzzer should reduce its memory consumption. Bug: 934326 Change-Id: If90a7483f826e83047114ac2b07ba1c62377c07d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1920121 Commit-Queue: Vaclav Brozek <vabr@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#718450}
-
Chromium WPT Sync authored
Using wpt-import in Chromium bbfce899. With Chromium commits locally applied on WPT: e83dd93c "Limit backtracking on regexp called from blink." 500a74f5 "Add a runtime flag to enforce strict MIME type checks for workers." 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: rob.buis@samsung.com: external/wpt/quirks NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: I83538e154e96d2b25783d7cec2623ffaa04dbf0c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932198Reviewed-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@{#718449}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/3bc4235baa96..50299de39976 git log 3bc4235baa96..50299de39976 --date=short --no-merges --format='%ad %ae %s' 2019-11-23 robertphillips@google.com Reland "Revert "Make FP optimizations helpers use SkAlphaType not GrColorType"" 2019-11-23 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 2019-11-23 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 2019-11-23 bsalomon@google.com Revert "Revert "Make FP optimizations helpers use SkAlphaType not GrColorType"" 2019-11-22 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 2019-11-22 bsalomon@google.com Test texture domain effect with local matrix. 2019-11-22 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 b8e748be6b94..4c7db77e0185 (21 commits) 2019-11-22 halcanary@google.com LICENSE file: clean up Created with: gclient setdep -r src/third_party/skia@50299de39976 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 robertphillips@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: chromium:None Tbr: robertphillips@google.com Change-Id: Ibb83f38b7fe64f1138cd6a31a7536c222167ecc5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932240Reviewed-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@{#718448}
-
Tim Schumann authored
and the subsequent PowerSaveBlockerLinux [3/4] and PowerSaveBlockerLinux [4/4] patches This reverts commit 2fdeba1b. ("Rewrite PowerSaveBlockerLinux [2/4]") This reverts commit 38fdf074. ("Rewrite PowerSaveBlockerLinux [3/4]") This reverts commit 3bc463b4. ("Rewrite PowerSaveBlockerLinux [4/4]") The patch caused timeouts in WakeLockTest.ChangeType() and other tests in the services_unittests on Linux Tests (dbg). More details in crbug/1027852. TBR=thomasanderson@chromium.org Bug: 1013812,1027852 Change-Id: I7688c72f9d9d6c12ff3731f3abde6a05508cfbeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932357 Commit-Queue: Tim Schumann <tschumann@chromium.org> Reviewed-by:
Tim Schumann <tschumann@chromium.org> Cr-Commit-Position: refs/heads/master@{#718447}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/485166945452..d8ffff17ad6f Created with: gclient setdep -r src-internal@d8ffff17ad6f 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: Ifaa804a17f33937722d774600087c1db3d7d0966 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932397Reviewed-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@{#718446}
-
Tim Schumann authored
This reverts commit 713420fb. Reason for revert: Causes MSAN failure More details in crbug/1027853 Original change's description: > ozone: Build with xkbcommon by default on is_linux && !is_chromecast > > This is necessary on Linux targets unless you don't want keyboard > support or only need support for a hardcoded US layout. > > Noticed because the CQ broke after building for ozone locally. This > reduces the number of flags needed to build as the Linux Ozone builder > does. > > Bug: 1026476 > > Change-Id: I77352c133b1f7552301b6e1f252d40a4f2c72106 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1925370 > Reviewed-by: Scott Violet <sky@chromium.org> > Reviewed-by: Kevin Schoedel <kpschoedel@chromium.org> > Commit-Queue: Michael Spang <spang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#718240} TBR=sky@chromium.org,spang@chromium.org,kpschoedel@chromium.org Change-Id: I4fa0de1881acbf1eaf7e70bfe4b24dbc84647f43 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1026476 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932436Reviewed-by:
Tim Schumann <tschumann@chromium.org> Commit-Queue: Tim Schumann <tschumann@chromium.org> Cr-Commit-Position: refs/heads/master@{#718445}
-
Antonio Gomes authored
Note that this CL does not move RTCSessionDescriptionPlatform to Oilpan heap. This is saved for a follow up step, in order to reduce th functionality change. R=guidou@chromium.org, haraken@chromium.org BUG=787254 Change-Id: Ib634f68efd25869518c16b564e0a2edcd05569f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1929497Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#718444}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 3c3fc55c. With Chromium commits locally applied on WPT: e83dd93c "Limit backtracking on regexp called from blink." 500a74f5 "Add a runtime flag to enforce strict MIME type checks for workers." Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Id84faba4a32d038b802098f5c40cc558efe29523 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932197Reviewed-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@{#718443}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4489e0e098c3..485166945452 Created with: gclient setdep -r src-internal@485166945452 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: Idd2a6169b97e33c23733c9d214b46cc5e6c45caf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932026Reviewed-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@{#718442}
-
Denis Kuznetsov authored
Remove unused oobe_change_picture that was not cleaned up during work that removed code that used it: https://chromium-review.googlesource.com/c/chromium/src/+/1488018 Bug: 903577 Change-Id: Ib327d1eaa3ff1ea2c62478c726cc8be6399ceefb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1919205Reviewed-by:
Roman Aleksandrov <raleksandrov@google.com> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#718441}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/031361ca8db6..4489e0e098c3 Created with: gclient setdep -r src-internal@4489e0e098c3 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: I2fd83cf2b42f12cbe1eef3683eb945f99daca6c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932025Reviewed-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@{#718440}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/86258aab8096..38b4db48f98c git log 86258aab8096..38b4db48f98c --date=short --no-merges --format='%ad %ae %s' 2019-11-23 cepheus@frii.com Fix #1983: __ is okay starting with ES 300, rather than 310. Created with: gclient setdep -r src/third_party/glslang/src@38b4db48f98c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/glslang-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,courtneygo@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Bug: None Tbr: radial-bots+chrome-roll@google.com,courtneygo@google.com Change-Id: If44dc21fe7ccb9ab6550c4aa71dfc26945f1cd11 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932022Reviewed-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@{#718439}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/16974ce1435b..031361ca8db6 Created with: gclient setdep -r src-internal@031361ca8db6 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: I0824897324d4e26cdfb5dbc4cb8a8e7f08788b82 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932021Reviewed-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@{#718438}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 5a1f7181. With Chromium commits locally applied on WPT: e83dd93c "Limit backtracking on regexp called from blink." 500a74f5 "Add a runtime flag to enforce strict MIME type checks for workers." Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Ib46d725e68fcd89e0988a2226b4d719a7d043a80 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932196Reviewed-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@{#718437}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/9d28b54b9e21..16974ce1435b Created with: gclient setdep -r src-internal@16974ce1435b 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 Bug: chromium:None Tbr: jbudorick@google.com Change-Id: I5d3055850d2803264940f0eee830ef192ef786f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932019Reviewed-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@{#718436}
-
David Tseng authored
- rename all *.extjs and *.unitjs files to *.js - fix all linter errors - run clang-format on all files (which are now enforced as .js files). Change-Id: Ia2064db1caed23e3cf85bee3f46ad716fac4e5dc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1925339 Commit-Queue: David Tseng <dtseng@chromium.org> Reviewed-by:
Anastasia Helfinstein <anastasi@google.com> Cr-Commit-Position: refs/heads/master@{#718435}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/733842c59dd2..8e9c3f663ccd git log 733842c59dd2..8e9c3f663ccd --date=short --no-merges --format='%ad %ae %s' 2019-11-23 cwallez@chromium.org Fix compilation on MSVC Created with: gclient setdep -r src/third_party/dawn@8e9c3f663ccd If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC cwallez@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:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel Bug: None Tbr: cwallez@google.com Change-Id: I4331a9b3d24bae0a07bd3aa94f7cd19afbd2618e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1932017Reviewed-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@{#718434}
-
nancy authored
When using AppService, all apps use launch process. However, for Chrome apps or Web apps, previous extension class uses ChromeLauncherController's ActivateApp interface: https://chromium-review.googlesource.com/c/chromium/src/+/1858538/16/chrome/browser/ui/app_list/search/extension_app_result.cc#b85 https://cs.chromium.org/chromium/src/chrome/browser/ui/app_list/app_list_client_impl.cc?q=ActivateApp&l=494&dr=CSs So update the AppService code to use the same interface for the non-platform apps, so that the user can switch back to the previous app window when using search, which is the consistent behaviour with M77. BUG=1026730 Change-Id: I3318f3cecc075394383dc2ed42836dad83c8edb3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1930245 Commit-Queue: Nancy Wang <nancylingwang@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#718433}
-
George Zhang authored
Change-Id: I996471f830c2fc5f00b4f9e45d9da5b2ed331b2b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1929844Reviewed-by:
ssid <ssid@chromium.org> Commit-Queue: ssid <ssid@chromium.org> Cr-Commit-Position: refs/heads/master@{#718432}
-
Sam McNally authored
low-pressure-touch-filtering is no longer used for any board. touch-noise-filtering and touch-calibration were only used by link, which is now AUE so remove the flags and related code. Bug: 1022236 Change-Id: I5df344823ed318b5f8880c6010724207ebc141bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913686Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Commit-Queue: Sam McNally <sammc@chromium.org> Cr-Commit-Position: refs/heads/master@{#718431}
-