Home | Search | chromium - Builders
Login

Builder Mac FYI 10.14 Release (Intel) Build 464 Apple OSX

Results:

Success

Trigger Info:

Projectchromium
Revision62dd51a1163937c255844b681c661a78d387b4b8
Got Revision62dd51a1163937c255844b681c661a78d387b4b8

Execution:

Steps and Logfiles:

Show:
  1. ( 331 ms ) setup_build
    running recipe: "chromium"
  2. ( 213 ms ) report builders
     
    running tester 'Mac FYI 10.14 Release (Intel)' on master 'chromium.gpu.fyi'
  3. ( 1 mins 26 secs ) bot_update
    [67GB/232GB used (28%)]
  4. ( 2 secs ) swarming.py --version
    0.13
  5. ( 401 ms ) ensure_installed
  6. ( 16 secs ) gclient runhooks
  7. ( 1 secs ) read test spec (chromium.gpu.fyi.json)
    path: /b/s/w/ir/cache/builder/src/testing/buildbot/chromium.gpu.fyi.json
  8. ( 379 ms ) lookup builder GN args
     
    build_angle_gles1_conform_tests = true
    dcheck_always_on = true
    ffmpeg_branding = "Chrome"
    internal_gles2_conform_tests = true
    is_component_build = false
    is_debug = false
    proprietary_codecs = true
    strip_absolute_paths_from_debug_symbols = true
    symbol_level = 1
    use_goma = true
    goma_dir = "/b/s/w/ir/cache/goma/client"
  9. ( 1 secs ) build directory
  10. ( 384 ms ) find isolated tests
  11. ( 2 secs ) angle_end2end_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Max shard duration: 17s (shard #0)
    Min shard duration: 15s (shard #1)
  12. ( 3 secs ) angle_unittests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 0:01:01.174203
  13. ( 1 mins 7 secs ) tab_capture_end2end_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Pending time: 21s
    Shard duration: 43s
  14. ( 25 secs ) gl_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 22s
  15. ( 13 secs ) gl_unittests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 10s
  16. ( 1 mins 6 secs ) gles2_conform_test on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 0:01:03.930301
  17. ( 34 secs ) gpu_unittests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 26s
  18. ( 27 secs ) services_unittests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 20s
  19. ( 2 secs ) swiftshader_unittests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 8s
  20. ( 2 mins 2 secs ) context_lost_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 0:01:58.985848
    Total tests: 10
    * Passed: 10 (10 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  21. ( 34 secs ) depth_capture_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 31s
    Total tests: 3
    * Passed: 3 (3 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  22. ( 1 mins 5 secs ) gpu_process_launch_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 0:01:01.923527
    Total tests: 15
    * Passed: 15 (15 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  23. ( 31 secs ) hardware_accelerated_feature_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 28s
    Total tests: 2
    * Passed: 2 (2 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  24. ( 30 secs ) info_collection_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 27s
    Total tests: 1
    * Passed: 1 (1 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  25. ( 1 mins 14 secs ) maps_pixel_test on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 0:01:10.351655
    Total tests: 1
    * Passed: 1 (1 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  26. ( 4 mins 49 secs ) pixel_test on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 0:04:38.747012
    Total tests: 57
    * Passed: 54 (54 expected, 0 unexpected)
    * Skipped: 3 (3 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  27. ( 1 mins 3 secs ) screenshot_sync_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 0:01:01.816224
    Total tests: 4
    * Passed: 4 (4 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  28. ( 2 mins 25 secs ) trace_test on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Shard duration: 0:02:19.783935
    Total tests: 36
    * Passed: 18 (18 expected, 0 unexpected)
    * Skipped: 18 (18 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  29. ( 2 hrs 47 mins ) webgl2_conformance_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Max pending time: 2:38:17.930144 (shard #19)
    Max shard duration: 0:09:35.413899 (shard #3)
    Min shard duration: 0:07:34.706779 (shard #1)
    Total tests: 2796
    * Passed: 2731 (2731 expected, 0 unexpected)
    * Skipped: 65 (65 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  30. ( 10 mins 16 secs ) webgl_conformance_tests on Intel GPU on Mac on Mac-10.14
    Run on OS: 'Mac-10.14'
    Max pending time: 0:02:50.911925 (shard #1)
    Max shard duration: 0:07:23.648398 (shard #1)
    Min shard duration: 0:02:43.992443 (shard #0)
    Total tests: 932
    * Passed: 925 (925 expected, 0 unexpected)
    * Skipped: 7 (7 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  31. ( 278 ms ) Tests statistics
     
    Stats
    Total shards: 43
    Total runtime: 3:03:29.330036
     

Build Properties:

NameValueSource
$recipe_engine/path {"cache_dir": "/b/s/w/ir/cache", "temp_dir": "/b/s/w/ir/tmp/rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "build289-m9" setup_build
buildbucket {"build": {"bucket": "luci.chromium.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1541766819625561, "id": "8930334031303458048", "project": "chromium", "tags": ["builder:Mac FYI 10.14 Release (Intel)", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/62dd51a1163937c255844b681c661a78d387b4b8", "gitiles_ref:refs/heads/master", "parent_buildername:GPU FYI Mac Builder", "parent_buildnumber:27334", "scheduler_invocation_id:9095764093539650704", "scheduler_job_id:chromium/Mac FYI 10.14 Release (Intel)", "user_agent:luci-scheduler", "user_agent:recipe"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "Mac FYI 10.14 Release (Intel)" setup_build
buildnumber 464 setup_build
mastername "chromium.gpu.fyi" setup_build
parent_buildername "GPU FYI Mac Builder" setup_build
parent_got_angle_revision "5d2dfa46c05c8c661304922ce1e1e7d7b045a342" setup_build
parent_got_buildtools_revision "13a00f110ef910a25763346d6538b60f12845656" setup_build
parent_got_dawn_revision "284f33439871e1ae2556d3ada34b5d1b75a4fb20" setup_build
parent_got_nacl_revision "99673869a3cd8731d924bd32fa486feebfdc6c4f" setup_build
parent_got_revision "62dd51a1163937c255844b681c661a78d387b4b8" setup_build
parent_got_revision_cp "refs/heads/master@{#606803}" setup_build
parent_got_swarming_client_revision "f78187ab77127de42555afe0ad410bebde6ac6a5" setup_build
parent_got_v8_revision "6130e69e8e0758f2769a6172c79d32c87bc39464" setup_build
parent_got_v8_revision_cp "refs/heads/7.2.284@{#1}" setup_build
parent_got_webrtc_revision "e769ed90c3599a131166d5a61625c1b64318fb91" setup_build
parent_got_webrtc_revision_cp "refs/heads/master@{#25544}" setup_build
parent_mastername "chromium.gpu.fyi" setup_build
path_config "generic" setup_build
recipe "chromium" setup_build
revision "62dd51a1163937c255844b681c661a78d387b4b8" setup_build
swarm_hashes {"angle_end2end_tests": "fdac938759f6d2a1c8a19af8dea0ae553e096d2a", "angle_unittests": "b2be6e8b061241ca84a2688b463370ac41c1e47a", "browser_tests": "f56073500cf49e0b557f55cc999e602041431873", "gl_tests": "e6a1452a58dd4b4bf73926f0650e23de52f3a644", "gl_unittests": "50da2bf0659c42cc5796b96e475ecc02283e24bb", "gles2_conform_test": "cc660a63b62e69cb816305fca4b258bbc8b1627d", "gpu_unittests": "86726b0290d981ee2734dd0c4cfc79c37209b955", "services_unittests": "70cf40181cc09a42930e8296cf2743fb70cd51b9", "swiftshader_unittests": "798caf633b7a656df19c59f6cf426d5e9e7f8814", "telemetry_gpu_integration_test": "4343f02c4d8852308e8679cde0ac01e667f5a19c"} setup_build
got_angle_revision "5d2dfa46c05c8c661304922ce1e1e7d7b045a342" bot_update
got_buildtools_revision "13a00f110ef910a25763346d6538b60f12845656" bot_update
got_dawn_revision "2960ec33667a11bb4445dfe8154aef133c741afe" bot_update
got_nacl_revision "99673869a3cd8731d924bd32fa486feebfdc6c4f" bot_update
got_revision "62dd51a1163937c255844b681c661a78d387b4b8" bot_update
got_revision_cp "refs/heads/master@{#606803}" bot_update
got_swarming_client_revision "f78187ab77127de42555afe0ad410bebde6ac6a5" bot_update
got_v8_revision "6130e69e8e0758f2769a6172c79d32c87bc39464" bot_update
got_v8_revision_cp "refs/heads/7.2.284@{#1}" bot_update
got_webrtc_revision "e769ed90c3599a131166d5a61625c1b64318fb91" bot_update
got_webrtc_revision_cp "refs/heads/master@{#25544}" bot_update
swarm_hashes {} find isolated tests

Blamelist:

  1. Patrik Höglund (phoglundohnoyoudont@chromium.org)
  2. chromium-internal-autoroll (chromium-internal-autorollohnoyoudont@skia-corp.google.com.iam.gserviceaccount.com)
  3. Clemens Arbesser (arbesserohnoyoudont@google.com)
  4. Benoît Lizé (lizebohnoyoudont@chromium.org)
  5. Chromium WPT Sync (wpt-autorollerohnoyoudont@chops-service-accounts.iam.gserviceaccount.com)
  6. v8-ci-autoroll-builder (v8-ci-autoroll-builderohnoyoudont@chops-service-accounts.iam.gserviceaccount.com)
  7. chromium-autoroll (chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com)
  8. Findit (findit-for-meohnoyoudont@appspot.gserviceaccount.com)
  9. Sheng-Hao Tsao (shenghaoohnoyoudont@google.com)
  10. Mathias Carlen (mcarlenohnoyoudont@chromium.org)
  11. Yi Su (mrsuyiohnoyoudont@chromium.org)
  12. Arthur Sonzogni (arthursonzogniohnoyoudont@chromium.org)
  13. Rune Lillesveen (futharkohnoyoudont@chromium.org)
  14. chromium-autoroll (chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com)
  15. chromium-internal-autoroll (chromium-internal-autorollohnoyoudont@skia-corp.google.com.iam.gserviceaccount.com)
  16. Hirokazu Honda (hirohohnoyoudont@chromium.org)
  17. Mario Sanchez Prada (marioohnoyoudont@igalia.com)
  18. Koji Ishii (kojiiohnoyoudont@chromium.org)
  19. Balazs Engedy (engedyohnoyoudont@chromium.org)
  20. chromium-autoroll (chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com)
  21. Colin Blundell (blundellohnoyoudont@chromium.org)
  22. 3su6n15k.default@developer.gserviceaccount.com (3su6n15k.defaultohnoyoudont@developer.gserviceaccount.com)
  23. Balazs Engedy (engedyohnoyoudont@chromium.org)
  24. Rune Lillesveen (futharkohnoyoudont@chromium.org)
  25. Patrik Höglund (phoglundohnoyoudont@chromium.org)
  26. Patrik Höglund (phoglundohnoyoudont@chromium.org)
  27. Peng Huang (penghuangohnoyoudont@chromium.org)
  28. Yi Su (mrsuyiohnoyoudont@chromium.org)
  29. Yutaka Hirano (yhiranoohnoyoudont@chromium.org)
  30. Mario Sanchez Prada (marioohnoyoudont@igalia.com)
  31. David Jean (djeanohnoyoudont@google.com)
  32. chromium-internal-autoroll (chromium-internal-autorollohnoyoudont@skia-corp.google.com.iam.gserviceaccount.com)
  33. Stephane Zermatten (szermattohnoyoudont@chromium.org)

Timing:

Create Friday, 09-Nov-18 12:33:39 UTC
Start Friday, 09-Nov-18 12:34:07 UTC
End Friday, 09-Nov-18 15:54:44 UTC
Pending 27 secs
Execution 3 hrs 20 mins

All Changes:

  1. Revert "[autofill] add iOS helpers to get aria-label and aria-description"

    Changed by Patrik Höglund - phoglundohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 12:12:29 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 62dd51a1163937c255844b681c661a78d387b4b8

    Comments

    Revert "[autofill] add iOS helpers to get aria-label and aria-description"
    
    This reverts commit 95c8ebc6e5b760aa85692b012f448c4fa10d53d6.
    
    Reason for revert: Breaks FillJsTest: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/ios-webview/1004
    
    JavaScript error: TypeError: __gCrWeb.fill.getAriaDescription is not a function.
    
    Original change's description:
    > [autofill] add iOS helpers to get aria-label and aria-description
    > 
    > This CL adds the getAriaLabel() and getAriaDescription() helper
    > functions to the __GCrWeb.fill "namespace". These functions return
    > the ARIA label and description strings associated with an element.
    > 
    > The ARIA Label is the text referenced by one or more containing element
    > ids in the labelled elements aria-labelledby attribute, or the value
    > of the element's aria-label attribute, with preference given to the
    > aria-labelledby attribute. For example:
    > 
    >     <div id='foo-label'>Label for Foo</div>
    >     <input name='foo' aria-labelledby='foo-label'>
    >     <input name='bar' aria-label='Label for Bar'>
    > 
    > The aria-description is similar to aria-labelledby. For example:
    > 
    >     <div id='foo-description'>Long descriptive text for Foo</div>
    >     <input name='foo' aria-describedby='foo-descrition'>
    > 
    > Bug: 896719
    > Change-Id: I300b84298fc20e1b86474f60d08cdd1c9569036b
    > Reviewed-on: https://chromium-review.googlesource.com/c/1325032
    > Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
    > Reviewed-by: Moe Ahmadi <mahmadi@chromium.org>
    > Commit-Queue: Roger McFarlane <rogerm@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#606534}
    
    TBR=rogerm@chromium.org,sebsg@chromium.org,mahmadi@chromium.org
    
    Change-Id: I00eb534e6639c829f6ed5b312406ce51ba2aa429
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 896719
    Reviewed-on: https://chromium-review.googlesource.com/c/1329002
    Reviewed-by: Patrik Höglund <phoglund@chromium.org>
    Commit-Queue: Patrik Höglund <phoglund@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606803}

    Changed files

    • components/autofill/ios/form_util/fill_js_unittest.mm
    • components/autofill/ios/form_util/resources/fill.js
  2. Roll src-internal 0ee41cbd4bf0..73158cecda02 (2 commits)

    Changed by chromium-internal-autoroll - chromium-internal-autorollohnoyoudont@skia-corp.google.com.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 12:01:37 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 9b80c161e76af7ca42437617fa5424948d0f207f

    Comments

    Roll src-internal 0ee41cbd4bf0..73158cecda02 (2 commits)
    
    https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/0ee41cbd4bf0..73158cecda02
    
    
    Created with:
      gclient setdep -r src-internal@73158cecda02
    
    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: Ie142f038c6f81038911815c552aa5de603838b62
    Reviewed-on: https://chromium-review.googlesource.com/c/1328730
    Reviewed-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@{#606802}

    Changed files

    • DEPS
  3. [Autofill Assistant] All messages to the server now contain the active field trials of the client as part of the ClientContextProto.

    Changed by Clemens Arbesser - arbesserohnoyoudont@google.com
    Changed at Friday, 09-Nov-18 11:58:38 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 085bbd72774018acd197bab9921b470a42725408

    Comments

    [Autofill Assistant] All messages to the server now contain the active field trials of the client as part of the ClientContextProto.
    
    This will allow the server to log the field trial state, which in turn will allow us to analyze how user behavior changes with respect to the active field trials. Serializing all active field trials makes this easier to maintain at the cost of some additional message overhead and log storage. For each active field trial, the trial name and the name of the randomly selected trial group are sent.
    
    Bug: 806868
    Change-Id: I0417b3d548f36ca8c98c80182ec314e8bb20eba0
    Reviewed-on: https://chromium-review.googlesource.com/c/1326014
    Reviewed-by: Stephane Zermatten <szermatt@chromium.org>
    Commit-Queue: Clemens Arbesser <arbesser@google.com>
    Cr-Commit-Position: refs/heads/master@{#606801}

    Changed files

    • components/autofill_assistant/browser/protocol_utils.cc
    • components/autofill_assistant/browser/service.proto
  4. blink/bindings: Record compressed size and savings for ParkableString.

    Changed by Benoît Lizé - lizebohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 11:53:38 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 43d0eed3db77a5e6167b5ebbf6ecb3710790272a

    Comments

    blink/bindings: Record compressed size and savings for ParkableString.
    
    Records the total size of parkable strings, the total compressed size, the
    average compression ratio and the total savings from compression. As these
    values are correlated, separate histograms are required.
    
    The recording happens 30s after compression tasks are kicked off, provided that
    the renderer's state (foreground/background) hasn't changed in the meantime, to
    help with the interpretation of results. 30s is chosen as it's long enough for
    compression to be done, and short enough to minimize the risk of the renderer
    getting killed and/or moving back to foreground.
    
    Also remove an outdated comment in parkable_string.cc.
    
    Bug: 877044
    Change-Id: Ic8cf4a410d182387ffd1d37b23d84b5426d227cb
    Reviewed-on: https://chromium-review.googlesource.com/c/1323549
    Reviewed-by: Kentaro Hara <haraken@chromium.org>
    Reviewed-by: Steven Holte <holte@chromium.org>
    Commit-Queue: Benoit L <lizeb@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606800}

    Changed files

    • third_party/blink/renderer/platform/bindings/parkable_string.h
    • third_party/blink/renderer/platform/bindings/parkable_string_manager.cc
    • third_party/blink/renderer/platform/bindings/parkable_string_manager.h
    • third_party/blink/renderer/platform/bindings/parkable_string_test.cc
    • tools/metrics/histograms/histograms.xml
  5. Import wpt@eed621bc2f995bced9481df2cead4c4632c72cdd

    Changed by Chromium WPT Sync - wpt-autorollerohnoyoudont@chops-service-accounts.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 11:53:34 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 80ae982b6309cb7b9a9e8320524ef384deeed972

    Comments

    Import wpt@eed621bc2f995bced9481df2cead4c4632c72cdd
    
    Using wpt-import in Chromium f038ea3445a10c7f8116f52afe7add030b77d243.
    With Chromium commits locally applied on WPT:
    270d4abddc "ServiceWorker: Add new WPT tests to make sure to update a registration with different script type and identical script content."
    
    
    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:
    kojii@chromium.org:
      external/wpt/css/css-text/overflow-wrap
    
    NOAUTOREVERT=true
    TBR=foolip
    
    No-Export: true
    Change-Id: I89a829c659b10bbbdfb07c86d8e997b9e9edbc08
    Reviewed-on: https://chromium-review.googlesource.com/c/1328711
    Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com>
    Reviewed-by: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/master@{#606799}

    Changed files

    • third_party/WebKit/LayoutTests/external/WPT_BASE_MANIFEST_5.json
    • third_party/WebKit/LayoutTests/external/wpt/cors/cors-safelisted-request-header.any-expected.txt
    • third_party/WebKit/LayoutTests/external/wpt/cors/cors-safelisted-request-header.any.js
    • third_party/WebKit/LayoutTests/external/wpt/cors/cors-safelisted-request-header.any.worker-expected.txt
    • third_party/WebKit/LayoutTests/external/wpt/css/css-text/overflow-wrap/overflow-wrap-min-content-size-001.html
    • third_party/WebKit/LayoutTests/external/wpt/css/css-text/overflow-wrap/overflow-wrap-min-content-size-002.html
    • third_party/WebKit/LayoutTests/external/wpt/css/css-text/overflow-wrap/overflow-wrap-min-content-size-003.html
  6. Update V8 to version 7.2.284.

    Changed by v8-ci-autoroll-builder - v8-ci-autoroll-builderohnoyoudont@chops-service-accounts.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 11:52:33 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 5cb8a569d871afd0b4ac8238e3edbe4b81fc6791

    Comments

    Update V8 to version 7.2.284.
    
    Summary of changes available at:
    https://chromium.googlesource.com/v8/v8/+log/c465c5ea..6130e69e
    
    Please follow these instructions for assigning/CC'ing issues:
    https://github.com/v8/v8/wiki/Triaging%20issues
    
    Please close rolling in case of a roll revert:
    https://v8-roll.appspot.com/
    This only works with a Google account.
    
    CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_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;luci.chromium.try:android_optional_gpu_tests_rel
    
    TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com
    
    Change-Id: Ic5c1b7a01e1e1fb5cdc8f73ba76ff53f7d97c3bf
    Reviewed-on: https://chromium-review.googlesource.com/c/1328712
    Reviewed-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@{#606798}

    Changed files

    • DEPS
  7. Roll src/third_party/perfetto 365e59b9d0c3..486203eeaffd (1 commits)

    Changed by chromium-autoroll - chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 11:52:29 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 3e84fb354691adf1a8d49c2be3c5758718a9f8bb

    Comments

    Roll src/third_party/perfetto 365e59b9d0c3..486203eeaffd (1 commits)
    
    https://android.googlesource.com/platform/external/perfetto.git/+log/365e59b9d0c3..486203eeaffd
    
    
    git log 365e59b9d0c3..486203eeaffd --date=short --no-merges --format='%ad %ae %s'
    2018-11-09 hjd@google.com Merge "perfetto_cmd: Teach perfetto_cmd to parse pbtxt configs"
    
    
    Created with:
      gclient setdep -r src/third_party/perfetto@486203eeaffd
    
    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: If1aa1235b1bf32d44d939c356c4b9b8d8bb590ae
    Reviewed-on: https://chromium-review.googlesource.com/c/1328728
    Reviewed-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@{#606797}

    Changed files

    • DEPS
  8. Revert "media/gpu/VEA unittest: Test DMABuf-backed input buffers"

    Changed by Findit - findit-for-meohnoyoudont@appspot.gserviceaccount.com
    Changed at Friday, 09-Nov-18 11:31:57 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision a2ebf50601155e617b5966d1e585734cb4ffaade

    Comments

    Revert "media/gpu/VEA unittest: Test DMABuf-backed input buffers"
    
    This reverts commit c4e9ae1b9abc167c177d659dbcc3d1f794e93064.
    
    Reason for revert:
    
    Findit (https://goo.gl/kROfz5) identified CL at revision 606788 as the
    culprit for failures in the build cycles as shown on:
    https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2M0ZTlhZTFiOWFiYzE2N2MxNzdkNjU5ZGJjYzNkMWY3OTRlOTMwNjQM
    
    Sample Failed Build: https://ci.chromium.org/buildbot/chromium/win32-rel/5951
    
    Sample Failed Step: compile
    
    Original change's description:
    > media/gpu/VEA unittest: Test DMABuf-backed input buffers
    > 
    > This adds the command line option, --natvie_input, that VEA unittest passes
    > DMABuf-backed video frame to VEA.
    > DMABufs are obtained by gbm though NativePixmap. To create NativePixmap on
    > Chrome OS, it needs to set up Ozone environment properly. Therefore, this change
    > also makes VEA unittest dependent on Ozone.
    > 
    > BUG=chromium:895230
    > TEST=VEA unittest on eve w/wo --native_input
    > 
    > Change-Id: I67526a1b6b6cf1ae74a96000ec2af14995dbd4fc
    > Reviewed-on: https://chromium-review.googlesource.com/c/1295636
    > Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
    > Reviewed-by: Pawel Osciak <posciak@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#606788}
    
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    BUG=chromium:895230
    
    Change-Id: I75c80e914ec2493c26b34ddb4644e53099ba6bf5
    Reviewed-on: https://chromium-review.googlesource.com/c/1328713
    Cr-Commit-Position: refs/heads/master@{#606796}

    Changed files

    • media/gpu/BUILD.gn
    • media/gpu/test/generic_dmabuf_video_frame_mapper.cc
    • media/gpu/test/rendering_helper.cc
    • media/gpu/test/texture_ref.cc
    • media/gpu/test/texture_ref.h
    • media/gpu/test/video_encode_accelerator_unittest_helpers.cc
    • media/gpu/test/video_encode_accelerator_unittest_helpers.h
    • media/gpu/test/video_frame_mapper_factory.cc
    • media/gpu/test/video_frame_mapper_factory.h
    • media/gpu/test/video_frame_validator.cc
    • media/gpu/test/video_frame_validator.h
    • media/gpu/video_decode_accelerator_unittest.cc
    • media/gpu/video_encode_accelerator_unittest.cc
  9. Only open GCA if ARC is P or above

    Changed by Sheng-Hao Tsao - shenghaoohnoyoudont@google.com
    Changed at Friday, 09-Nov-18 10:59:55 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision c028d7085fdc469ffdf4efec3f9ae8d22d1e3640

    Comments

    Only open GCA if ARC is P or above
    
    This CL changes the consolidated camera app icon logic so that it
    launches GCA/GCA migration app only if ARC is P or above. This is to
    address 2 problems:
    
    1. Android N and below do not support external cameras. We would like
    users to use CCA so that they can still utilize external cameras.
    2. Managed users in Android N still have the problem where GCA could be
    stripped out.
    
    Bug: b:118860552,chromium:896140
    Test: Used N devices (caroline) with google.com account and verified
    that CCA is opened.
    
    Change-Id: I7836d2b4256856028901e5e542f460e7107fc8db
    Reviewed-on: https://chromium-review.googlesource.com/c/1316982
    Commit-Queue: Sheng-hao Tsao <shenghao@google.com>
    Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
    Reviewed-by: Tao Wu <wutao@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606795}

    Changed files

    • chrome/browser/ui/app_list/internal_app/internal_app_metadata.cc
  10. [Autofill Assistant] Checkbox for terms acceptance.

    Changed by Mathias Carlen - mcarlenohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:55:38 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 652ddcf5b36955638fa64b013c88c0dd2f939fd8

    Comments

    [Autofill Assistant] Checkbox for terms acceptance.
    
    This patch introduces a checkbox in the Autofill Assistant payment request UI.
    The state of which is communicated in the processed action results to
    the backend.
    
    This logic only exists on the Java side and is opaquely passed through
    the c++ layer.
    
    Issue: b/117920169
    Screenshot: https://screenshot.googleplex.com/iiHvY18nLqn.png
    
    Bug: 806868
    Change-Id: Ica17176c4cac695c126d1d5695a8b7e5a31020e1
    Reviewed-on: https://chromium-review.googlesource.com/c/1316770
    Commit-Queue: Mathias Carlen <mcarlen@chromium.org>
    Reviewed-by: Theresa <twellington@chromium.org>
    Reviewed-by: Ganggui Tang <gogerald@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606794}

    Changed files

    • chrome/android/java/res_autofill_assistant/layout/autofill_assistant_payment_request.xml
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/AutofillAssistantPaymentRequest.java
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/AutofillAssistantUiController.java
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/ui/PaymentRequestUI.java
    • chrome/browser/android/autofill_assistant/ui_controller_android.cc
    • chrome/browser/android/autofill_assistant/ui_controller_android.h
    • components/autofill_assistant/browser/actions/get_payment_information_action.cc
    • components/autofill_assistant/browser/payment_information.h
    • components/autofill_assistant/browser/service.proto
    • components/autofill_assistant_strings.grdp
  11. Change feature flag of Custom Search Engine to be enabled by default.

    Changed by Yi Su - mrsuyiohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:53:11 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 8fbf09d888ad3453558caf7ba438acf381860d8a

    Comments

    Change feature flag of Custom Search Engine to be enabled by default.
    
    The feature of Custom Search Engine is nearly finished and ready to be
    tested. Chnage the feature flag to be enabled by default.
    
    Bug: 433824
    Change-Id: Ied1d61b1df73bd8ceee89a2088e3e53418cbf856
    Reviewed-on: https://chromium-review.googlesource.com/c/1326002
    Reviewed-by: Eric Noyau <noyau@chromium.org>
    Commit-Queue: Yi Su <mrsuyi@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606793}

    Changed files

    • ios/chrome/browser/search_engines/feature_flags.cc
  12. WebRequestProxyingURLLoader: Close the current loader before internal redirect.

    Changed by Arthur Sonzogni - arthursonzogniohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:48:36 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 4d3cf41c0598cf64570d293b73399c0f82ba7ccd

    Comments

    WebRequestProxyingURLLoader: Close the current loader before internal redirect.
    
    Please see the explanation here:
    https://bugs.chromium.org/p/chromium/issues/detail?id=882661#c72
    
    Generating a fake internal redirect causes conflicts with what the
    current URLLoader is doing. The latter doesn't know anything about the
    redirect. To work properly, the current URLLoader must be destroyed
    first. The current code automatically handles creating a new
    URLLoader after FollowRedirect().
    
    Bug: 882661
    Change-Id: Ib17dc2f29a2c63da735794b4792ef7805e7dbc58
    Reviewed-on: https://chromium-review.googlesource.com/c/1326481
    Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
    Reviewed-by: Ken Rockot <rockot@google.com>
    Reviewed-by: Clark DuVall <cduvall@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606792}

    Changed files

    • extensions/browser/api/web_request/web_request_proxying_url_loader_factory.cc
  13. Support ::placeholder after ::slotted().

    Changed by Rune Lillesveen - futharkohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:48:05 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 0cd41aa340d600b3c1f1725d94c2e06e11635531

    Comments

    Support ::placeholder after ::slotted().
    
    Tree-abiding pseudo elements are allowed after ::slotted() according to
    css-scoping-1. According to css-pseudo-4 they are ::before, ::after,
    ::marker, and ::placeholder. We already allowed ::before and ::after. We
    do not support ::marker. This CL allows ::placeholder.
    
    TEST=external/wpt/css/css-scoping/slotted-parsing.html
    TEST=external/wpt/css/css-scoping/slotted-placeholder.html
    
    Bug: 902518
    Change-Id: I963fa93e68c8fbd33fc8090e930be9a00f3c9bd4
    Reviewed-on: https://chromium-review.googlesource.com/c/1323049
    Commit-Queue: Rune Lillesveen <futhark@chromium.org>
    Reviewed-by: Fergal Daly <fergal@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606791}

    Changed files

    • third_party/WebKit/LayoutTests/TestExpectations
    • third_party/WebKit/LayoutTests/external/wpt/css/css-scoping/slotted-parsing-expected.txt
    • third_party/blink/renderer/core/css/css_selector.cc
    • third_party/blink/renderer/core/css/css_selector.h
    • third_party/blink/renderer/core/css/parser/css_parser_selector.h
    • third_party/blink/renderer/core/css/parser/css_selector_parser.cc
    • third_party/blink/renderer/core/css/parser/css_selector_parser_test.cc
    • third_party/blink/renderer/core/css/resolver/style_resolver.cc
  14. Roll src/third_party/skia 22b0ec4f83aa..88e15bdaf539 (2 commits)

    Changed by chromium-autoroll - chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 10:47:32 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f21161ed54556a4f278ed78747e7f96f4ad924d3

    Comments

    Roll src/third_party/skia 22b0ec4f83aa..88e15bdaf539 (2 commits)
    
    https://skia.googlesource.com/skia.git/+log/22b0ec4f83aa..88e15bdaf539
    
    
    git log 22b0ec4f83aa..88e15bdaf539 --date=short --no-merges --format='%ad %ae %s'
    2018-11-09 skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com Update go_deps asset
    2018-11-09 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 d856ca48f425..5d2dfa46c05c (2 commits)
    
    
    Created with:
      gclient setdep -r src/third_party/skia@88e15bdaf539
    
    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=allanmac@chromium.org
    
    Change-Id: I669e2dcceb60198f833772e058db4e4ffd2de923
    Reviewed-on: https://chromium-review.googlesource.com/c/1328723
    Reviewed-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@{#606790}

    Changed files

    • DEPS
  15. Roll src-internal 53aa50a0dced..0ee41cbd4bf0 (2 commits)

    Changed by chromium-internal-autoroll - chromium-internal-autorollohnoyoudont@skia-corp.google.com.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 10:45:51 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 8de008a2a6f8d8eaec1b095550660cb5322551e9

    Comments

    Roll src-internal 53aa50a0dced..0ee41cbd4bf0 (2 commits)
    
    https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/53aa50a0dced..0ee41cbd4bf0
    
    
    Created with:
      gclient setdep -r src-internal@0ee41cbd4bf0
    
    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: I97a39edb26a9fcecef541f317a1788d833f9fa14
    Reviewed-on: https://chromium-review.googlesource.com/c/1328727
    Reviewed-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@{#606789}

    Changed files

    • DEPS
  16. media/gpu/VEA unittest: Test DMABuf-backed input buffers

    Changed by Hirokazu Honda - hirohohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:41:08 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision c4e9ae1b9abc167c177d659dbcc3d1f794e93064

    Comments

    media/gpu/VEA unittest: Test DMABuf-backed input buffers
    
    This adds the command line option, --natvie_input, that VEA unittest passes
    DMABuf-backed video frame to VEA.
    DMABufs are obtained by gbm though NativePixmap. To create NativePixmap on
    Chrome OS, it needs to set up Ozone environment properly. Therefore, this change
    also makes VEA unittest dependent on Ozone.
    
    BUG=chromium:895230
    TEST=VEA unittest on eve w/wo --native_input
    
    Change-Id: I67526a1b6b6cf1ae74a96000ec2af14995dbd4fc
    Reviewed-on: https://chromium-review.googlesource.com/c/1295636
    Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
    Reviewed-by: Pawel Osciak <posciak@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606788}

    Changed files

    • media/gpu/BUILD.gn
    • media/gpu/test/generic_dmabuf_video_frame_mapper.cc
    • media/gpu/test/rendering_helper.cc
    • media/gpu/test/texture_ref.cc
    • media/gpu/test/texture_ref.h
    • media/gpu/test/video_encode_accelerator_unittest_helpers.cc
    • media/gpu/test/video_encode_accelerator_unittest_helpers.h
    • media/gpu/test/video_frame_mapper_factory.cc
    • media/gpu/test/video_frame_mapper_factory.h
    • media/gpu/test/video_frame_validator.cc
    • media/gpu/test/video_frame_validator.h
    • media/gpu/video_decode_accelerator_unittest.cc
    • media/gpu/video_encode_accelerator_unittest.cc
  17. Fix declaration of override method in IdentityManagerObserverBridge

    Changed by Mario Sanchez Prada - marioohnoyoudont@igalia.com
    Changed at Friday, 09-Nov-18 10:36:59 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision b6586665e93e9211f934364c00eee32433027df9

    Comments

    Fix declaration of override method in IdentityManagerObserverBridge
    
    IdentityManager::OnRefreshTokenRemovedForAccount() expects as a first
    parameter a "const std::string&" instead of a "const AccountInfo&",
    so this patch fixes it to prevent build failures when using this API
    because of OnRefreshTokenRemovedForAccount(const AccountInfo&) not
    being a valid override for the virtual method declared.
    
    Bug: 903391
    Change-Id: I46cd6778240a40af6863ca7fd3b8d9a88e735fde
    Reviewed-on: https://chromium-review.googlesource.com/c/1327161
    Reviewed-by: Colin Blundell <blundell@chromium.org>
    Commit-Queue: Mario Sanchez Prada <mario@igalia.com>
    Cr-Commit-Position: refs/heads/master@{#606787}

    Changed files

    • services/identity/public/objc/identity_manager_observer_bridge.h
    • services/identity/public/objc/identity_manager_observer_bridge.mm
  18. Reland "Make NGOffsetMapping available for non-LayoutNG LayoutBlockFlow"

    Changed by Koji Ishii - kojiiohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:35:54 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision de3b8c8dbf530a94a15e660ecb2b9cd80a7f9843

    Comments

    Reland "Make NGOffsetMapping available for non-LayoutNG LayoutBlockFlow"
    
    This is a reland of b43c6fca4349b7f2ff53b41926f5d7cb42ee4e0f
    
    The original CL had moved a call to ToString() to within
    DCHECK, because the result is used only by DCHECK. But the
    ToString() has a side effect that must be performed.
    
    Because of the change, this CL breaks if DCHECK is not
    enabled. Unfortunately, no build configs in trybots and
    CQs have DCHECK disabled.
    
    Reverted the change, and added TODO to clean it up.
    
    Original change's description:
    > Make NGOffsetMapping available for non-LayoutNG LayoutBlockFlow
    >
    > This patch allows computing NGOffsetMapping for legacy inline
    > formatting context.
    >
    > For an experimental project "Invisible DOM"[1][2] to support
    > the find-in-page feature, it is needed to compute the text
    > content with whitespace collapsing applied, along with the
    > mapping to the DOM offset. This patch adds an API to compute
    > NGOffsetMapping even when the LayoutBlockFlow is not laid out
    > by LayoutNG for that purpose.
    >
    > Note that the project is still in the early phase. We may
    > revisit the design as it moves forward.
    >
    > [1] https://github.com/rakina/searchable-invisible-dom
    > [2] https://www.chromestatus.com/feature/5105291213406208
    >
    > Bug: 636993, 873057
    > Change-Id: I101b411960813a7b9b5c9c6e2db85d28737af882
    > Reviewed-on: https://chromium-review.googlesource.com/c/1322184
    > Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
    > Reviewed-by: Emil A Eklund <eae@chromium.org>
    > Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
    > Reviewed-by: Rakina Zata Amni <rakina@chromium.org>
    > Commit-Queue: Koji Ishii <kojii@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#606292}
    
    TBR=yoshin@chromium.org, eae@chromium.org, xiaochengh@chromium.org, rakina@chromium.org
    
    Bug: 636993, 873057
    Change-Id: I8a29073fb18a1349d758d37033e6b046a54d5beb
    Reviewed-on: https://chromium-review.googlesource.com/c/1327582
    Reviewed-by: Koji Ishii <kojii@chromium.org>
    Commit-Queue: Koji Ishii <kojii@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606786}

    Changed files

    • third_party/blink/renderer/core/layout/ng/inline/ng_inline_node.cc
    • third_party/blink/renderer/core/layout/ng/inline/ng_inline_node.h
    • third_party/blink/renderer/core/layout/ng/inline/ng_offset_mapping_test.cc
  19. Document ContentSettings.Exceptions.media-engagement.

    Changed by Balazs Engedy - engedyohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:28:57 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 5d8eed38bec9b93351d1e3cd65f981380acf6e3c

    Comments

    Document ContentSettings.Exceptions.media-engagement.
    
    Add the missing histogram suffix `media-engagement` to histograms.xml.
    This is the content setting type with by far the largest number of
    exceptions, yet missing from the dashboards.
    
    Bug: 890129
    Change-Id: Iaca89cf7b28db59a9d3c9baf3f810e765a8308a7
    Reviewed-on: https://chromium-review.googlesource.com/c/1326147
    Reviewed-by: Jesse Doherty <jwd@chromium.org>
    Commit-Queue: Balazs Engedy <engedy@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606785}

    Changed files

    • tools/metrics/histograms/histograms.xml
  20. Roll src/third_party/depot_tools 98c0fdb927c4..6edb632ff8cd (1 commits)

    Changed by chromium-autoroll - chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 10:25:18 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision b1fdb6589087a7a301a61113f8b7fca1fb70bc52

    Comments

    Roll src/third_party/depot_tools 98c0fdb927c4..6edb632ff8cd (1 commits)
    
    https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/98c0fdb927c4..6edb632ff8cd
    
    
    git log 98c0fdb927c4..6edb632ff8cd --date=short --no-merges --format='%ad %ae %s'
    2018-11-09 vadimsh@chromium.org [cipd] Bump CIPD client and vpython versions.
    
    
    Created with:
      gclient setdep -r src/third_party/depot_tools@6edb632ff8cd
    
    The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-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=agable@chromium.org
    
    Change-Id: Ic57f994925c49696f5f1f3da863df1b317081f7e
    Reviewed-on: https://chromium-review.googlesource.com/c/1328523
    Reviewed-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@{#606784}

    Changed files

    • DEPS
  21. Create target in //components/signin for shared code

    Changed by Colin Blundell - blundellohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:24:47 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 9bef2ebf0c9a8c06070e9de14cd17f5a5cf1b4d9

    Comments

    Create target in //components/signin for shared code
    
    //components/signin/core/browser contains three different types of code:
    
    1. Code that forms the guts of IdentityManager and will form the guts of
    the Identity Service implementation (e.g., SigninManager).
    
    2. Code that will be clients of IdentityManager (e.g., SigninTracker).
    
    3. Code that will be used on both sides (e.g., AccountInfo).
    
    1 and 2 have already been split via a preceding CL. This CL creates 3.
    To start off with, 3 contains two pieces of code that were already in
    their own standalone targets for similar motivations as the above:
    account_info.* and signin_metrics.*. In the future more code will
    likely move in.
    
    Bug: 901859
    Change-Id: Ibcae3c9f1ecff2dc7ccd013046b74c97b3dd2b7a
    Reviewed-on: https://chromium-review.googlesource.com/c/1323114
    Reviewed-by: Tom Sepez <tsepez@chromium.org>
    Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
    Commit-Queue: Colin Blundell <blundell@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606783}

    Changed files

    • components/autofill/core/browser/BUILD.gn
    • components/signin/core/browser/BUILD.gn
    • components/sync/BUILD.gn
    • ios/chrome/browser/ui/authentication/BUILD.gn
    • services/identity/BUILD.gn
    • services/identity/public/cpp/BUILD.gn
    • services/identity/public/cpp/account_info.typemap
  22. Automated Commit: LKGM 11244.0.0 for chromeos.

    Changed by 3su6n15k.default@developer.gserviceaccount.com - 3su6n15k.defaultohnoyoudont@developer.gserviceaccount.com
    Changed at Friday, 09-Nov-18 10:24:14 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 8c814268e0605b5134669e52db357d13aadd08fb

    Comments

    Automated Commit: LKGM 11244.0.0 for chromeos.
    
    BUG=762641
    TBR=rkc@chromium.org
    
    Change-Id: Ic280acd363864f15e43d89b7ee466b554fc2e614
    Reviewed-on: https://chromium-review.googlesource.com/c/1328841
    Reviewed-by: ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com>
    Commit-Queue: ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/master@{#606782}

    Changed files

    • chromeos/CHROMEOS_LKGM
  23. Prevent use-after-free in SiteEngagementService.

    Changed by Balazs Engedy - engedyohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:12:19 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f70115b4df17a675652ef65521ba6ddf81f59037

    Comments

    Prevent use-after-free in SiteEngagementService.
    
    The profile and its KeyedServices are normally destroyed before the
    TaskScheduler shuts down background threads, so the background task to
    record site engagement metrics needs to avoid using any members of
    SiteEngagementService
    
    Bug: 900022
    Change-Id: Ibdebbd5a64d59fad29b7715be3557eeed411d741
    Reviewed-on: https://chromium-review.googlesource.com/c/1326441
    Reviewed-by: Dominick Ng <dominickn@chromium.org>
    Commit-Queue: Balazs Engedy <engedy@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606781}

    Changed files

    • chrome/browser/engagement/site_engagement_service.cc
  24. Do style propagation to generated content in LayoutObject.

    Changed by Rune Lillesveen - futharkohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:07:14 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision a944866662c9cd3841c2cc1c8e8335ff6fd838e4

    Comments

    Do style propagation to generated content in LayoutObject.
    
    It belongs with the other anonymous LayoutObject propagation code.
    There are still two loops for pseudo elements where we propagate style
    to anonymous table boxes like before but propagate style from the
    pseudo element to the generated content directly, skipping other
    anonymous boxes, in a separate loop. This is just keeping the same
    functionality for generated content without trying to propagate style
    through others anonymous boxes. Left a TODO for that.
    
    Change-Id: I3c38bc9d831b8a767fd601d47e4db0f81923ba47
    Reviewed-on: https://chromium-review.googlesource.com/c/1328041
    Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
    Commit-Queue: Rune Lillesveen <futhark@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606780}

    Changed files

    • third_party/blink/renderer/core/dom/pseudo_element.cc
    • third_party/blink/renderer/core/dom/pseudo_element.h
    • third_party/blink/renderer/core/layout/layout_object.cc
  25. Revert "Route SSLPrivateKey provider names into NetLog."

    Changed by Patrik Höglund - phoglundohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:06:07 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 3fe2afddb677e9cf62b0f24c9dd2b176156d34a7

    Comments

    Revert "Route SSLPrivateKey provider names into NetLog."
    
    This reverts commit db889e10709b97c8e0c872e167cef871147afcdb.
    
    Reason for revert: Speculative revert: A bunch of SSL related tests failing on Mac: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.10%20Tests/36146
    
    Original change's description:
    > Route SSLPrivateKey provider names into NetLog.
    > 
    > Improve debugging in the future so we don't have to keep asking "are you
    > using a smartcard? which?".
    > 
    > Bug: none
    > Cq-Include-Trybots: luci.chromium.try:linux_mojo
    > Change-Id: Ifbaf6f5aa0aa73dcd19be0ccc0046c2c2c069319
    > Reviewed-on: https://chromium-review.googlesource.com/c/1292509
    > Commit-Queue: David Benjamin <davidben@chromium.org>
    > Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
    > Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
    > Reviewed-by: Ryan Sleevi <rsleevi@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#606700}
    
    TBR=rsleevi@chromium.org,kinuko@chromium.org,davidben@chromium.org,emaxx@chromium.org
    
    Change-Id: Icb8db36c96ba199e0857296cd21fa9d3e8826c90
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: none
    Cq-Include-Trybots: luci.chromium.try:linux_mojo
    Reviewed-on: https://chromium-review.googlesource.com/c/1328882
    Reviewed-by: Patrik Höglund <phoglund@chromium.org>
    Commit-Queue: Patrik Höglund <phoglund@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606779}

    Changed files

    • chrome/browser/chromeos/certificate_provider/certificate_provider_service.cc
    • chrome/browser/chromeos/certificate_provider/certificate_provider_service_unittest.cc
    • content/browser/network_service_client.cc
    • net/android/java/src/org/chromium/net/AndroidKeyStore.java
    • net/android/keystore.cc
    • net/android/keystore.h
    • net/log/net_log_event_type_list.h
    • net/socket/ssl_client_socket_impl.cc
    • net/ssl/ssl_client_auth_cache_unittest.cc
    • net/ssl/ssl_platform_key_android.cc
    • net/ssl/ssl_platform_key_mac.cc
    • net/ssl/ssl_platform_key_nss.cc
    • net/ssl/ssl_platform_key_win.cc
    • net/ssl/ssl_private_key.h
    • net/ssl/test_ssl_private_key.cc
    • net/ssl/threaded_ssl_private_key.cc
    • net/ssl/threaded_ssl_private_key.h
    • net/url_request/url_request_unittest.cc
    • services/network/public/mojom/network_service.mojom
    • services/network/url_loader.cc
    • services/network/url_loader.h
    • services/network/url_loader_unittest.cc
  26. Revert "Enable unified garbage collections part two"

    Changed by Patrik Höglund - phoglundohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:03:45 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 9453dfbffd2000b3eafab2a59169da733d8cf08a

    Comments

    Revert "Enable unified garbage collections part two"
    
    This reverts commit 79ffe683c7cc22ab3478b119adcc9b49584bd9e2.
    
    Reason for revert: Speculative revert: lots of garbage-collection related failures on WebKit Linux Trusty ASAN (see https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20ASAN/17912)
    
    Original change's description:
    > Enable unified garbage collections part two
    > 
    > This flag overrides the default Blink value for content layer.
    > 
    > Sheriffs: This is a dry-run of the feature and will be reverted after
    > cycling through the waterfall and performance bots.
    > 
    > Tbr: jochen@chromium.org, lfg@chromium.org
    > Bug: 843903
    > Change-Id: Ie362301e4874198eb775346d051bf8267837aaf8
    > Reviewed-on: https://chromium-review.googlesource.com/c/1320970
    > Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    > Reviewed-by: Kentaro Hara <haraken@chromium.org>
    > Reviewed-by: Hannes Payer <hpayer@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#606446}
    
    TBR=haraken@chromium.org,hpayer@chromium.org,mlippautz@chromium.org,jochen@chromium.org,lfg@chromium.org
    
    Change-Id: I189aca41c8f8ba11c358a4c15bc7f56591e38196
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 843903
    Reviewed-on: https://chromium-review.googlesource.com/c/1328902
    Reviewed-by: Patrik Höglund <phoglund@chromium.org>
    Commit-Queue: Patrik Höglund <phoglund@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606778}

    Changed files

    • chrome/browser/apps/guest_view/web_view_browsertest.cc
    • content/public/common/content_features.cc
    • third_party/WebKit/LayoutTests/fast/dom/StyleSheet/gc-rule-children-wrappers-expected.txt
    • third_party/WebKit/LayoutTests/fast/dom/gc-9-expected.txt
    • third_party/WebKit/LayoutTests/fast/xpath/xpath-iterator-result-should-mark-its-nodeset-expected.txt
    • third_party/WebKit/LayoutTests/fast/xpath/xpath-other-nodeset-result-should-mark-its-nodeset-expected.txt
    • third_party/WebKit/LayoutTests/fast/xpath/xpath-snapshot-result-should-mark-its-nodeset-expected.txt
  27. Move context state related methods out of DecoderContext

    Changed by Peng Huang - penghuangohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 10:03:03 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision bea0d6b3c467e05183fb88d06a829a6d7e7d136a

    Comments

    Move context state related methods out of DecoderContext
    
    This CL adds a new GLContextVirtualDelegate interface to contain all
    context state related methods from DecoderContext. After this one, I am
    going to implement GLContextVirtualDelegate in RasterDecoderContextState
    and only create one GLContextVirtual for all raster decoder and display
    compositor.
    
    Bug: 900941, 902904
    Change-Id: I10d80bea8531885c449a2e676ce0552e9363b34c
    Reviewed-on: https://chromium-review.googlesource.com/c/1312338
    Reviewed-by: Antoine Labour <piman@chromium.org>
    Commit-Queue: Peng Huang <penghuang@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606777}

    Changed files

    • gpu/command_buffer/service/BUILD.gn
    • gpu/command_buffer/service/decoder_context.h
    • gpu/command_buffer/service/gl_context_virtual.cc
    • gpu/command_buffer/service/gl_context_virtual.h
    • gpu/command_buffer/service/gl_context_virtual_delegate.h
    • gpu/command_buffer/service/gl_state_restorer_impl.cc
    • gpu/command_buffer/service/gl_state_restorer_impl.h
    • gpu/command_buffer/service/raster_decoder.cc
    • gpu/command_buffer/service/raster_decoder_context_state.cc
    • gpu/command_buffer/service/raster_decoder_context_state.h
    • gpu/command_buffer/service/wrapped_sk_image.cc
  28. Add custom search engines list in Search Engine Settings.

    Changed by Yi Su - mrsuyiohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 09:45:23 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 2b549d861a796421a3907cb9a62eec272c2a0359

    Comments

    Add custom search engines list in Search Engine Settings.
    
    This CL changes how search engines are displayed in Search Engine Settings.
    
    Previous(1 list):
      1. search engines that are prepopulated or created by policy.
    
    Current(2 lists):
      1. Search engines that are prepopulated or created by policy, and possibly
         one custom search engine at the end if it's selected as default search
         engine;
      2. All remaining search engines.
    
    Unittests are also updated.
    
    Bug: 433824
    Change-Id: Ib70a18f0544101760c331bb349ee9818956ed231
    Reviewed-on: https://chromium-review.googlesource.com/c/1323509
    Reviewed-by: Mark Cogan <marq@chromium.org>
    Commit-Queue: Yi Su <mrsuyi@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606776}

    Changed files

    • ios/chrome/app/strings/ios_strings.grd
    • ios/chrome/browser/ui/settings/search_engine_settings_collection_view_controller.mm
    • ios/chrome/browser/ui/settings/search_engine_settings_collection_view_controller_unittest.mm
  29. Remove ReadableStreamOperations use from BodyStreamBuffer

    Changed by Yutaka Hirano - yhiranoohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 09:44:10 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 6105631e4f6362a083d8ad05999f21fc7918b67f

    Comments

    Remove ReadableStreamOperations use from BodyStreamBuffer
    
    Replace use of ReadableStreamOperations in BodyStreamBuffer with
    ReadableStream. This CL doesn't change the behavior.
    
    Bug: 894357, 888165, 902633
    
    Change-Id: Icd478dcc59ca86e020d540229d69f0c61fb689c0
    Reviewed-on: https://chromium-review.googlesource.com/c/1319408
    Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
    Reviewed-by: Adam Rice <ricea@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606775}

    Changed files

    • third_party/blink/renderer/core/fetch/body.cc
    • third_party/blink/renderer/core/fetch/body.h
    • third_party/blink/renderer/core/fetch/body_stream_buffer.cc
    • third_party/blink/renderer/core/fetch/body_stream_buffer.h
    • third_party/blink/renderer/core/fetch/request.cc
    • third_party/blink/renderer/core/fetch/response.cc
    • third_party/blink/renderer/core/fetch/response.h
    • third_party/blink/renderer/core/streams/readable_stream.cc
    • third_party/blink/renderer/core/streams/readable_stream.h
    • third_party/blink/renderer/core/streams/readable_stream_operations.cc
    • third_party/blink/renderer/core/streams/readable_stream_operations.h
    • third_party/blink/renderer/core/streams/readable_stream_operations_test.cc
    • third_party/blink/renderer/core/streams/readable_stream_test.cc
  30. Migrate clear_browsing_data_manager_unittest.mm to the Identity service

    Changed by Mario Sanchez Prada - marioohnoyoudont@igalia.com
    Changed at Friday, 09-Nov-18 09:43:50 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision b72eeec68ad1991b2f2989eb599e7486fe6a2b04

    Comments

    Migrate clear_browsing_data_manager_unittest.mm to the Identity service
    
    Use IdentityTestEnvironmentChromeBrowserStateAdaptor instead of manually
    creating the browser's state by providing specific factories, and then
    use the APIs from IdentityTestEnvironment, instead of SigninManagerBase.
    
    Bug: 890826
    Change-Id: Id9d422e002d8fd2edc0ef1ad158f29e95c6dce64
    Reviewed-on: https://chromium-review.googlesource.com/c/1318974
    Commit-Queue: Mario Sanchez Prada <mario@igalia.com>
    Reviewed-by: Sergio Collazos <sczs@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606774}

    Changed files

    • ios/chrome/browser/ui/settings/BUILD.gn
    • ios/chrome/browser/ui/settings/clear_browsing_data_manager_unittest.mm
  31. [ios] add manual fallback credit card and address view controller

    Changed by David Jean - djeanohnoyoudont@google.com
    Changed at Friday, 09-Nov-18 09:38:11 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision d9f5a2e523469d0570b1b9ae0bc5818523b2528b

    Comments

    [ios] add manual fallback credit card and address view controller
    
    Added base class that should be used in passwords later.
    
    Bug: 845472
    Change-Id: I1c1d2204d920ae141448239cfeb1c1c5a0e93a20
    Reviewed-on: https://chromium-review.googlesource.com/c/1319714
    Commit-Queue: David Jean <djean@chromium.org>
    Reviewed-by: Moe Ahmadi <mahmadi@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606773}

    Changed files

    • ios/chrome/browser/ui/autofill/manual_fill/BUILD.gn
    • ios/chrome/browser/ui/autofill/manual_fill/address_view_controller.h
    • ios/chrome/browser/ui/autofill/manual_fill/address_view_controller.mm
    • ios/chrome/browser/ui/autofill/manual_fill/card_view_controller.h
    • ios/chrome/browser/ui/autofill/manual_fill/card_view_controller.mm
    • ios/chrome/browser/ui/autofill/manual_fill/fallback_view_controller.h
    • ios/chrome/browser/ui/autofill/manual_fill/fallback_view_controller.mm
  32. Roll src-internal 27fe62b3b2f9..53aa50a0dced (4 commits)

    Changed by chromium-internal-autoroll - chromium-internal-autorollohnoyoudont@skia-corp.google.com.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 09:15:59 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision fd575eafdb7ca5f5800c6b866836f84583ea6baf

    Comments

    Roll src-internal 27fe62b3b2f9..53aa50a0dced (4 commits)
    
    https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/27fe62b3b2f9..53aa50a0dced
    
    
    Created with:
      gclient setdep -r src-internal@53aa50a0dced
    
    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: I0ceb97585997b51d4374733c0324a20c8d817333
    Reviewed-on: https://chromium-review.googlesource.com/c/1328724
    Reviewed-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@{#606772}

    Changed files

    • DEPS
  33. [Autofill Assistant] Allow FocusAction to restrict access to some elements

    Changed by Stephane Zermatten - szermattohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 09:13:18 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 32d31b5886e3493539864668484205446da536e4

    Comments

    [Autofill Assistant] Allow FocusAction to restrict access to some elements
    
    This change allows a focus action to provide a list of elements that can
    be accessed. On a touch event, the position of the touch is compared
    against the position of the elements before forwarding the event to the
    webpage (or not).
    
    The position of the element is updated periodically or after a scroll
    event.
    
    Currently, touches are just swallowed. A follow-up change will make it
    clear what's going on and offer alternatives.
    
    By default, the new behavior is disabled.
    
    Bug: 806868
    Change-Id: I5bce26d3a4a81a7167253b564ca416364d18b61e
    Reviewed-on: https://chromium-review.googlesource.com/c/1323489
    Commit-Queue: Stephane Zermatten <szermatt@chromium.org>
    Reviewed-by: Mathias Carlen <mcarlen@chromium.org>
    Reviewed-by: Theresa <twellington@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606771}

    Changed files

    • chrome/android/java/res_autofill_assistant/layout/autofill_assistant_sheet.xml
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/AutofillAssistantUiController.java
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/AutofillAssistantUiDelegate.java
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/ui/TouchEventFilter.java
    • chrome/android/java_sources.gni
    • chrome/browser/android/autofill_assistant/ui_controller_android.cc
    • chrome/browser/android/autofill_assistant/ui_controller_android.h
    • components/autofill_assistant/browser/BUILD.gn
    • components/autofill_assistant/browser/actions/action_delegate.h
    • components/autofill_assistant/browser/actions/focus_element_action.cc
    • components/autofill_assistant/browser/actions/focus_element_action.h
    • components/autofill_assistant/browser/actions/mock_action_delegate.h
    • components/autofill_assistant/browser/controller.cc
    • components/autofill_assistant/browser/controller.h
    • components/autofill_assistant/browser/element_area.cc
    • components/autofill_assistant/browser/element_area.h
    • components/autofill_assistant/browser/element_area_unittest.cc
    • components/autofill_assistant/browser/mock_run_once_callback.h
    • components/autofill_assistant/browser/mock_web_controller.h
    • components/autofill_assistant/browser/script_executor.cc
    • components/autofill_assistant/browser/script_executor.h
    • components/autofill_assistant/browser/service.proto
    • components/autofill_assistant/browser/ui_delegate.h
    • components/autofill_assistant/browser/web_controller.cc
    • components/autofill_assistant/browser/web_controller.h