- 06 Mar, 2019 40 commits
-
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/24b0b59e4409..47a505533310 git log 24b0b59e4409..47a505533310 --date=short --no-merges --format='%ad %ae %s' 2019-03-06 skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com Update go_deps asset Created with: gclient setdep -r src/third_party/skia@47a505533310 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-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=reed@chromium.org Change-Id: I93c0d51e92ae24c8a5869e662466931886d84198 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504797Reviewed-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@{#638041}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 4ad77ed4. With Chromium commits locally applied on WPT: 68d5240e "[ChromeDriver] Set key modifiers with mouse actions" 682affef "Implemented setPlaybackRate for scroll-linked Worklet Animations." 2b037bdc "[Animation Worklet] Fix flaky worklet-animation-with-fill-mode.https.html" 5fef8801 "Make -webkit-line-clamp respect line-height." 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: kenneth.r.christiansen@intel.com, rijubrata.bhaumik@intel.com: external/wpt/web-nfc NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: Ib32ca8a773ec22b0d2c1b8e3680bfdbc9f09d5c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504979Reviewed-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@{#638040}
-
Raul Tambre authored
This fixes many 64-bit truncation warnings. Bug: 488553, 879657, 921389 Change-Id: Ia3333d090f3a8137bbda8061fc205afca7be8566 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1484294Reviewed-by:
Asanka Herath <asanka@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Reviewed-by:
Takumi Fujimoto <takumif@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Raul Tambre <raul@tambre.ee> Cr-Commit-Position: refs/heads/master@{#638039}
-
Miyoung Shin authored
- unsigned long long / unsigned long long int -> uint64_t - long long / long long int -> int64_t - No logic change. - Reference: https://google.github.io/styleguide/cppguide.html#Integer_Types Bug: 929980 Change-Id: I1a44a3752f27b19cc3f2a493fec9540324c61dd9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504479Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Miyoung Shin <myid.shin@chromium.org> Cr-Commit-Position: refs/heads/master@{#638038}
-
Maksim Sisov authored
The issue with fences was identified a long time ago, but it did not concern Linux before. As long as we use surfaceless drawing and insert fences between each buffer swap, we noticed some perf regression. On a powerful laptop (dell 9560) with Intel 630 graphics, gpu::AsyncSwap took ~8ms with fences and ~1ms without fences. On Raspberry Pi3 with Broadcom GPU, swap took ~105ms and after the applied workaround it took 77ms. The FPS rate became 16-17FPS instead of 11-12FPS in aquarium webgl demo. https://docs.google.com/spreadsheets/d/1dKLIQ6dkql-X0ppdc5s5y-ws2hPM-RLgqgGzkFpgUe0/edit?usp=sharing Bug: 938286 Change-Id: I9fc616976a6ee3f9b4544ad7d8effc41e3ad4a91 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503576 Commit-Queue: Maksim Sisov <msisov@igalia.com> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#638037}
-
Robbie McElrath authored
This CL increases the shard count from 10 to 21, which matches the count specified in test_suite_exceptions.pyl. TBR=jbudorick@chromium.org Bug: 938656 Change-Id: I37bfa264ea7ee37fb092eec5f8b4267adb2ea2ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504978Reviewed-by:
Robbie McElrath <rmcelrath@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Cr-Commit-Position: refs/heads/master@{#638036}
-
Rakina Zata Amni authored
Some paths to Invisible-DOM related code (e.g. find-in-page) are not guarded sufficiently by runtime flag checks, so we're adding them in this CL. Bug: 873057 Change-Id: I1bfe49fe30bb0b84440c46604523b77146cc630f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1502257 Commit-Queue: Rakina Zata Amni <rakina@chromium.org> Reviewed-by:
Fergal Daly <fergal@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#638035}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/dbdb7ae5c8ed..327e9ae49a2d Created with: gclient setdep -r src-internal@327e9ae49a2d 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. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=shend@chromium.org,zmin@chromium.org,dvadym@chromium.org,petewil@chromium.org Change-Id: I98564bb7c9134a7ea2dc9298cf59903a85f31205 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504615Reviewed-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@{#638034}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/537ce575a441..24b0b59e4409 git log 537ce575a441..24b0b59e4409 --date=short --no-merges --format='%ad %ae %s' 2019-03-06 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 55959b0bb19f..a8300e56b550 (3 commits) Created with: gclient setdep -r src/third_party/skia@24b0b59e4409 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-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=reed@chromium.org Change-Id: I6a8eb9c5c564671a2076f8dc1def41fa2068b9e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504614Reviewed-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@{#638033}
-
Luciano Pacheco authored
Move helper function createShortcut to background.js to be able to re-use in directory_tree_context_menu.js and change its argument to be directory name instead of an object. This CL is a preparation to fix issue 925516 to make sure we don't break other uses of context menu. I'll add more tests in follow up CL for other types of roots in directory tree. Bug: 925516 Change-Id: Ifb475ac1fac7b8893c62a0270167ae8e357718ed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504473 Commit-Queue: Sam McNally <sammc@chromium.org> Reviewed-by:
Sam McNally <sammc@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#638032}
-
Leszek Swirski authored
The script streamer thread has now been replaced with posting tasks to the scheduler, so remove the flag picking between the two options and the dedicated streaming thread itself. Also, remove the streaming task re-posting now that the blocking/ non-blocking worker pools are merged. Bug: 874080 Change-Id: Ib9dabb91deddcc616d4edfe5efb13d8e584c0691 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503420Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Auto-Submit: Leszek Swirski <leszeks@chromium.org> Cr-Commit-Position: refs/heads/master@{#638031}
-
François Beaufort authored
This CL makes sure ActivateContents() is implemented for Chrome App windows so that Back to Tab button in the Picture-in-Picture window focuses back the window. Bug: 938250 Change-Id: I4814c52553b8b938bff4005f032a3f83f2e54ce8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503255Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Cr-Commit-Position: refs/heads/master@{#638030}
-
Jia authored
This cl contains the following changes 1. ModellerImpl will become an observer of ModelConfigLoader. 2. ModellerImpl will receive some parameters from ModelConfigLoader: (i). global curve spec (ii). model_als_horizon_seconds 3. ModellerImpl will have "average_log_als" param removed. 4. An equality operator for ModelConfig. Bug: 881215 Change-Id: I17cc5f2962b57e70921591bd84d862658a82a45d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1496412 Commit-Queue: Jia Meng <jiameng@chromium.org> Reviewed-by:
Andrew Moylan <amoylan@chromium.org> Cr-Commit-Position: refs/heads/master@{#638029}
-
Gayane Petrosyan authored
Bug: 936830,931806 Change-Id: I056956df2c7831b15f84cda5969a42ba583fe319 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1501493Reviewed-by:
Ramya Nagarajan <ramyan@chromium.org> Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Cr-Commit-Position: refs/heads/master@{#638028}
-
Gayane Petrosyan authored
Add default NTP font and make it platform specific by setting font on the document.body. Elements can further overwrite it if needed. Bug: 867759 Change-Id: Ia9332ce0f768474ba33e7b7948f7b4596d2e3271 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1490450 Commit-Queue: Gayane Petrosyan <gayane@chromium.org> Reviewed-by:
Kristi Park <kristipark@chromium.org> Reviewed-by:
Kyle Milka <kmilka@chromium.org> Cr-Commit-Position: refs/heads/master@{#638027}
-
Tao Bai authored
This is for QA to test the feature. Bug: 924681 Change-Id: Ia9856b4970197fc9da4ae04fbb83528d5482247d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1498382 Commit-Queue: Tao Bai <michaelbai@chromium.org> Reviewed-by:
Changwan Ryu <changwan@chromium.org> Reviewed-by:
Martin Šrámek <msramek@chromium.org> Cr-Commit-Position: refs/heads/master@{#638026}
-
Jinsuk Kim authored
This CL extends the present trusted_cdn.* to be a provider of the CDN publisher url for Java layer. Bug: 925242 Change-Id: I232b427cbc1a5ba66d6dcc6c9f0c64b2f314892d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1493335 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#638025}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/ad22645bad55..eb748c338927 git log ad22645bad55..eb748c338927 --date=short --no-merges --format='%ad %ae %s' 2019-03-06 tikuta@google.com Merge "proto: deserialize build dependency" Created with: gclient setdep -r src/third_party/perfetto@eb748c338927 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-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=perfetto-bugs@google.com Change-Id: I664ba2714ac4d0044ddc008e6d5720bb5af8bb8f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1505094Reviewed-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@{#638024}
-
Yuki Shiino authored
Animate no longer needs to take a ScriptState, v8::Isolate is enough for its implementation. Change-Id: I8fed12dea5aa88309d6b8f26ace55ac4949b7717 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503272Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#638023}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/f5f6e3c40e44..dbdb7ae5c8ed Created with: gclient setdep -r src-internal@dbdb7ae5c8ed 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. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome BUG=chromium:790708,chromium:790672 TBR=shend@chromium.org,zmin@chromium.org,dvadym@chromium.org,petewil@chromium.org Change-Id: I6f15f292fb2441a68aba25aafb6c51c29696c8b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504976Reviewed-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@{#638022}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/9390154c557e..cc4a756d48e9 git log 9390154c557e..cc4a756d48e9 --date=short --no-merges --format='%ad %ae %s' 2019-03-06 cepheus@frii.com Build: Update to latest SPIR-V header. Created with: gclient setdep -r src/third_party/glslang/src@cc4a756d48e9 The AutoRoll server is located here: https://autoroll.skia.org/r/glslang-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: I262f1d1d32ae93ca5bc35be9bd2485d7600c97fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504974Reviewed-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@{#638021}
-
Wenzhao Zang authored
Bug: 894270 Change-Id: Ibc35d9986fad4ec326806455389899959d280806 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1497539Reviewed-by:
Michael Giuffrida <michaelpg@chromium.org> Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org> Cr-Commit-Position: refs/heads/master@{#638020}
-
Darren Shen authored
TBR=flackr@chromium.org NOTRY=true Bug: 938780 Change-Id: I7cdcb81b8801cfafd19a1e9a56616ba0b85e5ace Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504480Reviewed-by:
Darren Shen <shend@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#638019}
-
Ian Kilpatrick authored
Previously the -webkit-line-clamp implementation didn't use the bottom edge of the line-box, instead using the bottom of any content inside it. While simple in the old layout engine, this adds complexity for the Firefox compat implementation, as well as an implementation on LayoutNG. Additionally Google front-end engineers are requesting we try and fix this issue. (see bug). The added test relies on the implementation of text-overflow to behave the same. Bug: 937931 Change-Id: I3cac20dc79e2487da59cd28ac453f900d4898f21 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1495439 Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#638018}
-
Christopher Thompson authored
The flag still had the old "HTTPBadPhase3" trial name, which was causing a DCHECK failure due to the conflict between the old field trial name and the new "HTTPReallyBadFinal" field trial name. Change-Id: I563ec74b1005aa360168d3ee918fdd3be5252d56 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1500024Reviewed-by:
Emily Stark <estark@chromium.org> Commit-Queue: Christopher Thompson <cthomp@chromium.org> Cr-Commit-Position: refs/heads/master@{#638017}
-
Tim Song authored
Currently, when the uninstall dialog is opened from the Launcher, the Launcher is dismissed, which is confusing to the user. By parenting the dialog to the Launcher, the UX is much better. Because we cannot directly access Ash windows anymore, this fix requires us to explicitly set the dialog's parent window via MUS properties. TEST=manually verified + new browser tests BUG=814619 Change-Id: Ic25cb376191aaba8048f95fa5d0032b0328b8bc8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1496447Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Tim Song <tengs@chromium.org> Cr-Commit-Position: refs/heads/master@{#638016}
-
Yue Li authored
If activity control consent has not been accepted, launch the Assistant opt-in flow instead of voice enrollment flow. Bug: None Test: Manual Test Change-Id: I3960715e17422b11c073942575c96557c2747112 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1500849Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Commit-Queue: Yue Li <updowndota@chromium.org> Cr-Commit-Position: refs/heads/master@{#638015}
-
Pin-chih Lin authored
Some devices could not reach 30 FPS for 2160p (4K, UHD) h264/vp8 encoding. Since we want to gather FPS performance data from devices which have 4k encoding capability, we disable the check of FPS larger than kMinPerfFPS, and just a warning log instead. BUG=chromium:927277 TEST=video_encode_accelerator_unittest on Eve and Caroline Change-Id: I3ff3fac117a6423d5e5949e49920ec6f3428410e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503178 Commit-Queue: Pin-chih Lin <johnylin@chromium.org> Auto-Submit: Pin-chih Lin <johnylin@chromium.org> Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#638014}
-
Alex Danilo authored
Quickview used an icon for the 'Open' action which is different to the 'Open' action in the main File Manager view. This CL changes the icon to the internationalized label for consistency. Bug: 762628 Change-Id: I059014dcead85028227b10a86ff75e2289d8046d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504596Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Alex Danilo <adanilo@chromium.org> Cr-Commit-Position: refs/heads/master@{#638013}
-
Darren Shen authored
Disabling: QuartcEndpointTest.ServerAcceptsOldVersion QuartcEndpointTest.ServerNegotiatesForOldVersion Culprit CL could not be reverted due to merge conflict, so disabling tests instead. TBR=vasilvv@chromium.org,rch@chromium.org NOTRY=true Bug: 938773 Change-Id: Id0c78e8d88519de1cfe62e2e5822e5b7db236f9b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504598Reviewed-by:
Darren Shen <shend@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#638012}
-
Alex Danilo authored
Selecting a file and then dragging the splitter to the left as far as possible causes the 'Cancel Selection' text to be cropped and the end replaced by an ellipsis. This change adds a tooltip so that hovering over the button will show it. Bug: 814273 Tests: Added test for this tooltip. Change-Id: I22465b933f562261c661add50bf336cf244fab81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1502256Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Alex Danilo <adanilo@chromium.org> Cr-Commit-Position: refs/heads/master@{#638011}
-
Gyuyoung Kim authored
Apply new review comments to the original CL more. This CL makes to use alternative ones instead of LocalFrame in the first argument of UseCounter::Count. Bug: 936187 Change-Id: I432a68708af7280c997fb5e2d61a7774eb8d2a99 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1498735 Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#638010}
-
Gyuyoung Kim authored
As a step to annotate the macros of Allocator for the Onion Soup, this CL annotates some classes of //third_party/blink/renderer/platform/network with the macros of Allocator. After some analyzes, it was concluded that for the case of ServerTimingHeader, EncodedFormData classes, they make sense to use USING_FAST_MALLOC because they have been used as a member variable with new() or smart pointers. Lastly, the rest of classes this CL annotates make sense to use STACK_ALLOCATED. Bug: 919389 Change-Id: I01489ed3ccc1f4e2b61850d3ebfc930431c59290 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504476Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com> Cr-Commit-Position: refs/heads/master@{#638009}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/bb71e01e26d0..f5f6e3c40e44 Created with: gclient setdep -r src-internal@f5f6e3c40e44 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. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=shend@chromium.org,zmin@chromium.org,dvadym@chromium.org,petewil@chromium.org Change-Id: I01022ddbfc8972d3f7353f6760b0a8b12a03bdbd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504973Reviewed-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@{#638008}
-
Kent Tamura authored
Add counters to investigate - if 'textarea' and 'searchfield-cancel-button' can be unexposed - if 'textfield' can be limit to textfield input types and temporal input types. Also, improve Menulist*ForOthersRendered counters. They should exclude input[type=color][list] and temporal input types on Android. Bug: 892586 Change-Id: Ic209f1d64be21a28f0f059ec8928e733368c8fa5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503173Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#638007}
-
Abhijeet Kandalkar authored
The primary motivation is that the function argument is not following the snake_case naming convention. As per current implementation function argument names in *probes_inl.h follow |paramFoo| naming convention. This CL modify build_param_name() in make_instrumenting_probes.py to rename function argument names to adopt |param_foo| naming convention. Bug: 936684 Change-Id: Idb63f1e6b78c98bb1830e74165b6cc69a8515a94 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1503179Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Abhijeet Kandalkar <abhijeet@igalia.com> Cr-Commit-Position: refs/heads/master@{#638006}
-
Alexey Baskakov authored
BookmarkAppHelper supports concurrent installs: many instances of BookmarkAppHelper can be created at call sites. Example: You can open 3 browser windows, initiate an install in each of them. And you see 3 modal dialogs to accept/dismiss. Bug: 915043 Change-Id: Ib7c1422bf341a43f4b45a82aa3204eb29c572369 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1498795Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Alexey Baskakov <loyso@chromium.org> Cr-Commit-Position: refs/heads/master@{#638005}
-
Yann Dago authored
In the chrome://management page, the "Safe browsing" text that appears in the browser reporting section is now a link to the safe browsing page. Bug: 936558 Change-Id: I88cc056e19b1cde29ae62cbd7e450cfb5fc8c6a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1492731 Commit-Queue: Yann Dago <ydago@chromium.org> Reviewed-by:
Varun Khaneja <vakh@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Thiemo Nagel <tnagel@chromium.org> Cr-Commit-Position: refs/heads/master@{#638004}
-
chromium-autoroll authored
The AutoRoll server is located here: https://autoroll.skia.org/r/fuchsia-sdk-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. CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: I142d4e8a8d2a79e9236b0d3aaca73675b8be0b2e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1504843Reviewed-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@{#638003}
-
Christopher Lam authored
This CL fixes permission toggling which had an erroneous assert, and adds a regression test. Bug: 906508 Change-Id: Ie026ac246c48b19290da5784199c6a8d75c71e89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1502259 Auto-Submit: calamity <calamity@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#638002}
-