Home | Search | chromium - Builders
Login

Builder Android arm64 Builder (dbg) Build 31824 Canonical Ubuntu

Results:

Success

Trigger Info:

Projectchromium
Revisionec900e71940eb3c8ef19f064e7e3e57967818731
Got Revisionec900e71940eb3c8ef19f064e7e3e57967818731

Execution:

Steps and Logfiles:

Show:
  1. ( 325 ms ) setup_build
    running recipe: "chromium"
  2. ( 1 secs ) remove .cipd
  3. ( 1 mins 10 secs ) bot_update
    [141GB/295GB used (47%)]
  4. ( 0 ) ensure_goma
  5. ( 960 ms ) swarming.py --version
    0.13
  6. ( 15 secs ) gclient runhooks
  7. ( 1 secs ) read test spec (chromium.android.json)
    path: /b/swarming/w/ir/cache/builder/src/testing/buildbot/chromium.android.json
  8. ( 4 secs ) clean local files
  9. ( 218 ms ) tree truth steps
  10. ( 35 secs ) generate_build_files
  11. ( 29 mins 3 secs ) compile
  12. ( 34 secs ) compile confirm no-op
    This should have been a no-op, but it wasn't.
  13. ( 5 mins 15 secs ) isolate tests
  14. ( 44 mins 53 secs ) package build
  15. ( 804 ms ) trigger

Build Properties:

