Skip to content

GitLab

T
tangled
Switch branch/tag
  1. 18 Dec, 2017 32 commits
  3. 17 Dec, 2017 8 commits
    • Chromium WPT Sync's avatar
      Import wpt@4102aee71ff90177a4118605a5c55eef2e5c86ef · 63d294e5
      Chromium WPT Sync authored 
      Using wpt-import in Chromium ba99d192.

Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/8068

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:
drott@chromium.org, kojii@chromium.org:
  external/wpt/css/css-fonts

TBR=foolip

No-Export: true
Change-Id: Iac278f92a962d13def4e8b8c00e2fb0bf1265c8b
Reviewed-on: https://chromium-review.googlesource.com/831268
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: default avatarBlink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524632}
      63d294e5
    • Piotr Swigon's avatar
      Removes piotrs@ from WATCHLIST. · e1f9263c
      Piotr Swigon authored 
      Change-Id: Ib0a76cc5919b1e4242b75180c749a13cf81346a4
Reviewed-on: https://chromium-review.googlesource.com/831371Reviewed-by: default avatarMatt Giuca <mgiuca@chromium.org>
Reviewed-by: default avatarPiotr Swigon <piotrs@chromium.org>
Commit-Queue: Piotr Swigon <piotrs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524631}
      e1f9263c
    • Roman Sorokin's avatar
      Chromad: Use GetUserStatusRequest for GetUserStatus · 8a8503bb
      Roman Sorokin authored 
      It is an effort to make changing API easier.

BUG=chromium:785956

Change-Id: If65bb11804a6ff84ea960bda4178c5a48ddae98c
Reviewed-on: https://chromium-review.googlesource.com/829813Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Roman Sorokin <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524630}
      8a8503bb
    • Findit's avatar
      Revert "Display mirroring persistency changes" · ba99d192
      Findit authored 
      This reverts commit fa8dfaa6.

Reason for revert:

