- 01 Nov, 2018 33 commits
-
-
David Staessens authored
The TextureRef class will also be used by video encode tests, so it doesn't make sense to keep it in a decode-specific helper file. No code was added or changed, this CL is only moving the class to a separate file. TEST=ran VDA unittests on eve BUG=900433 Change-Id: Id104701554ea0bd1e12c10221346e946ff1b9c50 Reviewed-on: https://chromium-review.googlesource.com/c/1309381 Commit-Queue: David Staessens <dstaessens@chromium.org> Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Cr-Commit-Position: refs/heads/master@{#604487}
-
Oystein Eftevaag authored
This caused trace events for these threads to go missing as Perfetto was not able to recover them from the shared memory chunks. Also: * Removed an unused variable from TraceEventDataSink, * Read the same member out of a TraceEvent union as the one we write to (thread_id vs process_id). R=primiano@chromium.org Bug: 900669 Change-Id: I80d4e39b154b138a540770e227f1006fec0a91b2 Reviewed-on: https://chromium-review.googlesource.com/c/1311125Reviewed-by:
Primiano Tucci <primiano@chromium.org> Commit-Queue: oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#604486}
-
Joel Hockey authored
Bug: 860355 Change-Id: I73b7eb298cf462c5e981f0cfb3a0132499b0083c Reviewed-on: https://chromium-review.googlesource.com/c/1312134Reviewed-by:
Stuart Langley <slangley@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#604485}
-
Paul Wankadia authored
https://chromium.googlesource.com/external/github.com/google/re2.git/+log/46155299edb9..9a227bed7a0f $ git log 46155299e..9a227bed7 --date=short --no-merges --format='%ad %ae %s' 2018-10-31 junyer Make Prefilter::Info::Walker::PostVisit() handle kRegexpAnyByte too. Created with: roll-dep src/third_party/re2/src R=mmoroz@chromium.org Bug: 900671 Change-Id: Iad2dd959bdda48eeef6c5a4f6d7aa028c9c79538 Reviewed-on: https://chromium-review.googlesource.com/c/1311494Reviewed-by:
Max Moroz <mmoroz@chromium.org> Commit-Queue: Max Moroz <mmoroz@chromium.org> Cr-Commit-Position: refs/heads/master@{#604484}
-
Christian Biesinger authored
When a flex item has a definite specified height (e.g. height: 100px), then percentages in children of the flex item should resolve even if the flexbox does not have an explicit height, ie. does not match the condition in https://drafts.csswg.org/css-flexbox/#definite-sizes Bug: 885185 Change-Id: Iba226f30e1e02e3a11273fa45fcdf1cef897120c Reviewed-on: https://chromium-review.googlesource.com/c/1311534 Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#604483}
-
Livvie Lin authored
Metrics show that the Superfish interstitial shows up effectively 0% of the time, and we rarely receive Superfish certificate reports. This cl deletes the code for the Superfish interstitial. Bug: 899987 Change-Id: Ice48aad16fd06d74d1150c306fb5caf62c119022 Reviewed-on: https://chromium-review.googlesource.com/c/1306773 Commit-Queue: Livvie Lin <livvielin@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
Mustafa Emre Acer <meacer@chromium.org> Cr-Commit-Position: refs/heads/master@{#604482}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/e3f4b1f5ee13..f92910970a44 git log e3f4b1f5ee13..f92910970a44 --date=short --no-merges --format='%ad %ae %s' 2018-11-01 oprypin@chromium.org Basic Python3 compatibility for catapult tracing Created with: gclient setdep -r src/third_party/catapult@f92910970a44 The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-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_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=sullivan@chromium.org Change-Id: I9eb7a0a23536b9367332cd0942ce2f84a162e50b Reviewed-on: https://chromium-review.googlesource.com/c/1312054Reviewed-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@{#604481}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/06768fc14b7a..d7a7586b0d2f git log 06768fc14b7a..d7a7586b0d2f --date=short --no-merges --format='%ad %ae %s' 2018-10-31 caryclark@skia.org delete color4f pin from docs 2018-10-31 mtklein@google.com clean up some odd SkRefCnt features 2018-10-31 brianosman@google.com Remove SkRGBA4f pinning functions 2018-10-31 herb@google.com Move from a functional to object based interface between Painter and SkDraw 2018-10-31 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-10-31 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader e36ca90f33e9..adfbbcbed6e7 (3 commits) 2018-10-31 mtklein@google.com implement SkPixmap::erase() as a draw 2018-10-31 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 d60e42f8e79e..580baf379673 (3 commits) 2018-10-31 scroggo@google.com Treat start(Incremental/Scanline)Decode errors as failures 2018-10-31 mtklein@google.com try removing SkDraw::drawPaint() fast-path 2018-10-31 herb@google.com Refactor to create the blitter in the PaintAllMasks 2018-10-31 brianosman@google.com Replace GrColor4h with SkPMColor4f 2018-10-31 bsalomon@google.com Change mechanism that lets GrRenderTargetContext caller get op list ID. 2018-10-31 jvanverth@google.com More checks in ReflexHash setup Created with: gclient setdep -r src/third_party/skia@d7a7586b0d2f The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel TBR=herb@chromium.org Change-Id: I018e6ff8f1ffa84b43d2a9be6da22ee1b91ed018 Reviewed-on: https://chromium-review.googlesource.com/c/1311675Reviewed-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@{#604480}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/28fd7569985c..378f5ac017f8 Created with: gclient setdep -r src-internal@378f5ac017f8 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=mmoss@chromium.org Change-Id: I8d5e8365eb3339efc3f7be360dafb15961cb589f Reviewed-on: https://chromium-review.googlesource.com/c/1312053Reviewed-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@{#604479}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/580baf379673..2197dc529052 git log 580baf379673..2197dc529052 --date=short --no-merges --format='%ad %ae %s' 2018-10-31 syoussefi@chromium.org Vulkan: Implement masked depth & stencil clear Created with: gclient setdep -r src/third_party/angle@2197dc529052 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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:android_optional_gpu_tests_rel;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=ynovikov@chromium.org Change-Id: Icaa2803eed875a5686f901c9f0bfc841012df31d Reviewed-on: https://chromium-review.googlesource.com/c/1311836Reviewed-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@{#604478}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/062a691cae73..838643550f0c git log 062a691cae73..838643550f0c --date=short --no-merges --format='%ad %ae %s' 2018-10-31 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 6271fcdc..734e273d (604273:604373) 2018-10-31 mellem@webrtc.org Add interfaces for using MediaTransport as the transport for data channels. Created with: gclient setdep -r src/third_party/webrtc@838643550f0c The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-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:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng BUG=chromium:None TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I1dc52ef1f77af0bbd9c542586fdfbf2168ad2cc1 Reviewed-on: https://chromium-review.googlesource.com/c/1311993Reviewed-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@{#604477}
-
Kenneth Russell authored
Needed to generate new reference images. Tbr: mcasas@chromium.org Bug: 865957 Change-Id: I87e8f15550a7bddef25f2f7ce22d07184d13ea17 Reviewed-on: https://chromium-review.googlesource.com/c/1311842Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#604476}
-
Kenichi Ishibashi authored
Two reasons to remove these histograms: - We haven't been looking at these histograms. - We added milestone-style histograms for main/sub resource loading and some of these track event dispatching delays. Bug: 898020, 872567 Change-Id: I9f61da0101aa8e38ff4bca519126ef339b877863 Reviewed-on: https://chromium-review.googlesource.com/c/1307015Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#604475}
-
Brandon Wylie authored
A summary: - A bunch of work was being done on the UI thread. Now it's being done off-thread. - Network requests were dispatched all at once, slowing everything down. Now they're dispatched one at a time so the first thumbnail is loaded in quickly. - One weak ptr was being passed across threads, resulting in a crash. This is fixed now. Bug: 864634 Change-Id: I8dc3797a66c0ccc048b9d9613333df7450ec50c1 Reviewed-on: https://chromium-review.googlesource.com/c/1306634 Commit-Queue: Brandon Wylie <wylieb@chromium.org> Reviewed-by:
Filip Gorski <fgorski@chromium.org> Reviewed-by:
Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#604474}
-
Jun Mukai authored
The current behavior makes TransferEventsTo() before the window move starts, but don't care at its end. This causes errors like crbug.com/900363. But we can't simply transfer back to the original window unconditionally since someone (like TabDragController) may also want to continue dragging on a window other than the original source window. This CL introduces a new scoped class to control this; it makes TransferEventsTo first, and then invokes TransferEventsTo back again at the end, but it skips invoking the second transfer if someone else also transfers the touch events on the same window. BUG=900363 TEST=the new test case Change-Id: I81792872a13f0e4bb2c88b526c92cf689d656b71 Reviewed-on: https://chromium-review.googlesource.com/c/1310501 Commit-Queue: Jun Mukai <mukai@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#604473}
-
Aleks Totic authored
128 bits gone Bug: 835484 Change-Id: I50045dc72b3037edca348c94eecf2d00acf037d6 Reviewed-on: https://chromium-review.googlesource.com/c/1311805Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Aleks Totic <atotic@chromium.org> Cr-Commit-Position: refs/heads/master@{#604472}
-
Ted Choc authored
The suggestion view no longer has direct references to any of the backend suggestion data. BUG=898522 Change-Id: Iacb03cff130d659c845e134f593f87b8416c16ae Reviewed-on: https://chromium-review.googlesource.com/c/1306350Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#604471}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/22a617f93038..9d88af8f4ebe git log 22a617f93038..9d88af8f4ebe --date=short --no-merges --format='%ad %ae %s' 2018-10-31 thestig@chromium.org Roll third_party/skia/ 7258e97e8..c994a93b3 (302 commits; 17 trivial rolls) Created with: gclient setdep -r src/third_party/pdfium@9d88af8f4ebe The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: If66f68f380076047cdf00c2fd4b224e1b1ce4e27 Reviewed-on: https://chromium-review.googlesource.com/c/1311674Reviewed-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@{#604470}
-
Ned Nguyen authored
This is a pure refactoring change to make the code cleaner. There is no intended effects change. Bug: 893618 Change-Id: I68d59e804c637da07cefac7fcdcdef3c70626227 Reviewed-on: https://chromium-review.googlesource.com/c/1311801Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#604469}
-
Josh Gao authored
The official builds apparently doesn't print the error message from CHECK failure, so delete the death test regexes. While we're at it, switch from the ifdef to EXPECT_DEATH_IF_SUPPORTED. TBR=dcheng@chromium.org Bug: 900764 Change-Id: I61448b5fcb903b35a4cc01e0eb599ba35be1d402 Reviewed-on: https://chromium-review.googlesource.com/c/1311838Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Commit-Queue: Josh Gao <jmgao@google.com> Cr-Commit-Position: refs/heads/master@{#604468}
-
Amos Lim authored
Move: build/scripts/{ => core/css}/make_css_primitive_value_unit_trie.py build/scripts/{ => core/css}/templates/css_primitive_value_unit_trie.cc.tmpl This CL has no behavior changes. Bug: 732657 Change-Id: Iffd9b98f0eede7fc7349c89d26eb885de9953600 Reviewed-on: https://chromium-review.googlesource.com/c/1309377Reviewed-by:Kent Tamura <tkent@chromium.org> Commit-Queue: Amos Lim <eui-sang.lim@samsung.com> Cr-Commit-Position: refs/heads/master@{#604467}
-
Esmael El-Moslimany authored
Bug: 866926 Change-Id: I4811f8cafd4febc39955ea4c076b83fb7c0035a9 Reviewed-on: https://chromium-review.googlesource.com/c/1309094 Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#604466}
-
Joel Hockey authored
If 2 child dirs such as /a/a, and /a/b, are shared, then parent /a is shared, we no longer register both children as being shared and we only show the parent in settings page as being shared. If the parent is subsequently unshared, then children will also be unshared. Bug: 878324 Change-Id: I1c85d75ad934d48cd353299943f7de893ca62017 Reviewed-on: https://chromium-review.googlesource.com/c/1306945Reviewed-by:
Timothy Loh <timloh@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#604465}
-
Peter Boström authored
Removes Widget::SetFocusTraversableParent* calls since we don't want focus to escape bubble dialogs once they get here. There are several bubbles that close once focus escapes. We still set kAnchoredDialogKey of the anchor view to make sure that focus can move into attached dialogs (this makes them keyboard accessible). Bug: chromium:899601, chromium:899996 Change-Id: Idc5d60c2f41ae20a343324ae75b0fd304f273c94 Reviewed-on: https://chromium-review.googlesource.com/c/1311090Reviewed-by:
Trent Apted <tapted@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#604464}
-
Yuichiro Hanada authored
When active IME is changed in Android side, the event is sent to OnActiveImeChanged(). ArcInputMethodManagerService should switch an active IME in Chrome OS side according to the event. Bug: 900756 Test: Ran on DUT. Change-Id: Ia8002ec678775f6349020b90766423987a4f714b Reviewed-on: https://chromium-review.googlesource.com/c/1311853 Commit-Queue: Yuichiro Hanada <yhanada@chromium.org> Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Cr-Commit-Position: refs/heads/master@{#604463}
-
Tom Anderson authored
BUG=900182 R=thakis TBR=dschuff Change-Id: I73402d71c1deb98564ec19d07d0e19e340bb6473 Reviewed-on: https://chromium-review.googlesource.com/c/1308097Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#604462}
-
Sergey Berezin authored
This reverts commit b93f1513. Reason for revert: The outage is over, returning the bot to CQ. Original change's description: > [cq] EMERGENCY remove win10_chromium_x64_rel_ng from CQ > > The builder is experiencing a massive failure due to a swarming error. > Making it non-blocking for CQ while the fix is being worked on. > > TBR=tandrii@chromium.org > > Bug: 900742 > Change-Id: Ie6f95ec39062107b77fca6942992dbcb35557ec7 > Reviewed-on: https://chromium-review.googlesource.com/c/1311128 > Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#604384} TBR=sergeyberezin@chromium.org,tandrii@chromium.org,bpastene@chromium.org Change-Id: I56714ba00d32fbcc9c8d06d5d2dcf04b169b619f No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 900742 Reviewed-on: https://chromium-review.googlesource.com/c/1311811Reviewed-by:
Sergey Berezin <sergeyberezin@chromium.org> Commit-Queue: Sergey Berezin <sergeyberezin@chromium.org> Cr-Commit-Position: refs/heads/master@{#604461}
-
Eric Karl authored
GPU raster caused slight differences around antialiased div edges. TBR=kbr@chromium.org Bug: 900599 Change-Id: I2531fb642c50df820c5f4947d8670818217141c9 Reviewed-on: https://chromium-review.googlesource.com/c/1311835 Commit-Queue: Eric Karl <ericrk@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#604460}
-
Hitoshi Yoshida authored
Before this CL, it was difficult to handle IDLDictionaries with WTF containers and to support partial dictionary[1]. And to support such use cases, we have non standard hacks for them. e.g. HeapVector<IDLDictionary> This CL makes IDLDictionaryBase inherited from GarbageCollected<>, and hence we can remove strange restrictions and hacks, and it will be easy to support partial dictionaries. [1] https://heycam.github.io/webidl/#dfn-partial-dictionary TBR=peter, mkwst Bug: 714575, 579896 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ia999589e5329cdf863d54ccb1bcea385f607fb62 Reviewed-on: https://chromium-review.googlesource.com/c/1226613 Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Kenichi Ishibashi <bashi@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#604459}
-
Kent Tamura authored
https://google.github.io/styleguide/cppguide.html#Namespace_Names > Namespace names are all lower-case. This CL has no behavior changes. Bug: 889726 Change-Id: I77ac4cf7c882d3d55e7f4f3e1c6a445c49bc108f Reviewed-on: https://chromium-review.googlesource.com/c/1309895Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#604458}
-
David Grogan authored
Includes moving the stretched size computation from legacy to FlexItem so that NG can use it. A big hack is that stretched items don't use the height of their parent lines, they use the whole container height because we haven't implemented line size calculation yet. This works for the majority of cases. About 6 tests newly fail. Some multi-line flex containers now fail because of the above hack. But some tests with images that are flex items fail because NG doesn't copy their data to legacy because there are unplaced floats internal to the image (something related to alt text). I haven't looked into why this used to work but fails now. This patch nets roughly 110 layout test lines removed. Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: I0083e80797c1f01c8cc4c63ac696d68063f67ff2 Bug: 845235 Reviewed-on: https://chromium-review.googlesource.com/c/1285851Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: David Grogan <dgrogan@chromium.org> Cr-Commit-Position: refs/heads/master@{#604457}
-
Wez authored
Newly-added SharedMulticastAddress fails on Fuchsia, and the existing JoinMulticastGroup test now fails due to a change in the parameters passed to Listen() for non-Windows. Bug: 900709 Change-Id: Ia98ec45897f88696e7eb872938822040ab5940c6 Reviewed-on: https://chromium-review.googlesource.com/c/1311662Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#604456}
-
Joel Hockey authored
Bug: 886546 Change-Id: Ice466abf61544783407bac9601b23a01982c4055 Reviewed-on: https://chromium-review.googlesource.com/c/1309378Reviewed-by:
Timothy Loh <timloh@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#604455}
-
- 31 Oct, 2018 7 commits
-
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/454cfea9e32c..28fd7569985c Created with: gclient setdep -r src-internal@28fd7569985c 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. BUG=chromium:891313,chromium:891734 TBR=mmoss@chromium.org Change-Id: Ib6d640a1b75fbde186a44f606949125575bec101 Reviewed-on: https://chromium-review.googlesource.com/c/1311677Reviewed-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@{#604454}
-
dpapad authored
Bug: None Change-Id: Id54affe66c2ae4d31152461d32b2f97f39964579 Reviewed-on: https://chromium-review.googlesource.com/c/1310753Reviewed-by:
Scott Chen <scottchen@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#604453}
-
Alan Cutter authored
This CL updates a DCHECK to allow for the BrowserNonClientFrameView getting recreated when the browser window frame is updated via NonClientView::UpdateFrame(). This also fixes a UI bug where we don't initialise the content setting icons properly for PWA windows resulting in uninitialised state after toggling GTK themes. Before: https://bugs.chromium.org/p/chromium/issues/attachment?aid=365396&signed_aid=Z-gwLwzrT9I5krg3sUoY7Q==&inline=1 After (ignore Linux screen glitching): https://bugs.chromium.org/p/chromium/issues/attachment?aid=365397&signed_aid=e_kvzb_AGaLLi5zzKY_o7w==&inline=1 Bug: 878639 Change-Id: I57bd1a6f7f94687a0f95557914f34eda737129ee Reviewed-on: https://chromium-review.googlesource.com/c/1308960Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#604452}
-
Kyle Milka authored
Bug: None Change-Id: I2918bedb89b3f8682d7fa5d5fdfabab06f6d56bb Reviewed-on: https://chromium-review.googlesource.com/c/1311873 Commit-Queue: Kyle Milka <kmilka@chromium.org> Reviewed-by:
Ramya Nagarajan <ramyan@chromium.org> Cr-Commit-Position: refs/heads/master@{#604451}
-
Matt Menke authored
Previously, each the following was treated the same: Content-Length: 7 Content-Length: ,7 Content-Length: 7, Content-Length: , ,7, , And all condensed into a single header with a value of 7. There's nothing in any spec that instructs us to do this. Moreover, the spec explicitly states that the two lines: Content-Length: 7 Content-Length: Can be safely merged into: Content-Length: 7, Without changing meaning, but in the former case, Chrome treats two headers lines as if they were separate header lines, while in the latter, as mentioned, it ignores the existence of the comma. Also, the original header line could never be retrieved in this case, since the extra information was just silently purged. This CL treats both responses as having two separate Content-Length headers (And this applies to all other non-coalescing headers as well). This behavior seems to match that of all other browsers, so breakages should be minimal. Bug: 896233 Change-Id: I1acb26a3face0eef8d47c1dbe727aab5ec7dd7bd Reviewed-on: https://chromium-review.googlesource.com/c/1297251 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Reviewed-by:
Asanka Herath <asanka@chromium.org> Cr-Commit-Position: refs/heads/master@{#604450}
-
Scott Violet authored
This changes SyntheticGestureTargetAura to have an aura::EventInjector rather than constantly creating one. In mash, EventInjector sends events to the window service for dispatch. EventInjector establishes the connection as necessary, but as SyntheticGestureTargetAura continually recreates EventInjector the connection must be brough up (and destroyed) for each event. This is non-trivially and adds unnecessary latency to perf tests. BUG=none TEST=none Change-Id: I22cc6ffe37808586ccd8904b75637e14f2532aea Reviewed-on: https://chromium-review.googlesource.com/c/1311536Reviewed-by:
Jun Mukai <mukai@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#604449}
-
Peter Boström authored
Breaks out the use of GetColorWithMinimumContrast for bookmarks-bar items to a separate function and makes sure all callers use that instead of directly fetching COLOR_BOOKMARK_TEXT. In practice this fixes the two items themes that don't specify good contrasting colors for COLOR_BOOKMARK_TEXT. This also probably makes the folder icon prettier on Mac in the same case as it uses a more contrasting white. Bug: None Change-Id: Ief016b9bbd45991b2a357055c112704fc7f60097 Reviewed-on: https://chromium-review.googlesource.com/c/1311660Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#604448}
-