- 09 Nov, 2017 40 commits
-
-
Koji Ishii authored
This patch handles margins in line height quirks[1] as defined in the spec. Edge and Gecko don't do this, but Blink and WebKit do, and there are wpt tests. [1] https://quirks.spec.whatwg.org/#the-line-height-calculation-quirk Bug: 636993 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I38c5b22075cd5bd24dba46b9491e3b0f7fe93abf Reviewed-on: https://chromium-review.googlesource.com/757884Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#515241}
-
Dmitry Skiba authored
Child process is blocked until it receives the invitation, but sending it requires a hop to the client thread. Renderers are started from the UI thread on Android, and the UI thread can be busy with other tasks, delaying renderer initialization. This CL separates sending Mojo invitation from other child process notification tasks, and sends the invitation from the launcher thread. Bug: 768253 Change-Id: Id0c7397b239241b1693e2f9c108c7caba6d910c5 Reviewed-on: https://chromium-review.googlesource.com/687821 Commit-Queue: Dmitry Skiba <dskiba@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#515240}
-
webrtc-autoroll@chromium.org authored
https://webrtc.googlesource.com/src.git/+log/8c34a824b458..675513b96a7a $ git log 8c34a824b..675513b96 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc BUG=None The AutoRoll server is located here: https://webrtc-chromium-roll.skia.org 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=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I5fb5a8fe30df4ba4328ba17bc14da8cdd10d7379 Reviewed-on: https://chromium-review.googlesource.com/760397Reviewed-by:
WebRTC Roll Bot <webrtc-autoroll@chromium.org> Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#515239}
-
skia-deps-roller@chromium.org authored
https://skia.googlesource.com/skia.git/+log/11691efe9d7d..0d8766c84c80 $ git log 11691efe9..0d8766c84 --date=short --no-merges --format='%ad %ae %s' 2017-11-09 reed Remove MakeForLocalSpace since picture image is sufficient 2017-11-09 herb Remove dead code for the incorrect handling of small radii blurs. 2017-11-08 csmartdalton Harden SkClassifyCubic 2017-11-09 brianosman Allow changing the transfer function in xform canvas mode 2017-11-09 angle-deps-roller Roll skia/third_party/externals/angle2/ 3ef06a9f8..70b715c9b (1 commit) 2017-11-09 liyuqian Compute more accurate containedInClip 2017-11-08 fmalita Simplify analytical GPU gradient impls 2017-11-09 angle-deps-roller Roll skia/third_party/externals/angle2/ 0e8831341..3ef06a9f8 (1 commit) 2017-11-08 robertphillips Remove destination pointer from GrCopySurfaceOp Created with: roll-dep src/third_party/skia BUG=743617 The AutoRoll server is located here: https://autoroll.skia.org 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=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel TBR=egdaniel@chromium.org Change-Id: I76f4dce6fd78c1df531cef14d4a510a937e63779 Reviewed-on: https://chromium-review.googlesource.com/760665Reviewed-by:
Skia Deps Roller <skia-deps-roller@chromium.org> Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#515238}
-
Matt Mueller authored
Bug: 513534, 782487 Change-Id: I56befe3da12959689762b0be60df0c186d5eaa14 Reviewed-on: https://chromium-review.googlesource.com/619659Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Matt Mueller <mattm@chromium.org> Cr-Commit-Position: refs/heads/master@{#515237}
-
Eric Karl authored
This reverts commit a28eb737. Further debugging indicates that this fix isn't addressing the original issues, and that the WebGL issues likely have a different root cause. TBR=kbr,piman,junov Bug: 773705, 778770 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I2eba58f1c8e87773fa6ed51e0273adbe24e33174 Reviewed-on: https://chromium-review.googlesource.com/758964 Commit-Queue: Eric Karl <ericrk@chromium.org> Reviewed-by:
Justin Novosad <junov@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#515236}
-
Lei Zhang authored
BUG=375330 Change-Id: If2cc9bf7457d94a52b8b7b3ae5a5c1e1b7a38733 Reviewed-on: https://chromium-review.googlesource.com/758884Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#515235}
-
liberato@chromium.org authored
Previously, DialogOverlayCore didn't receive a release() call from DSImpl, since DSImpl drops the reference to DSCore immediately after notifying the client of surface destruction. This caused problems on some cast devices, where the overlay wasn't being deleted properly. Now, DSCore calls release() synchronously after waiting for DSImpl to close the overlay. This keeps the dismiss() call sync with respect to onSurfaceDestroyed. Even if DSImpl did call DSCore.release(), it would be after onSurfaceDestroyed completed. Bug: 782282 Change-Id: I25f98643063545e0a44ccb7ab04145e7c6728d3a Reviewed-on: https://chromium-review.googlesource.com/756932 Commit-Queue: Frank Liberato <liberato@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#515234}
-
Xing Liu authored
This CL converts a few more tests to use embedded test server. Most of them are not listed in the mojo filter but also not working correctly. Change-Id: I02a9e0962a5b3efc61a3fbc05eb9df618c4fc1a7 Bug: 779121, 715630 Change-Id: I02a9e0962a5b3efc61a3fbc05eb9df618c4fc1a7 Reviewed-on: https://chromium-review.googlesource.com/755839 Commit-Queue: Xing Liu <xingliu@chromium.org> Reviewed-by:
Min Qin <qinmin@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#515233}
-
Ivan Sandrk authored
Design doc @ go/locked-fullscreen-mode Bug: chromium:755696 Change-Id: I78b774a40993b5672f24a2e1ae2f8a8aaa4eed8a Reviewed-on: https://chromium-review.googlesource.com/723422 Commit-Queue: Ivan Šandrk <isandrk@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#515232}
-
Qiang Xu authored
changes: Crash happens when doing profile switching, we reset SearchController but didn't reset AppListModel's search results. When switching back, search controller is accessing stale search results' stale pointers, which causes the crash. Bug: 781499 Test: test that crash doesn't happen any more Change-Id: Id020581d2c94fd8e14637ec25b8bd1ab266fecde Reviewed-on: https://chromium-review.googlesource.com/758520Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Vadim Tryshev <vadimt@chromium.org> Commit-Queue: Qiang(Joe) Xu <warx@chromium.org> Cr-Commit-Position: refs/heads/master@{#515231}
-
Vadim Tryshev authored
Change-Id: Ia56ff060ef0f478d5ce577290cd0322c2ae9aed8 Reviewed-on: https://chromium-review.googlesource.com/759376Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Vadim Tryshev <vadimt@chromium.org> Cr-Commit-Position: refs/heads/master@{#515230}
-
Tarun Bansal authored
Currently, the non-persistent client hints may be sent on non-secure URLs. This CL changes it so that client hints (persistent or non-persistent) are sent only on secure or local origins. Bug: 782381 Change-Id: I50a184d7aa19813eacc59e4d9fca5e74ec8855f2 Reviewed-on: https://chromium-review.googlesource.com/758464 Commit-Queue: Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#515229}
-
Kevin Marshall authored
These tests take a very long time to execute on emulated ARM cores, but they do pass on real ARM hardware. Bug: 780652 Change-Id: Iaa608ab980dd63616b9c44b4e5b386340af2664b Reviewed-on: https://chromium-review.googlesource.com/750076 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Scott Graham <scottmg@chromium.org> Cr-Commit-Position: refs/heads/master@{#515228}
-
Doug Arnett authored
Bug: 782864 Change-Id: I445d55f1a17b43b95850aaaf4a5e45237ae87d38 Reviewed-on: https://chromium-review.googlesource.com/759404 Commit-Queue: Doug Arnett <dougarnett@chromium.org> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Cr-Commit-Position: refs/heads/master@{#515227}
-
Steven Bennetts authored
TBR=chrome-os-gardeners@google.com Change-Id: I99afde99885e3fbefc0295a6172cec0df53aae49 Reviewed-on: https://chromium-review.googlesource.com/761076Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#515226}
-
Erik Luo authored
Add some margin so the watch expression prompt does not go off screen. Today, text can overflow outside of the prompt, so we should hide overflow. Screenshot: https://imgur.com/a/pQfSV Bug: 639284 Change-Id: Ib23fc2561ce6895164850474788279e9889fe53d Reviewed-on: https://chromium-review.googlesource.com/759426Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Erik Luo <luoe@chromium.org> Cr-Commit-Position: refs/heads/master@{#515225}
-
Robert Phillips authored
Change-Id: I5ac1891d1ceb8b3809b607e6f81b016ed72e912e Reviewed-on: https://chromium-review.googlesource.com/760639Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Robert Phillips <robertphillips@google.com> Cr-Commit-Position: refs/heads/master@{#515224}
-
Adrienne Walker authored
This tests most of the inputs to the oop raster extension so that we can verify that the GLES2Implementation behaves the same as RasterSource. Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel Change-Id: I4592627423ec543232263401bb9d86e1d4fcf934 Reviewed-on: https://chromium-review.googlesource.com/758489Reviewed-by:
Antoine Labour <piman@chromium.org> Reviewed-by:
vmpstr <vmpstr@chromium.org> Commit-Queue: enne <enne@chromium.org> Cr-Commit-Position: refs/heads/master@{#515223}
-
Alex Newcomer authored
Replace home-baked ScopedRTL and other Scoped RTL objects with the more commonly used version. Modify ctor of ScopedRestoreICUDefaultLocale to optionally accept a locale code to set. Bug: 777997 Change-Id: I23bebd574b008b1f7e8d16f2be5dda780d46ba63 Reviewed-on: https://chromium-review.googlesource.com/742442Reviewed-by:
danakj <danakj@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Commit-Queue: Alex Newcomer <newcomer@chromium.org> Cr-Commit-Position: refs/heads/master@{#515222}
-
spqchan authored
In OSX 10.10+, some combination of the titlebar and toolbar always show in fullscreen mode. We do not want either to show so we hide the window that contains these views. This way, only our custom toolbar will show on the screen. However, this cause an issue where the titlebar disappears for windows without our custom toolbar (Ex/ Chrome app windows and popups). This CL adds a check so that that we only hide the window if it has a toolbar. Bug: 601038 Change-Id: Ia125273a534d69b4ad406dc2d5329d31f2d97602 Reviewed-on: https://chromium-review.googlesource.com/758500 Commit-Queue: Sarah Chan <spqchan@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#515221}
-
Tim Volodine authored
The android targets are not compiled on windows, therefore the "is_win" related bits can be removed. (as per original comment on https://chromium-review.googlesource.com/c/chromium/src/+/738240/1) BUG=668515 Change-Id: Ibf16b209dc6fac1f83790c2abf40b3add43f5db4 Reviewed-on: https://chromium-review.googlesource.com/753344 Commit-Queue: Tim Volodine <timvolodine@chromium.org> Reviewed-by:
Tim Volodine <timvolodine@chromium.org> Reviewed-by:
Nathan Parker <nparker@chromium.org> Cr-Commit-Position: refs/heads/master@{#515220}
-
Kurt Horimoto authored
Bug: 782399 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Icafc8d57aaa229acb3d66628378aa49146d9b8cd Reviewed-on: https://chromium-review.googlesource.com/758083Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#515219}
-
Sophie Chang authored
This validates the hints protobuf file and sends a notification when the hints have been validated and processed. Bug: 777892 Change-Id: I72ca225a0d327f11849f0cd3f3198fe6c2abb06b Reviewed-on: https://chromium-review.googlesource.com/753290 Commit-Queue: Sophie Chang <sophiechang@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Sorin Jianu <sorin@chromium.org> Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Reviewed-by:
Doug Arnett <dougarnett@chromium.org> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Cr-Commit-Position: refs/heads/master@{#515218}
-
Michael van Ouwerkerk authored
Bug: 720383 Change-Id: Ibfa33fcf247473361e8f7b2496eff480163ffd80 Reviewed-on: https://chromium-review.googlesource.com/758582 Commit-Queue: Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Cr-Commit-Position: refs/heads/master@{#515217}
-
Gauthier Ambard authored
This tests is making sure the tests are able to run with ContentSuggestions enabled. Bug: 764720 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ia7ea99614f6020841273733ae4ea126e3d84155a Reviewed-on: https://chromium-review.googlesource.com/758648 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#515216}
-
gogerald authored
Bug: 781283 Change-Id: I273fc466d0af9115193972f3d862fb1bcbb6e539 Reviewed-on: https://chromium-review.googlesource.com/757090 Commit-Queue: Ganggui Tang <gogerald@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#515215}
-
Menglu Huang authored
Add EarlGrey tests tryjobs to ios changes. We want to have EG tests running for at least ios changes to reduce tree breakages. Since the ios-simulator-full-configs has only 8 builders, iOS CLs might experience extra landing time due to the tryjob size and limited capacity. Bug: 782735 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I8c00dbe1963893af19c37d7410e6d57c8d37ab5d Reviewed-on: https://chromium-review.googlesource.com/759166Reviewed-by:
Rohit Rao (ping after 24h) <rohitrao@chromium.org> Commit-Queue: Menglu Huang <huangml@chromium.org> Cr-Commit-Position: refs/heads/master@{#515214}
-
Michael Giuffrida authored
v1 packaged apps use the app.launch.local_path key. This was incorrectly changed when the doc was expanded. Change-Id: Iebcb59acf3e878364b2ed46f9554a7f0424e6f22 Reviewed-on: https://chromium-review.googlesource.com/728410Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Michael Giuffrida <michaelpg@chromium.org> Cr-Commit-Position: refs/heads/master@{#515213}
-
Troy Hildebrandt authored
Currently, we don't check for the 3 hours to pass before opening the bottom sheet to half if opened on the tab switcher. This adds that check. Bug: 781052 Change-Id: If680a1ed06d5e18161fdc8df60892592a6a55ddb Reviewed-on: https://chromium-review.googlesource.com/758825Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Troy Hildebrandt <thildebr@chromium.org> Cr-Commit-Position: refs/heads/master@{#515212}
-
Owen Min authored
This is the revert of crrev.com/b77864ad Turns out that fix causes crbug.com/775526. In the mean time, revert won't re-trigger the crbug.com/759007 as it's fixed by crrev.com/9990e87e. Bug: 759007 Bug: 775526 Change-Id: I2cd5acefb3d8588613629a99dde592f648650a93 Reviewed-on: https://chromium-review.googlesource.com/757718Reviewed-by:
anthonyvd <anthonyvd@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#515211}
-
Yixin Wang authored
Also fixes the bugs that were uncovered from adding these checks. Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Iecac97f4b91c24bc3aeae4b2fc336e39c25df781 Reviewed-on: https://chromium-review.googlesource.com/759690Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Yixin Wang <wangyix@chromium.org> Cr-Commit-Position: refs/heads/master@{#515210}
-
Sylvain Defresne authored
Those tests calls UIView -frame on a background thread which is an error on iOS 11.0+ causing the tests to fail. Bug: 782760 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-eg Change-Id: Iadb1e5d917724dbc6d1341d4de3dcbe2fcf7f3cd Reviewed-on: https://chromium-review.googlesource.com/758867Reviewed-by:
Elodie Banel <lod@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#515209}
-
Sylvain Defresne authored
Those tests calls UIView -frame on a background thread which is an error on iOS 11.0+ causing the tests to fail. Bug: 782760 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-eg Change-Id: I83d4338263bde003000be22ff38ef7199a35da64 Reviewed-on: https://chromium-review.googlesource.com/760361 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Elodie Banel <lod@chromium.org> Cr-Commit-Position: refs/heads/master@{#515208}
-
liberato@chromium.org authored
AVDA used to receive a "frame hidden" signal from WMPI. This would let it give up the overlay in case some other frame needs it. Now that DialogOverlayImpl destroys the overlay when the WebContents is hidden / etc., this isn't needed. It will be handled normally. Bug: 782827 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I9b5857f3b37545da2c44462de0424b00de8cb3ae Reviewed-on: https://chromium-review.googlesource.com/759143Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Commit-Queue: Frank Liberato <liberato@chromium.org> Cr-Commit-Position: refs/heads/master@{#515207}
-
Lei Zhang authored
For net::NetworkQualitiesPrefsManager. Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Ie4fddd3363d078dc468d3df61aac1d6336c8ac49 Reviewed-on: https://chromium-review.googlesource.com/760216Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#515206}
-
Hayato Ito authored
"Distribution" is likely a word which implies "an old mechanism without IncrementalShadowDOM". SupportsAssignment would be better here. Bug: 776656 Change-Id: Id1cfbbb9df054b7f0a47352c15aaf0bd57203792 Reviewed-on: https://chromium-review.googlesource.com/760117 Commit-Queue: Hayato Ito <hayato@chromium.org> Reviewed-by:
Takayoshi Kochi <kochi@chromium.org> Cr-Commit-Position: refs/heads/master@{#515205}
-
chrome://gpuMiguel Casas-Sanchez authored
This CL adds some extra info re. support for encrypted streams to the chrome://gpu video acceleration entry, to avoid the duplication of formats described in the bug. Bug: 783100 Change-Id: I06bf1bd450e71cdf2db88ec27f4ffff98814accb Reviewed-on: https://chromium-review.googlesource.com/760478Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#515204}
-
Aleks Totic authored
Promotes ResizeObserver to stable Intent to ship: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/z6ienONUb5A LGTM1 from dglazgov, LGTM2 from foolip, LGTM3 from chrishtr on Oct 17 Bug: 612962 Change-Id: I3dc59fd0ab725acc6f40bf5a3bbd70de5e889f0f Reviewed-on: https://chromium-review.googlesource.com/754562 Commit-Queue: Aleks Totic <atotic@chromium.org> Reviewed-by:
Yoav Weiss <yoav@yoav.ws> Cr-Commit-Position: refs/heads/master@{#515203}
-
Mikhail Fomitchev authored
We want UKM scroll latency reporting to work for --enable-viz (i.e. for the case when display compositor is in the GPU process). This means that we need to report these metrics in LatencyTracker::ReportUkmScrollLatency, not in RWHLatencyTrcker::ReportUkmScrollLatency. This CL adds ukm::SourceId to LatencyInfo. The source id is set on the LatencyInfo object in RWHLatencyTracker::OnInputEvent. This way LatencyTracker (which does not have access to RWHDelegate) can use the ukm::SourceId from LatencyInfo when reporting UKM scroll latency. TBR=sadrul@chromium.org Bug: 717629 Change-Id: Ib449bcaaf6153b67d2ffdf2f28bf4749f78c80fa Reviewed-on: https://chromium-review.googlesource.com/744702 Commit-Queue: Mikhail Fomitchev <mfomitchev@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Reviewed-by:
Timothy Dresser <tdresser@chromium.org> Cr-Commit-Position: refs/heads/master@{#515202}
-