Findit (https://goo.gl/kROfz5) identified CL at revision 524627 as the
culprit for failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2ZhOGRmYWE2MjFjMTcwMWRjZTg0MWY3NmVmZmVjZDA2NGZmMzc1M2UM

Sample Failed Build: https://ci.chromium.org/buildbot/chromium.chromiumos/linux-chromeos-dbg/3252

Original change's description:
> Display mirroring persistency changes
> 
> Changes:
> 1. Change mirroring persistency rule to:
> If previous mirror mode is not set (e.g. no external display in previous
> configuration), turn on mirror mode if at least one of the external
> displays was in mirror mode before and turn off mirror mode otherwise.
> If previous mirror mode is set, mirror mode remains its previous value
> as long as there are more than 1 displays.
> For example:
> Case 1: Connect display A; Turn on mirror mode; Connect display B. B
> should be in mirror mode as well.
> Case 2: Connect display A; Turn on mirror mode; Remove A; Reconnect A; A
> should be in mirror mode.
> Case 3: Connect display A; Turn on mirror mode; Remove A; Connect
> display B. B should not be in mirror mode.
> Case 4: Connect display A; Turn on mirror mode; Connect display B;
> Remove B; Remove A; Reconnect B; B should be in mirror mode.
> 
> 2. Fix a bug in MirrorWindowController: mirroring source display may be
> changed in mirror mode (e.g. Connect two external displays; Turn on
> mirror mode; Close internal display lid; Then the internal display will
> be replaced with one external display as the mirroring source.),
> reflector and mirror windows should be updated in this case.
> 
> 3. Fix broken test.
> 
> Bug: 785416,792207
> Test: MultiMirroringTest.*
> Change-Id: Ia1da9f25df2bcc04f98ab087478a14685fff7708
> Reviewed-on: https://chromium-review.googlesource.com/804647
> Commit-Queue: Weidong Guo <weidongg@chromium.org>
> Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
> Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#524627}

Change-Id: Ie18b8dc47d031984590ed98af331fdb659b17941
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 785416,792207
Reviewed-on: https://chromium-review.googlesource.com/831290
Cr-Commit-Position: refs/heads/master@{#524629}
      ba99d192
    • Morten Stenshorne's avatar
      [LayoutNG] Fragmentainer breaks between line boxes. · f410ff66
      Morten Stenshorne authored 
      Constraint spaces for inline nodes also need to support block
fragmentation, due to floats.

We need to allow a (0,0) inline break token if we break before the first
line. If there's no break token for the inline layout context, it means
that we have finished all lines. Which is quite the opposite of being at
the very beginning.

There are some new test failures. One source is that we don't yet
support column-span:all, and now that we break between lines, what used
to pass by accident may no longer pass.

Making floats in an inline formatting context play along was tricky, and
I'm pretty sure that the solution I came up with needs to be improved.
For now, skip floats associated with the line when we resume layout at.
Floats that got broken inside will create break token children of the
token created for the containing block node, and will be break token
siblings of the inline break token to resume at (although they are not
siblings, node-wise).

The legacy layout write-back code doesn't work too great, especially
with LayoutNGPaintFragments enabled (but there are problems if that flag
is disabled too).

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I01634daf75a7d80a663ec24991eec8b4094aeeda
Reviewed-on: https://chromium-review.googlesource.com/829376
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Reviewed-by: default avatarEmil A Eklund <eae@chromium.org>
Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524628}
      f410ff66
    • Weidong Guo's avatar
      Display mirroring persistency changes · fa8dfaa6
      Weidong Guo authored 
      Changes:
1. Change mirroring persistency rule to:
If previous mirror mode is not set (e.g. no external display in previous
configuration), turn on mirror mode if at least one of the external
displays was in mirror mode before and turn off mirror mode otherwise.
If previous mirror mode is set, mirror mode remains its previous value
as long as there are more than 1 displays.
For example:
Case 1: Connect display A; Turn on mirror mode; Connect display B. B
should be in mirror mode as well.
Case 2: Connect display A; Turn on mirror mode; Remove A; Reconnect A; A
should be in mirror mode.
Case 3: Connect display A; Turn on mirror mode; Remove A; Connect
display B. B should not be in mirror mode.
Case 4: Connect display A; Turn on mirror mode; Connect display B;
Remove B; Remove A; Reconnect B; B should be in mirror mode.

2. Fix a bug in MirrorWindowController: mirroring source display may be
changed in mirror mode (e.g. Connect two external displays; Turn on
mirror mode; Close internal display lid; Then the internal display will
be replaced with one external display as the mirroring source.),
reflector and mirror windows should be updated in this case.

3. Fix broken test.

Bug: 785416,792207
Test: MultiMirroringTest.*
Change-Id: Ia1da9f25df2bcc04f98ab087478a14685fff7708
Reviewed-on: https://chromium-review.googlesource.com/804647
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Reviewed-by: default avatarMitsuru Oshima <oshima@chromium.org>
Reviewed-by: default avatarAhmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524627}
      fa8dfaa6
    • Rohit Rao's avatar
      [ios] Tests for nullptr before using _popupView. · 3e4254d9
      Rohit Rao authored 
      When the BVC presentation experiment is enabled and the last incognito tab is
closed, transition animations continue to run even after the incognito
BrowserState is destroyed.  Variables that are cleared in
|browserStateDestroyed| must be tested for nullptr before use.

BUG=787821

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ic5443690337d311e1ce3e347e563d3111cc06f5e
Reviewed-on: https://chromium-review.googlesource.com/831095
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Reviewed-by: default avatarJustin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524626}
      3e4254d9
    • Luke Sorenson's avatar
      Add Component version number to MPP API callback. · e9dca222
      Luke Sorenson authored 
      Bug: 794273
Change-Id: I340114ad83e2dcdd5fd2b8d0dea721d0705e53b3
Reviewed-on: https://chromium-review.googlesource.com/822820Reviewed-by: default avatarToni Barzic <tbarzic@chromium.org>
Commit-Queue: Luke Sorenson <lasoren@chromium.org>
Cr-Commit-Position: refs/heads/master@{#524625}
      e9dca222