- 12 Sep, 2019 40 commits
-
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/2b6c52bb82a0..5bd596e325f5 git log 2b6c52bb82a0..5bd596e325f5 --date=short --no-merges --format='%ad %ae %s' 2019-09-12 tneda@google.com Merge "perfetto-ui: Adding new track for heap profile" 2019-09-12 ilkos@google.com Merge "Profile visitor to use the AppendedData symbol packets" Created with: gclient setdep -r src/third_party/perfetto@5bd596e325f5 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=perfetto-bugs@google.com Bug: None Change-Id: I2cd2e4fa8030eb35c6ca2b3414126679c7b240a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801834Reviewed-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@{#696135}
-
Jon Mann authored
This time limit exemption was added as a temporary workaround for background notifications in Android Messages on ChromeOS and is no longer needed. Bug: 869714 Change-Id: Idd25f1cd8c8224621361a5031b389f6f81075123 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1789966Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Jon Mann <jonmann@chromium.org> Cr-Commit-Position: refs/heads/master@{#696134}
-
Nico Weber authored
This was added very early during GN bringup. Nowadays, pdb files are listed as outputs of the solink, solink_module, and link tools in build/toolchains/win/BUILD.gn, and data_deps does the right thing with these outputs. No intended behavior change. To make sure this is behavior-preserving, I ran the following command with and without this patch (which behind the scenes is what the isolate generation uses): gn gen out/gn --runtime-deps-list-file=<(echo -e \ //chrome/test:performance_test_suite\\n\ //net:net_unittests\\n\ //content/test:content_browsertests) I then compared the contents of these 3 files with and without the patch: out/gnwin/net_unittests.exe.runtime_deps out/gnwin/content_browsertests.exe.runtime_deps out/gnwin/obj/chrome/test/performance_test_suite.stamp.runtime_deps Without this patch, the pdbs I'm removing are all listed twice. With it, they're still listed, but only once (which is enough). Bug: none Change-Id: I5a992f854e69f0c09f10c6ec993974608b6a8550 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801495Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
David Benjamin <davidben@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#696133}
-
Tommy Li authored
This feature has launched to 100% now, and we are now just cleaning up the client side code. Bug: 958655 Change-Id: I3fa869ac03358a14d947f48b46a6a7a1566f51db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1783699Reviewed-by:
Dan Beam <dbeam@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#696132}
-
Mohammad Refaat authored
Now that CookieStore is launched by default, Remove the flag and the conditions that depends on it. Bug: 989554 Change-Id: I4172766b1c9fcf5cbd25e721bd002aba2c1417d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1729253 Commit-Queue: Mohammad Refaat <mrefaat@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Auto-Submit: Mohammad Refaat <mrefaat@chromium.org> Cr-Commit-Position: refs/heads/master@{#696131}
-
Sean Gilhuly authored
Bug: 972621, 954328 Change-Id: I8cad41da4a84ddee529ed8cc55287ed6a8335a65 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800522 Commit-Queue: Jonathan Ross <jonross@chromium.org> Reviewed-by:
Jonathan Ross <jonross@chromium.org> Auto-Submit: Sean Gilhuly <sgilhuly@chromium.org> Cr-Commit-Position: refs/heads/master@{#696130}
-
Min Qin authored
The current logic on passing DownloadEntry to DownloadItemImpl is very weird. First it is created in InProgressDownloadManager, and cached there. Later when DownloadItemImpl is created, it will retrieve the entry from InProgressDownloadManager. This CL fixes the above behavior by passing DownloadEntry directly to DownloadItemImpl ctor. So we don't need to expose some interface from DownloadItemImplDelegate. No functionality change, just refactoring. Change-Id: Iba7f0b54877149d1f4681cb161eac62dc5f872d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1797700Reviewed-by:
Xing Liu <xingliu@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#696129}
-
Matt Wolenetz authored
This change: * Adds a base::Feature "RevokeMediaSourceObjectURLOnAttach", disabled by default. * Adds logic to auto-revoke the MediaSource object URL immediately on successful start of attachment to HTMLMediaElement using that URL. This logic is used only if the feature, above, is enabled. * Updates URLFileAPI to also have a revokeObjectURL method that takes an ExecutionContext*, since finding a suitable ScriptState* for the existing version of revokeObjectURL is fraught with complexity due to the asynchronous selection of, and loading of, various URLs by an HTMLMediaElement. * Lets HTMLMediaElement find the new base::Feature, which is in media::, including relaxing the associated presubmit audit similar to other relaxations. Since unrevoked MSE Object URLs retain a strong reference to the underlying MediaSource objects, MSE Object URL autorevocation is hoped to reduce memory bloat from idle, otherwise unreferenced, attached HTMLMediaElement+MediaSource object groups by letting them be garbage collected. With this change, short-term experimentation using Chrome variations of the new feature can help validate if autorevocation is helping reduce such bloat. BUG=920044 Change-Id: Ibc15c59d0e8166a0b1d28fcf3e0eb275bb69244d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1708897Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Reviewed-by:
Dan Sanders <sandersd@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org> Cr-Commit-Position: refs/heads/master@{#696128}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/google/googletest.git/+log/33a0d4f6d76a..c7a03daa99e7 git log 33a0d4f6d76a..c7a03daa99e7 --date=short --no-merges --format='%ad %ae %s' 2019-09-12 shaindel@google.com Merge pull request #2387 from kuzkry:iff 2019-09-12 hgsilverman@gmail.com Googletest export 2019-09-12 absl-team@google.com Googletest export Created with: gclient setdep -r src/third_party/googletest/src@c7a03daa99e7 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/googletest-chromium-autoroll Please CC isamsonov@google.com,staphany@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:mac_optional_gpu_tests_rel;luci.chromium.try:linux_chromium_cfi_rel_ng;luci.chrome.try:win_chrome_official TBR=isamsonov@google.com,staphany@google.com Bug: None Change-Id: I06162dce47def1cd27a2aff46f5756a000814c50 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800733Reviewed-by:
-
Clark DuVall authored
Throw WebLayer specific exeptions instead of AndroidRuntimeException. Also use a null File for in-memory profile. Change-Id: I2d72226b5e5b8d08f4734e8467f948b9ac618624 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787466 Commit-Queue: Clark DuVall <cduvall@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#696126}
-
Donn Denman authored
Allows Android Smart Text Selection to be enabled along with full Contextual Search semantics. Both systems look at page text. There used to be a bottleneck in getting surrounding text that only allowed a single request/response, but now that's been moved to Mojo, so multiple requests are allowed. The limitation was in the IPC browser/renderer connection, which Mojo avoids. This CL removes the logic that limited these systems when they were both active. Also annotate all the SelectionEventType ints correctly. BUG=961114 Change-Id: If588a532ccb3fe46a39528ec725c277ec77e7e0b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769532Reviewed-by:
Shimi Zhang <ctzsm@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Auto-Submit: Donn Denman <donnd@chromium.org> Cr-Commit-Position: refs/heads/master@{#696125}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/032cf12f2b84..de4456f6f23a git log 032cf12f2b84..de4456f6f23a --date=short --no-merges --format='%ad %ae %s' 2019-09-12 robertphillips@google.com Add bench for SkDDLRecorder detach Created with: gclient setdep -r src/third_party/skia@de4456f6f23a 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 mtklein@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=mtklein@google.com Bug: None Change-Id: I1aed5013f4fc3523f2f5f3030c166ab49b0c65cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800734Reviewed-by:
-
Shimi Zhang authored
This is a followup of http://crrev/c/1685837. See more at https://docs.google.com/document/d/1Jwfbzbe8ozaoilhqj5mAPYbYGpgZCen_XAAAdwmyP1E/edit Bug: 955171 Change-Id: Ic526fc11165050fe1a5a4af4995bdc40eb309755 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1795998Reviewed-by:
Oksana Zhuravlova <oksamyt@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Shimi Zhang <ctzsm@chromium.org> Cr-Commit-Position: refs/heads/master@{#696123}
-
Andrew Grieve authored
The APK when built in this mode is not configured correctly since system_webview_apk does not exist when !public_android_sdk. Bug: 1000763 Change-Id: I22490d94f06ef764c6b75490a39c7c11c0bbbb67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790381Reviewed-by:
Sam Maier <smaier@chromium.org> Reviewed-by:
-
Jazz Xu authored
Bug: 998028 Change-Id: I93e7279cefc0f718e86a34e51c8c81769d8aa740 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788511Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Jazz Xu <jazzhsu@chromium.org> Cr-Commit-Position: refs/heads/master@{#696121}
-
Darin Fisher authored
Bug: 1001053 Change-Id: Idb8474645e8a141d1439a64acaf06e6764def1e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790420 Commit-Queue: Darin Fisher <darin@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#696120}
-
Matt Menke authored
The enum is no longer being used in net/. However, it's still used by the WebRequestAPI to indicate how an auth challenge received through the NetworkService API should be responded to, so rather than remove the enum, this CL moves it to ExtensionWebRequestEventRouter. This CL also converts it to an enum class, for better type safety, at the cost of additional verbosity. NetworkDelegate::AuthCallback is similarly moved to ExtensionWebRequestEventRouter. Bug: 934009 Change-Id: I04dca2c6d66ca52b8cc4c8fa44e7d5af1429646f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799367Reviewed-by:
Eric Roman <eroman@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#696119}
-
chrome://media-router-internalsdpapad authored
- Convert HTML Imports to JS modules. - Replace CallJavascriptFunction with ResolveJavascriptCallback. - Add JS type checking. Bug: 937747 Test: Page still loads, no HTML import warnings in DevTools Change-Id: I8074a5074cc07d5994714ad1fe62f0e57671e194 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799687Reviewed-by:
Brandon Tolsch <btolsch@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#696118}
-
Henrique Nakashima authored
HashUtil is a simple move so the build target can be created. Other moves from chrome/lib, chrome/android/java/src/org/chromium/chrome/ browser/util will follow. It is important to have a build target so that a dependency from clank can be established before the code that clank uses is moved. Bug: 995916 Change-Id: Ifda5212436856d47b4e628cc00c7039ce1520464 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792794Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Cr-Commit-Position: refs/heads/master@{#696117}
-
Matt Mueller authored
Provide a SystemTrustStoreProvider::CreateDefaultForSSL static method to create a default instance of SystemTrustStoreProvider that calls CreateSslSystemTrustStore. Change-Id: I5178307d0fd586b2a0dedcbb6223ad7ab1aff713 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1791637Reviewed-by:
Clark DuVall <cduvall@chromium.org> Reviewed-by:
-
Pavel Yatsuk authored
******************************Resource Sizes Diff****************************** Specifics: +73,728 bytes main lib size +618,840 bytes main dex size +1,012,450 bytes normalized apk size InstallSize: +389,856 bytes APK size +1,113,899 bytes Estimated installed size (Android Go) +2,889,969 bytes Estimated installed size InstallBreakdown (+2,890,011 bytes): +3,954 bytes Native resources (no l10n) size +2,278 bytes Non-compiled Android resources size +73,728 bytes Native code size +3,412 bytes PNG drawables size +634 bytes Package metadata size +2,778,013 bytes Java code size +27,992 bytes Compiled Android resources size Dex: +2,929 entries fields +2,487 entries strings +3,790 entries unique methods +1,264 entries types +3,790 entries methods R=pnoland@chromium.org Bug: 948739 Change-Id: I4e08cbea448b47a3fb876a3dbd0caf5def7545b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801059Reviewed-by:Patrick Noland <pnoland@chromium.org> Commit-Queue: Pavel Yatsuk <pavely@chromium.org> Cr-Commit-Position: refs/heads/master@{#696115}
-
dpapad authored
- i18n_template.js - js_template_compiled.js - load_time_data.js Had to update jstemplate_builder.cc to manually decompress load_time_data.js and js_templacte_compiled.js, since it is directly reading those files from the resource bundle. Bug: 965089,985906 Change-Id: I969a8378daa63cf0ef52f9ce28758925a8843207 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1709156Reviewed-by:
-
Antonio Gomes authored
This is yet another prepration CL that removes the references to RenderThreadImpl (including its inclusion header //content/renderer/render_thread_impl.h) from peer_connection_dependency_factory.cc. The corresponding blink::Platform method is used. BUG=787254 R=guidou@chromium.org, haraken@chromium.org Change-Id: I6f180397b56d5dbe2758e200315fec3933608531 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799085 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@{#696113}
-
Scott Haseley authored
This CL performs the following renames: - Task --> DOMTask - TaskQueue --> DOMTaskQueue - Scheduler --> DOMScheduler The generic nature of the previous names led to some confusion between classes in base/ and platform/scheduler. "DOM" is commonly used as a prefix to differentiated web-exposed classes. Bug: 979017 Change-Id: I285e31d2ef72f3b6fc0b17e0b3cfbaf16ef4fe7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796115 Commit-Queue: Scott Haseley <shaseley@chromium.org> Reviewed-by:
Nate Chapin <japhet@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#696112}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: Ia6b0a3994055266ead3310e67a63f30664492a53 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800732Reviewed-by:
-
Alex Clarke authored
We were incurring the binary size cost of passing a scoped_refptr<TaskRunner> by value which impacts PostTaskAndReply. This patch removes that overhead saving ~4kb. Bug: 906125 Change-Id: Iac456b5328b96e2b61f811f704f96e0beffecd66 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790448 Commit-Queue: Alex Clarke <alexclarke@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Etienne Pierre-Doray <etiennep@chromium.org> Cr-Commit-Position: refs/heads/master@{#696110}
-
Stephen Chenney authored
The compositor thread sends input and looks at renderer_deferral_state_, while the main thread sets that state through other methods. Make renderer_deferral_state_ atomic. R: danakj@chromium.org Bug: 1002124, 999189, 998313 Change-Id: I4bef32551892f1a096a9993246f719ac53f13d3c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796265Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Commit-Queue: Stephen Chenney <schenney@chromium.org> Auto-Submit: Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#696109}
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /chromeos/dbus/power. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=abhishekbh@chromium.org Bug: 891670 Change-Id: I18c556c1f133cd90acd9f09e488e2ac7a5791966 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1786688 Auto-Submit: Gabriel Charette <gab@chromium.org> Commit-Queue: Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Cr-Commit-Position: refs/heads/master@{#696108}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/d94d6a3d1103..c9fa9fe9379f git log d94d6a3d1103..c9fa9fe9379f --date=short --no-merges --format='%ad %ae %s' 2019-09-12 digit@google.com [memfd] Create support class for Linux memfd-backed region. Created with: gclient setdep -r src/third_party/swiftshader@c9fa9fe9379f If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Please CC swiftshader-team+autoroll@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_chromium_msan_rel_ng;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=swiftshader-team+autoroll@google.com Bug: chromium:b/140419396 Change-Id: Ib38f1b5d2be2cb612e6e90325da1faa57d0f24f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800730Reviewed-by:
-
David Black authored
Per guidelines from UX, when the view is un-hovered, the auto-close timer should resume with whatever time is left remaining. Bug: b:140645793 Change-Id: I0370d2ac6f534a9e0b3ac8dd4486d6298fe48ce2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1797344 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Reviewed-by:
Tao Wu <wutao@chromium.org> Cr-Commit-Position: refs/heads/master@{#696106}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/9baaac63..52bc8577 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: Ic4266a4b038cfe2429f950f7e2a1c9d8c27adeef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801070Reviewed-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@{#696105}
-
Kevin Plybon authored
Change-Id: I034db408f5787f8d8a4dc4ad40c53319410b9756 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801069 Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by:
Max Moroz <mmoroz@chromium.org> Cr-Commit-Position: refs/heads/master@{#696104}
-
Peter Kasting authored
Cleanup only, no functional change. Bug: 1000055 TBR: sadrul,kinuko,thestig, mthiesse, zmo, msarda, csharrison, jochen Change-Id: Idc3d434d6cd5a5d9125c5e00047cbc6219616769 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790724 Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#696103}
-
Pavel Yatsuk authored
This CL adds boolean column created_from_play_api to keywords table. The column will indicate search engine created from information retrieved from Play API. R=pkasting@chromium.org,wylieb@chromium.org BUG=1002271 Change-Id: Ib3b814a382675cc2c9b6c7d3cb7c6c539df07a8c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1798883Reviewed-by:
Kevin Bailey <krb@chromium.org> Reviewed-by:
Brandon Wylie <wylieb@chromium.org> Commit-Queue: Pavel Yatsuk <pavely@chromium.org> Cr-Commit-Position: refs/heads/master@{#696102}
-
Aleks Totic authored
NG was computing inline_size of replaced elements with no intrinsic size (only aspect ratio) incorrectly. Element's margin was not considered. The fixes are: - update code to use margin when computing inline_size - update wpt exaustive replaced size test to include margins - remove many stale expectations for position-absolute-replaced-minmax.html With the updated test, Legacy still fails single test. I might have to update expectations. Bug: 1002748 Change-Id: I98b8595ce0e4dc0d19d1d8020e33fe17e125d41d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799422 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#696101}
-
Xianzhu Wang authored
After we removed MULTIPLE_TEXTURE_MASK, SINGLE_TEXTURE_MASK just indicated whether a layer was a mask. As whether a layer is a mask is only determined by Layer::SetMaskLayer(), the status can be indicated with a bool flag. This prepares for simplification of mask layer implementation. Bug: 994361 Change-Id: I40f062fe0c08221316a4ffb5a479777053fe9d54 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1798760 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#696100}
-
Jerome Jiang authored
https://chromium.googlesource.com/webm/libvpx.git/+log/5a0242ba5c8f..c094391e954a $ git log 5a0242ba5..c094391e9 --date=short --no-merges --format='%ad %ae %s' 2019-09-10 jianj vpx_clear_system_state after drop due to overshoot Created with: roll-dep src/third_party/libvpx/source/libvpx R=johannkoenig@google.com BUG=999780 Change-Id: I03d4a6231088a20c6cfe3d51b2b49aba2fcc7205 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799310 Commit-Queue: Johann Koenig <johannkoenig@google.com> Reviewed-by:
Johann Koenig <johannkoenig@google.com> Cr-Commit-Position: refs/heads/master@{#696099}
-
Gabriel Charette authored
These tests were previously migrated from single-threaded MessageLoop to a multi-threaded TaskEnvironment (then named ScopedTaskEnvironment) as part of crbug.com/891670. //base OWNERS decided in retrospect that it was better to keep a single-threaded option for TaskEnvironment and introduced SingleThreadTaskEnvironment. This CL retrofits that decision for /components/gcm_driver. This CL is a no-op if it passes CQ. This CL was uploaded by git cl split. R=fgorski@chromium.org Bug: 891670 Change-Id: Ic03b7ee5179a0e97bff844b3df0913b574f25efb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787138 Auto-Submit: Gabriel Charette <gab@chromium.org> Reviewed-by:
Jian Li <jianli@chromium.org> Commit-Queue: Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#696098}
-
Antonio Gomes authored
... when peer_connection_dependency_factory.cc|h gets Onion soup'ed. Namely, the methods are: - GetWebRTCMediaPermission() - GetWebRTCRendererPreferences() In order to test the code being added right away, the CL also routes the callsite through the newly added methods. The CL eliminates the needs for PeerConnectionDependencyFactory::CreatePortAllocator(), to reference RenderFrameImpl and RenderViewImpl directly. In a follow up CL, further methods will be added to replace the remaining references to base::CommandLine & switches. BUG=787452 R=guidou@chromium.org, haraken@chromium.org Change-Id: Ia897d304a21395c89fb032a4bfc80ec1a82ff858 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1799102 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
-
Emily Stark authored
This reverts commit 060c246f. Reason for revert: A blocker bug has been fixed, so this revert re-enables HTTP auth committed interstitials on ToT. Original change's description: > Disable HTTP auth committed interstitials by default > > Committed interstitials for HTTP auth renders the 401 body underneath > the auth prompt. This has proven to be problematic in some cases (see > bug). This CL disables committed interstitials for HTTP auth by > default while I work on a fix for avoiding rendering the 401 body. > > Bug: 992639 > Change-Id: I63195a66c4ad81452e24d260afdc7dd938eab769 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1756638 > Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org> > Commit-Queue: Emily Stark <estark@chromium.org> > Cr-Commit-Position: refs/heads/master@{#687695} TBR=vasilii@chromium.org,estark@chromium.org Bug: 992639 Change-Id: I8c31e07ab4ea2ab70bd4f7006a0d19970d7f7d34 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1801057Reviewed-by:
Emily Stark <estark@chromium.org> Commit-Queue: Emily Stark <estark@chromium.org> Cr-Commit-Position: refs/heads/master@{#696096}
-