NameValueSource
$build/goma {"jobs": 500} setup_build
$recipe_engine/path {"cache_dir": "/b/swarming/w/ir/cache", "temp_dir": "/b/swarming/w/ir/tmp/rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "swarm990-c4" setup_build
branch "refs/heads/master" setup_build
buildbucket {"build": {"bucket": "luci.chromium.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1531407481974110, "id": "8941196584139366960", "project": "chromium", "tags": ["builder:Android arm64 Builder (dbg)", "buildset:commit/git/ec900e71940eb3c8ef19f064e7e3e57967818731", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/ec900e71940eb3c8ef19f064e7e3e57967818731", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9106626646370671312", "scheduler_job_id:chromium/Android arm64 Builder (dbg)", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "Android arm64 Builder (dbg)" setup_build
buildnumber 31824 setup_build
mastername "chromium.android" setup_build
path_config "generic" setup_build
recipe "chromium" setup_build
repository "https://chromium.googlesource.com/chromium/src.git" setup_build
revision "ec900e71940eb3c8ef19f064e7e3e57967818731" setup_build
got_angle_revision "5598148b761380773c4e650b5b67f47553b0f090" bot_update
got_buildtools_revision "0dd5c6f980d22be96b728155249df2da355989d9" bot_update
got_nacl_revision "e6ce828ef60c4c1438867b535efbbb5d9a177c0e" bot_update
got_revision "ec900e71940eb3c8ef19f064e7e3e57967818731" bot_update
got_revision_cp "refs/heads/master@{#574559}" bot_update
got_swarming_client_revision "9a518d097dca20b7b00ce3bdfc5d418ccc79893a" bot_update
got_v8_revision "8b17db68f75a95d69debfa85b70aea3ec0766aaf" bot_update
got_v8_revision_cp "refs/heads/6.9.352@{#1}" bot_update
got_webrtc_revision "58c79f66dd3afef0ec2d77053e44f26b998b84b1" bot_update
got_webrtc_revision_cp "refs/heads/master@{#23950}" bot_update
swarm_hashes_refs/heads/master(at){#574559}_without_patch {"android_webview_unittests": "c67567e1c5a5bde36bd1265b9bf4ce71ab632460", "angle_unittests": "77932599ba11affae28003593c031956173d98b2", "base_unittests": "3cad31fde1acbe75a9a2c36c51b5a02dbb990f3b", "blink_common_unittests": "8882114132c35987844af5b287608527e5d4a907", "blink_heap_unittests": "e1a158a92f061d83122e362d7ae8d361f15e620c", "blink_platform_unittests": "df3d800e482454358d326006cf0be81e2a8e7605", "boringssl_crypto_tests": "90affe09b2078c9b8ea220413f37a264fdf3d70b", "boringssl_ssl_tests": "b91084f11b383643ec2b893f44e5b1222f87b754", "breakpad_unittests": "bd1bb2693b5782df3f98d72ee1c00244e169df63", "cacheinvalidation_unittests": "30f08533a863b9da5c508b7f8983c042df0878d7", "capture_unittests": "179a8094e9ea8c290990738116d6173bcc22ef1d", "cast_unittests": "25889911e7bbb1b1ad33b0a001ff4e85b64edce0", "cc_unittests": "989d9a17b79bf213e0c29cd909a9acc61e3a37ee", "chrome_public_test_apk": "680889a9f726f68287ce9e91fe73879e5190773a", "chrome_public_test_vr_apk": "111fc31b4d6b1969f24e0ac209d2b552a4170455", "chrome_sync_shell_test_apk": "7d8241a28a6d768c56534b193df33c999eb4d0a3", "components_browsertests": "9e1bce685dba1857047bf7861e8818e4527597e5", "components_unittests": "5dcbfad7f95855fde8db893a65107e02d71c748d", "content_browsertests": "f01b78f137695fd1a59e7afb1636352b097f5395", "content_shell_test_apk": "6f22408bfd9cf205099eaa455e1f8fe837111bb1", "content_unittests": "196c5dc9b425143af024625cf0ed4db0dde9e5ba", "crypto_unittests": "c9d61a8d7a7379b77d8b9fa8247ee1f49bc974b8", "device_unittests": "b4fa57d214b193177e1e20918fa4c9bc506cd3ff", "display_unittests": "637471cc2ca9cdc7e2a63c1a491314e16871c1c4", "events_unittests": "a382a900db302e00d42c36951ad3ef0936fb8360", "gcm_unit_tests": "8ec9fe0af21f1ba333e1010f40a0a4e6f6c81f5b", "gfx_unittests": "c47e4008ff3a8a54a3d764353623cca63cf6d6c4", "gin_unittests": "d1dddb2036ba5776c3bfbbe36e2e0f86d2696171", "gl_tests": "259e5019d606633143508c6c4c9642be3b3f88f9", "gl_unittests": "023da8fd1eefb14836e8fe1e588398d8045d10d1", "google_apis_unittests": "d4482c54562568137ae986deeeca4a43930a67a3", "gpu_unittests": "5fb03ef4ebbc56b80f73972fca5bb98fca08ebf2", "ipc_tests": "66fde598bd5f043f0e173ada6c7010374faf5199", "jingle_unittests": "c15d7edf5cf6b4618b00af9be23e057535c6993c", "latency_unittests": "e9a848d0fee4f6bdf295d372bc0d18fc67e2acb8", "libjingle_xmpp_unittests": "11456a4a8e0a9c35b93a863dd385f80f87cc6cb9", "media_blink_unittests": "b5d79bfff314c1fde1c201eae29f1d6b10147c0a", "media_service_unittests": "e38c4c7ccba4b59915b0287b00371f7107d680b1", "media_unittests": "f2cf66ce422ed31518bc0b6c2a0926b2e0e536bf", "midi_unittests": "1db91250aca4fa76b2fac81db9f08c7540fa137e", "mojo_test_apk": "74a3a31bfbe2e78657455ed8b6c6a6c83618d656", "mojo_unittests": "703a43ad352cd8daea19bbfb3327a5d0f5b310dc", "net_unittests": "646ab869d165db340f96465df535cbf77f75065f", "sandbox_linux_unittests": "356c492745d718d89479327cef0222c5c5ea832d", "services_unittests": "150bffc1d09326e31554c5077bfdb97c948e46ce", "shell_dialogs_unittests": "c63a0facab0363f10aa99e01405dadb1542f486b", "skia_unittests": "dcfa7d1c7c6d69e60dace4cd50f42698975a734b", "sql_unittests": "c293740aca64ddc2bf12443f90ea8b193bd05da9", "storage_unittests": "8ff33915d4dc22508f22a7caf2ad4d8b46cb8b25", "ui_android_unittests": "6e495cc7d2d51f76aa2ed3688f5b99ebbe050028", "ui_base_unittests": "6d52f46335df15a433f3f5910898491e8342f306", "ui_touch_selection_unittests": "7c7733acdc8fe4a3196dfde99e8b7889bdd048c9", "unit_tests": "8e02db1042d07841138b942209940044fde2b690", "url_unittests": "7df1d3948e6064a3e54b3c3fa2f5637b6d3e54e7", "viz_unittests": "61354119c66d8f12ae00e26c0b1aaf8b8bf1a4e1", "vr_common_unittests": "a14bf6b7989a8edcd4aef1725f21edadb29cc94b", "vr_pixeltests": "2d6f0ec8328b02bec2bf5614e9eea1cc087868b9", "webkit_unit_tests": "3276a2434838ec292c5850a5809e877e77f0ee0c", "webview_instrumentation_test_apk": "c54da802273dee5b5400d58b5a63422e4889865c", "wtf_unittests": "7a9b915ae805f2240b38c56ac04c356fcf277a44"} isolate tests
build_archive_url "gs://chromium-android-archive/chromium.android/Android arm64 Builder (dbg)/full-build-linux_ec900e71940eb3c8ef19f064e7e3e57967818731.zip" package build

Blamelist:

  1. Orsi Batiz (orsibatizohnoyoudont@google.com)
  2. Gabriel Charette (gabohnoyoudont@chromium.org)
  3. Boris Sazonov (bsazonovohnoyoudont@chromium.org)
  4. pdfium-chromium-autoroll (pdfium-chromium-autorollohnoyoudont@skia-buildbots.google.com.iam.gserviceaccount.com)
  5. Tiger Oakes (tigeroohnoyoudont@google.com)
  6. skia-chromium-autoroll (skia-chromium-autorollohnoyoudont@skia-buildbots.google.com.iam.gserviceaccount.com)
  7. clamy (clamyohnoyoudont@chromium.org)
  8. Jan Wilken Doerrie (jdoerrieohnoyoudont@chromium.org)
  9. Gauthier Ambard (gambardohnoyoudont@chromium.org)
  10. Colin Blundell (blundellohnoyoudont@chromium.org)
  11. stkhapugin@chromium.org (stkhapuginohnoyoudont@chromium.org)

Timing:

Create Thursday, 12-Jul-18 14:58:01 UTC
Start Thursday, 12-Jul-18 14:58:04 UTC
End Thursday, 12-Jul-18 16:22:17 UTC
Pending 2 secs
Execution 1 hrs 24 mins

All Changes:

  1. DOMParser changed to accept StringOrTrustedHTML arguments

    Changed by Orsi Batiz - orsibatizohnoyoudont@google.com
    Changed at Thursday, 12-Jul-18 14:40:43 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision ec900e71940eb3c8ef19f064e7e3e57967818731

    Comments

    DOMParser changed to accept StringOrTrustedHTML arguments
    
    Previously existing constructor made private and its name is now parseFromStringInternal
    parseFromString wraps parseFromStringInternal
    
    Bug: 739170
    Change-Id: I3c4dd3aae13bb478ed054d328c77aa2d4ea5d3ef
    Reviewed-on: https://chromium-review.googlesource.com/1128970
    Reviewed-by: Mike West <mkwst@chromium.org>
    Commit-Queue: Orsolya Bernadett Batiz <orsibatiz@google.com>
    Cr-Commit-Position: refs/heads/master@{#574559}

    Changed files

    • third_party/WebKit/LayoutTests/external/wpt/trusted-types/DOMParser-requiresTrustedTypes.tentative.html
    • third_party/WebKit/LayoutTests/external/wpt/trusted-types/DOMParser.tentative.html
    • third_party/blink/renderer/core/xml/dom_parser.cc
    • third_party/blink/renderer/core/xml/dom_parser.h
    • third_party/blink/renderer/core/xml/dom_parser.idl
  2. [MessageLoop] Fix flaky death-on-destruction test

    Changed by Gabriel Charette - gabohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 14:40:40 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision fa9ef2b1e549c225a401873e66295bb7afdd51ee

    Comments

    [MessageLoop] Fix flaky death-on-destruction test
    
    gtest complains about the multi-threadedness of these tests not being
    supported in POSIX death tests. The multi-threadedness is caused by
    the WITH_TASK_SCHEDULER test variant which is unnecessary to test this.
    Hence, detaching the test from the fixture and testing independently
    will fix the flake I think.
    
    R=danakj@chromium.org
    
    Bug: 860733
    Change-Id: I0972e3c65614145825b8b40d5bb2fa9ccea9108b
    Reviewed-on: https://chromium-review.googlesource.com/1134346
    Commit-Queue: Gabriel Charette <gab@chromium.org>
    Reviewed-by: danakj <danakj@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574558}

    Changed files

    • base/message_loop/message_loop_unittest.cc
  3. [Unity][Android] Increase char limit for 'Sync and Google services'

    Changed by Boris Sazonov - bsazonovohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 14:33:45 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f261109a91970ace70f10388bf8a39acb9977d4b

    Comments

    [Unity][Android] Increase char limit for 'Sync and Google services'
    
    This CL increases the char limit for 'Sync and Google services'
    translations to 40 characters. It also enables text wrapping for
    preference title that uses this string.
    
    Bug: 814728
    Change-Id: If1b2c3bb9d83a01a2fa13ddd38d39323dc94e689
    Reviewed-on: https://chromium-review.googlesource.com/1131126
    Reviewed-by: Theresa <twellington@chromium.org>
    Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574557}

    Changed files

    • chrome/android/java/res/xml/main_preferences.xml
    • chrome/android/java/strings/android_chrome_strings.grd
  4. Roll src/third_party/pdfium 5ff09fb5ee90..6aa2190f70a8 (1 commits)

    Changed by pdfium-chromium-autoroll - pdfium-chromium-autorollohnoyoudont@skia-buildbots.google.com.iam.gserviceaccount.com
    Changed at Thursday, 12-Jul-18 14:29:37 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision df1df391ec8f9534d21219ea753c260c4357588b

    Comments

    Roll src/third_party/pdfium 5ff09fb5ee90..6aa2190f70a8 (1 commits)
    
    https://pdfium.googlesource.com/pdfium.git/+log/5ff09fb5ee90..6aa2190f70a8
    
    
    git log 5ff09fb5ee90..6aa2190f70a8 --date=short --no-merges --format='%ad %ae %s'
    2018-07-12 thestig@chromium.org Use JSGetObject() in even more places.
    
    
    Created with:
      gclient setdep -r src/third_party/pdfium@6aa2190f70a8
    
    The AutoRoll server is located here: https://pdfium-roll.skia.org
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
    
    If the roll is causing failures, please contact the current sheriff, who should
    be CC'd on the roll, and stop the roller if necessary.
    
    
    
    BUG=chromium:862059
    TBR=dsinclair@chromium.org
    
    Change-Id: I1f8681742658f04204fb7df938c0667e814d84c6
    Reviewed-on: https://chromium-review.googlesource.com/1134951
    Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
    Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/master@{#574556}

    Changed files

    • DEPS
  5. Adds favicon for supersize UI

    Changed by Tiger Oakes - tigeroohnoyoudont@google.com
    Changed at Thursday, 12-Jul-18 14:25:01 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision e6d9ffa49bc8e4b239cf5ebe1e7b886764280f5a

    Comments

    Adds favicon for supersize UI
    
    Creates a simple icon by taking the pie chart icon and giving it colors
    from the Chromium logo.
    (https://material.io/tools/icons/?search=pie_chart&icon=pie_chart)
    
    Icon:
    https://notwoods.github.io/chrome-supersize-reports/icon.png
    
    Bug: 847599
    Change-Id: Ib711bf0c717b9339914ec4ba2d9f369b33fb84d7
    Reviewed-on: https://chromium-review.googlesource.com/1134195
    Reviewed-by: agrieve <agrieve@chromium.org>
    Reviewed-by: Peter Wen <wnwen@chromium.org>
    Reviewed-by: Eric Stevenson <estevenson@chromium.org>
    Commit-Queue: Tiger Oakes <tigero@google.com>
    Cr-Commit-Position: refs/heads/master@{#574555}

    Changed files

    • tools/binary_size/libsupersize/html_report.py
    • tools/binary_size/libsupersize/template_tree_view/favicon.ico
    • tools/binary_size/libsupersize/template_tree_view/index.html
  6. Roll src/third_party/skia 4d6a844365e7..56a426a516d0 (2 commits)

    Changed by skia-chromium-autoroll - skia-chromium-autorollohnoyoudont@skia-buildbots.google.com.iam.gserviceaccount.com
    Changed at Thursday, 12-Jul-18 14:11:59 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 84a2e1ca7620207a3183a275d73d247035d4c078

    Comments

    Roll src/third_party/skia 4d6a844365e7..56a426a516d0 (2 commits)
    
    https://skia.googlesource.com/skia.git/+log/4d6a844365e7..56a426a516d0
    
    
    git log 4d6a844365e7..56a426a516d0 --date=short --no-merges --format='%ad %ae %s'
    2018-07-12 angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/angle2 5598148b7613..c9c259cccb2e (1 commits)
    2018-07-12 recipe-roller@chromium.org Roll recipe dependencies (trivial).
    
    
    Created with:
      gclient setdep -r src/third_party/skia@56a426a516d0
    
    The AutoRoll server is located here: https://autoroll.skia.org
    
    Documentation for the AutoRoller is here:
    https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
    
    If the roll is causing failures, please contact the current sheriff, who should
    be CC'd on the roll, and stop the roller if necessary.
    
    CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;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=robertphillips@chromium.org
    
    Change-Id: Iae08a82a8704081e534320f9183df4acbccb30d5
    Reviewed-on: https://chromium-review.googlesource.com/1134950
    Reviewed-by: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
    Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/master@{#574554}

    Changed files

    • DEPS
  7. Store multiple NavigationRequests in RenderFrameHost

    Changed by clamy - clamyohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 13:52:18 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision d3bfdb09875f6b6cdadd5075d7c1f7305b5118ec

    Comments

    Store multiple NavigationRequests in RenderFrameHost
    
    This CL makes it so that RenderFrameHost can store several
    NavigationRequests waiting to commit instead of just one. This avoids
    deleting a NavigationRequest that would later commit (which can happen
    in quick navigations with a renderer process that is slow to process the
    commit).
    
    The regression test was originally written by arthursonzogni@chromium.org in
    https://chromium-review.googlesource.com/c/chromium/src/+/671351.
    
    The background for this change can be found in the following design doc:
    https://docs.google.com/document/d/1mXjxYJptb_bZ_EqGMF-c4LTSnhjt6Gn_WVvSrsinpq8/edit#.
    
    Bug: 738177
    Change-Id: Idd666e41bd2094c90fa576c77cdc26fb7b54cd7f
    Reviewed-on: https://chromium-review.googlesource.com/1082440
    Commit-Queue: Camille Lamy <clamy@chromium.org>
    Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574553}

    Changed files

    • content/browser/browser_side_navigation_browsertest.cc
    • content/browser/frame_host/navigation_controller_impl_browsertest.cc
    • content/browser/frame_host/navigation_controller_impl_unittest.cc
    • content/browser/frame_host/navigation_request.cc
    • content/browser/frame_host/navigator_impl_unittest.cc
    • content/browser/frame_host/render_frame_host_impl.cc
    • content/browser/frame_host/render_frame_host_impl.h
    • content/browser/frame_host/render_frame_host_impl_browsertest.cc
    • content/browser/frame_host/render_frame_host_manager.cc
    • content/browser/frame_host/render_frame_host_manager_unittest.cc
    • content/browser/renderer_host/render_widget_host_view_browsertest.cc
    • content/browser/security_exploit_browsertest.cc
    • content/browser/site_per_process_browsertest.cc
    • content/browser/web_contents/web_contents_impl_unittest.cc
    • content/browser/webauth/webauth_browsertest.cc
    • content/public/test/browser_test_utils.cc
    • content/public/test/navigation_simulator.cc
    • content/public/test/test_renderer_host.cc
    • content/test/data/infinite_load_1.html
    • content/test/data/infinite_load_2.html
    • content/test/did_commit_provisional_load_interceptor.cc
    • content/test/did_commit_provisional_load_interceptor.h
    • content/test/test_render_frame_host.cc
    • content/test/test_render_frame_host.h
    • content/test/test_web_contents.cc
  8. [bluetooth] Move Map of Characteristics in Gatt Service

    Changed by Jan Wilken Doerrie - jdoerrieohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 13:43:27 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision efd642e61e6933f8e78a607f02b2478652547e92

    Comments

    [bluetooth] Move Map of Characteristics in Gatt Service
    
    This change cleans up BluetoothRemoteGattService and its implementations
    by moving the map of characteristics into the base class. This allows
    for less code duplication and a simplified interface. Furthermore, a
    AddCharacterisitic helper method is introduced.
    
    Bug: 821766
    Change-Id: Ib370c599f1c8519d497a379c976c89c48d7ecb56
    Reviewed-on: https://chromium-review.googlesource.com/1131178
    Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
    Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574552}

    Changed files

    • content/shell/browser/layout_test/layout_test_bluetooth_adapter_provider.cc
    • device/bluetooth/bluetooth_remote_gatt_characteristic.h
    • device/bluetooth/bluetooth_remote_gatt_service.cc
    • device/bluetooth/bluetooth_remote_gatt_service.h
    • device/bluetooth/bluetooth_remote_gatt_service_android.cc
    • device/bluetooth/bluetooth_remote_gatt_service_android.h
    • device/bluetooth/bluetooth_remote_gatt_service_mac.h
    • device/bluetooth/bluetooth_remote_gatt_service_mac.mm
    • device/bluetooth/bluetooth_remote_gatt_service_win.cc
    • device/bluetooth/bluetooth_remote_gatt_service_win.h
    • device/bluetooth/bluetooth_remote_gatt_service_winrt.cc
    • device/bluetooth/bluetooth_remote_gatt_service_winrt.h
    • device/bluetooth/bluez/bluetooth_remote_gatt_service_bluez.cc
    • device/bluetooth/bluez/bluetooth_remote_gatt_service_bluez.h
    • device/bluetooth/cast/bluetooth_remote_gatt_service_cast.cc
    • device/bluetooth/cast/bluetooth_remote_gatt_service_cast.h
    • device/bluetooth/device_unittest.cc
    • device/bluetooth/test/fake_remote_gatt_service.cc
    • device/bluetooth/test/fake_remote_gatt_service.h
    • device/bluetooth/test/mock_bluetooth_gatt_service.cc
    • device/bluetooth/test/mock_bluetooth_gatt_service.h
    • device/fido/fido_ble_connection_unittest.cc
  9. [iOS] Create FindInPage UI for UI Refresh

    Changed by Gauthier Ambard - gambardohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 13:41:04 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 81f3e175814bb20d103aa178a71272c2cd422193

    Comments

    [iOS] Create FindInPage UI for UI Refresh
    
    This CL creates the FindInPage UI to be used with UI Refresh.
    
    Bug: 805504
    Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
    Change-Id: Ib6f5adcfde4560a589fed323a717b02c4566621d
    Reviewed-on: https://chromium-review.googlesource.com/1133160
    Commit-Queue: Gauthier Ambard <gambard@chromium.org>
    Reviewed-by: Stepan Khapugin <stkhapugin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574551}

    Changed files

    • ios/chrome/browser/ui/find_bar/BUILD.gn
    • ios/chrome/browser/ui/find_bar/find_bar_controller_ios.mm
    • ios/chrome/browser/ui/find_bar/find_bar_ui_element.h
    • ios/chrome/browser/ui/find_bar/find_bar_view.h
    • ios/chrome/browser/ui/find_bar/find_bar_view.mm
  10. MutablePO2TS: Fire token available when adding invalid token at startup

    Changed by Colin Blundell - blundellohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 13:40:21 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision d373e8d468f12b8ca7bbf6b979961e3a3f29cc73

    Comments

    MutablePO2TS: Fire token available when adding invalid token at startup
    
    ProfileOAuth2TokenService maintains the invariant that if the user is
    signed in at startup, the primary account is always present in
    PO2TS::GetAccounts().
    
    To maintain this invariant, MutableProfileOAuth2TokenServiceDelegate
    performs the following action when the load of tokens from the on-disk
    database is complete: if the user is signed in, it checks whether there
    is a token present for the primary account and if not, inserts an
    invalid token for that account. (The token can be absent due to e.g. a
    failure to decrypt the database on disk for whatever reason).
    
    However, MutablePO2TSDelegate does *not* fire the
    OnRefreshTokenAvailable() callback in that case. This breaks the
    implicit expectation of clients that when ProfileOAuth2TokenService adds
    an account, it will always fire OnRefreshTokenAvailable(). It is also
    inconsistent with the behavior if the primary account happens to have
    an invalid token on disk that is successfully decrypted: in that case,
    MutablePO2TSDelegate will fire the observer callback, but of course,
    from clients' POV there is no difference whatsoever between these two
    cases.
    
    In particular, the failure of this invariant can cause IdentityManager
    to crash if a user in this situation later signs out, as the token
    being revoked violates IdentityManager's expectation that it had
    previously been made aware of that token via a notification that it
    was available.
    
    This CL remedies the problem by firing OnRefreshTokenAvailable() in
    this case. It also extends the relevant MutablePO2TSDelegate unittest
    to fail without this change and succeed with this change.
    
    Bug: 860011
    Change-Id: I66cebda29ec56d79f209f6b2c93dff67797c57b4
    Reviewed-on: https://chromium-review.googlesource.com/1126861
    Commit-Queue: Colin Blundell <blundell@chromium.org>
    Reviewed-by: David Roger <droger@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574550}

    Changed files

    • chrome/browser/signin/mutable_profile_oauth2_token_service_delegate.cc
    • chrome/browser/signin/mutable_profile_oauth2_token_service_delegate.h
    • chrome/browser/signin/mutable_profile_oauth2_token_service_delegate_unittest.cc
  11. [iOS] Hide clear button when the textfield is empty.

    Changed by stkhapugin@chromium.org - stkhapuginohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 13:35:49 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision a0056ea7bc75754d4ac5388b0ee9108cc604945d

    Comments

    [iOS] Hide clear button when the textfield is empty.
    
    Makes UI Refresh omnibox hide the clear button when the textfield
    is empty.
    
    Bug: 854622
    Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
    Change-Id: I7905efa0c5ead8c7c2884c27474ec992b970355c
    Reviewed-on: https://chromium-review.googlesource.com/1122627
    Reviewed-by: Justin Cohen <justincohen@chromium.org>
    Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574549}

    Changed files

    • ios/chrome/browser/ui/omnibox/omnibox_view_controller.mm