GitLab

Remove the version of FindViewAtLocation that takes a gfx::Point. All
the callers can use the version that uses a gfx::PointF wihout having
to do extra work. So update the code to do that.

Also, make TargetData::delta a gfx::Vector2dF because blink event
locations are floats.

BUG=796649

Change-Id: Ib1f8d19671a642aed5f111b03a22a632521e33af
Reviewed-on: https://chromium-review.googlesource.com/841724Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526034}

https://skia.googlesource.com/skia.git/+log/1fa9c434fc0d..92c7fa6b009b

$ git log 1fa9c434f..92c7fa6b0 --date=short --no-merges --format='%ad %ae %s'
2017-12-22 ethannicholas fix usage of deprecated SkMatrix44 constructor
2017-12-22 kjlubick Quarantine android bots instead of shutting off device

Created with:
  roll-dep src/third_party/skia


The AutoRoll server is located here: https://autoroll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=csmartdalton@chromium.org

Change-Id: Ia78c37ce90fe05647189d435d99c58f43764bc1e
Reviewed-on: https://chromium-review.googlesource.com/843064
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526033}

The tests has nothing to do with canonicalization, so this patch removes
the usage of VisibleSelection to keep the test cases simple.

Bug: 657237
Change-Id: Ib0d36734cface043d89118374e339c45938f945f
Reviewed-on: https://chromium-review.googlesource.com/843046Reviewed-by: Ryan Landay <rlanday@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526032}

https://chromium.googlesource.com/catapult.git/+log/5626f3590eea..3b3f9e1e789d

$ git log 5626f3590..3b3f9e1e7 --date=short --no-merges --format='%ad %ae %s'
2017-12-22 charliea Remove Benchmark.GetExpectations

Created with:
  roll-dep src/third_party/catapult


The AutoRoll server is located here: https://catapult-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I2736e53aca3945df1652af7f47170322e1acdfd4
Reviewed-on: https://chromium-review.googlesource.com/842109
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526031}

testScreenTapsNotRegistered would fail when run locally with an
INJECT_EVENTS permission error. This CL fixes that by injecting the
event in a slightly different way.

Bug: 
Change-Id: I04277ea74feb3de70a8472839992c9976763544d
Reviewed-on: https://chromium-review.googlesource.com/843203Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526030}

Change-Id: I3a9785519bfe9d6b10972bbe7b70a3c9c3f5fdbc
Reviewed-on: https://chromium-review.googlesource.com/842803Reviewed-by: Camille Lamy <clamy@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#526029}

The animation code has two different
kDefaultTransationDurationMs depending on what code is used.
This caused issues with jumbo builds where one of them
shadowed the other one (scary). This renames them so that they
have unique names, and also moves them into the same scope to
keep things consistent.

Bug: 773275
Change-Id: I77b5ab72e48a9c967105d8eb0b2610cfbd6f0b40
Reviewed-on: https://chromium-review.googlesource.com/841063Reviewed-by: danakj <danakj@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#526028}

Explicitly specifies that 'Flash' is the name of a plugin and should not
be translated.

Bug: 766447
Change-Id: I00d53a7628697720640cacfe5cd1e7f677d5e048
Reviewed-on: https://chromium-review.googlesource.com/831088Reviewed-by: Anthony LaForge <laforge@chromium.org>
Commit-Queue: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526027}

This CL updates fullscreen to:
- not perform an animation if a scroll ends when the toolbar is fully
  visible or hidden.
- scale the scroll end adjustment animation duration to the fraction
  of progress change required.

Bug: 796597
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I8939fbdd93735155e41a9ed1e2d19d1b72902b4a
Reviewed-on: https://chromium-review.googlesource.com/841535Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Reviewed-by: Louis Romero <lpromero@chromium.org>
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526026}

Bug: 796975
Change-Id: I61fc830dd8a6fdba1ed0eacd1d33416553b57e45
Reviewed-on: https://chromium-review.googlesource.com/842844Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Jeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526025}

This test flakily times out on all platforms regularly.

TBR=qyearsley@chromium.org

Bug: 796668
Change-Id: I7412389b460b371c3d53976960d3b39cc40fc4a9
Reviewed-on: https://chromium-review.googlesource.com/842465Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526024}

Adds support for usage of --dynamic-linker flag when using gcc.
This allows user to specify path to custom dynamic linker.

Bug: 
Change-Id: I3fcd549664e90e8a1eb2cbd190f47aa98b7f1f34
Reviewed-on: https://chromium-review.googlesource.com/763347Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526023}

I've fixed various issues in telemetry that were keeping pinch-zoom
tests from being enabled (see commits in bug). There should be nothing
blocking this anymore so lets turn them back on.

Bug: 610021
Change-Id: I71c380322655af213c20b73bb8361cd199d2b64e
Reviewed-on: https://chromium-review.googlesource.com/843122Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526022}

After Skia rolls this recent CL to fix the alpha clamping issue in high filter
quality (https://skia-review.googlesource.com/c/skia/+/87000),
we can refactor ImageBitmap code path to unify high quality premul and unpremul.

Bug: 787034, 796797
Change-Id: I2efc0c86b15b2f182e4af99afc538918d11f2f0d
Reviewed-on: https://chromium-review.googlesource.com/840829Reviewed-by: Olivia Lai <xlai@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Mohammad Reza Zakerinasab <zakerinasab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526021}

https://webrtc.googlesource.com/src.git/+log/5ec7e1276029..221b12666ff6

$ git log 5ec7e1276..221b12666 --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtcBUG=None,None


The AutoRoll server is located here: https://webrtc-chromium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: I804691e61357ba78fd2a2f47c5c6c26de3d3d88b
Reviewed-on: https://chromium-review.googlesource.com/843007Reviewed-by: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526020}

This CL removes the content features kMojoVideoEncodeAccelerator
and its surfacing to chrome://flags, because crrev.com/c/585512
removed the old IPC parts in favour of the Mojo path, thus
rendering this flag effectively meaningless.

Bug: 736517
Change-Id: Id0264d1c7e2382425f609219d7031ec799fa5f9f
Reviewed-on: https://chromium-review.googlesource.com/843123
Commit-Queue: Avi Drissman <avi@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526019}

Moves the native URL lists that VrShellNativeUiTest#testUrlOnNativeUi
and VrShellNavigationTest#testNativeNavigateAndInteraction use to a
constant, and switches testUrlOnNativeUi to use a loop instead of
duplicating lines for each URL.

Bug: 
Change-Id: I0a4e7889a25375ebd760bdd408ba2014442f1927
Reviewed-on: https://chromium-review.googlesource.com/841402Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526018}

This CL cleans up the old navigation code from RenderFrameImpl. In
particular, it removes the StartNavigationParams and the
FrameMsg_Navigate IPC, which are no longer used.

Bug: 789577
Change-Id: I3c4e7a1ed8b735095f05234acaabbbe028c45156
Reviewed-on: https://chromium-review.googlesource.com/842543
Commit-Queue: Camille Lamy <clamy@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526017}

Since the locationBarContainer is expanded and its stackView is not, this CL creates a margin
constant and anchors the bottomAnchor to the toolbar bottomAnchor, this way the Y position of the
StackView is not changed during expansion or contraction.

Bug: 796482
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I69aac20beb0a2b3b31d516b78da7b1b4ba3a6ab8
Reviewed-on: https://chromium-review.googlesource.com/841442Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526016}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/b2af9586a0f6..f2cb0f5b3ea5

$ git log b2af9586a..f2cb0f5b3 --date=short --no-merges --format='%ad %ae %s'
2017-12-21 joaodasilva Updated the gsutil version used by download_from_google_storage.

Created with:
  roll-dep src/third_party/depot_tools


The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=phajdan.jr@chromium.org

Change-Id: Ia55e13e39b29793ecc7ff97ffad1612e249e1bdc
Reviewed-on: https://chromium-review.googlesource.com/842904Reviewed-by: depot-tools-roller . <depot-tools-roller@chromium.org>
Commit-Queue: depot-tools-roller . <depot-tools-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526015}

This add two Feature params for calculating an inflation value to
apply for a loaded page with NoScript previews active. It uses the
DataReductionProxyDataUseObserver to report the estimated extra
original bytes (first capturing the committed PreviewsType during
the main frame load and then making the calc at DidFinishLoad time
based on the total network bytes).

Bug: 781885
Change-Id: I0618fd2d8801364f72ba52178aeb57df7d4a098b
Reviewed-on: https://chromium-review.googlesource.com/834807
Commit-Queue: Doug Arnett <dougarnett@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526014}

This change implements OpenFile and CloseFile in SmbFileSystem.

Bug: 757625
Test: Open and Close called manually via UI, producing expected results
Change-Id: Ib45025963afdaced2da646bda610e6a54b079f40
Reviewed-on: https://chromium-review.googlesource.com/840730
Commit-Queue: Bailey Berro <baileyberro@chromium.org>
Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526013}

Display 'No internet connection, connect to a network and try again'
message on failures to download the cleaner. Only for user-initiated
cleanups.

Bug: 776538
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I75f3a934e5d9ad1470e8f60497783ab6c73a0992
Reviewed-on: https://chromium-review.googlesource.com/841265
Commit-Queue: Veranika Liaukevich <veranika@chromium.org>
Reviewed-by: proberge <proberge@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526012}

Downstream bots are not compiling due to the profile not being there.
Rather than add the profile, I think it's better to disable AFDO for
cronet for now, since there's no reason to believe that the profile
would be accurate for cronet.

Bug: 796921
Change-Id: Id8337e695f06095e037c1611fa6e9a9ba038029c
Reviewed-on: https://chromium-review.googlesource.com/842944Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526011}

https://skia.googlesource.com/skia.git/+log/03a527c718ed..1fa9c434fc0d

$ git log 03a527c71..1fa9c434f --date=short --no-merges --format='%ad %ae %s'
2017-12-11 mtklein attempt 3: add experimental bilerp_clamp_8888 stage
2017-12-22 caryclark handle closed dashed rects

Created with:
  roll-dep src/third_party/skia


The AutoRoll server is located here: https://autoroll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=csmartdalton@chromium.org

Change-Id: Ice6287b636262cdeec22298e4d1faea9f7c1cc00
Reviewed-on: https://chromium-review.googlesource.com/843006Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526010}

Bug: 769140
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Icc8d85e38b0ca8d48182f0a96b7beb0428c27e15
Reviewed-on: https://chromium-review.googlesource.com/839780Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526009}

Bug: 721395
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I1a304a73965a57bc9dae35004f1ae1e84d61456a
Reviewed-on: https://chromium-review.googlesource.com/798190
Commit-Queue: Randy Smith <rdsmith@chromium.org>
Reviewed-by: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526008}

This CL cleans up the old navigation code from RenderFrameHostManager.

Bug: 789577
Change-Id: Ib7b01df9e7fd3b87361f386725837e4022c4c454
Reviewed-on: https://chromium-review.googlesource.com/840030
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Egor Pasko <pasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526007}

Bug: 
Change-Id: I2d56d62f3635b66c7c4e3dc8882947dbdf0e8f87
Reviewed-on: https://chromium-review.googlesource.com/836748Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Brian Osman <brianosman@google.com>
Cr-Commit-Position: refs/heads/master@{#526006}

https://webrtc.googlesource.com/src.git/+log/1e53f8ad040a..5ec7e1276029

$ git log 1e53f8ad0..5ec7e1276 --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtcBUG=None


The AutoRoll server is located here: https://webrtc-chromium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: I65401df3a5c5dab85c3757cf53cdfab7865bd2dc
Reviewed-on: https://chromium-review.googlesource.com/842107Reviewed-by: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526005}

This adds collections of several "phases" for each process, evenly
spaced. Additional processing will allow to construct a list of startup
only / commonly called functions.

Bug: 758566
Change-Id: I644fc80af42451d62a7d5d71dd0506680de80cae
Reviewed-on: https://chromium-review.googlesource.com/840021
Commit-Queue: Benoit L <lizeb@chromium.org>
Reviewed-by: Matthew Cary <mattcary@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526004}

This CL adds an output bool to the quad-based hit testing APIs to
SurfaceHitTest. When the bool is true, it means that there is reduced
confidence in the hit test result being returned, and the caller should
invoke InputTargetClient to perform an asynchronous hit test in the
renderer before routing a given input event.

Bug: 796648
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I5d5f4bf18977a85c5e4d7f4c4291a4ec89bca70f
Reviewed-on: https://chromium-review.googlesource.com/840644
Commit-Queue: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Lucas Gadani <lfg@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526003}

This patch removes WebTaskRunner::PostDelayedTask() for CrossThread-
Closure. New code must use a new namespace-level PostDelayedCrossThread-
Task() instead. These are functionally identical.

This patch also converts all the existing usage. Since the number of the
use sites is small, it's convenient to do migration and removal together.

Bug: 794845
Change-Id: I8c9deafe1e6fbc6c23c9d38864b97d9396e72320
Reviewed-on: https://chromium-review.googlesource.com/842962Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526002}

In this cl, we make touch-events feature in chrome://flag
default to be disabled.

TouchEvent APIs:
- ontouch* members on window, document, Element
- document.createTouch, document.createTouchList
- document.createEvent("TouchEvent")
will be disabled by default on desktop.

Intent to Deprecate and Remove:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/KV6kqDJpYiE

Bug: 392584
Change-Id: I76716f3c4e529e5143bace95ae1b637b461c2ff5
Reviewed-on: https://chromium-review.googlesource.com/729220Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Navid Zolghadr <nzolghadr@chromium.org>
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Commit-Queue: Ella Ge <eirage@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526001}

Testing "App Paths" specifically isn't done here since handling of it is
a bit tricky (see https://crbug.com/312092). Coverage added by this
change is a whole lot better than none at all.

BUG=730059

Change-Id: I27f34979e1fe5d0c20bc7016577dc921730c57e8
Reviewed-on: https://chromium-review.googlesource.com/842883Reviewed-by: Patrick Monette <pmonette@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#526000}

Bug: chromium:757440
Change-Id: I2d5f2a3e440de770461ea58cd7b2892b645e2caa
Reviewed-on: https://chromium-review.googlesource.com/842982
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#525999}

https://skia.googlesource.com/skia.git/+log/8bc327bffb7a..03a527c718ed

$ git log 8bc327bff..03a527c71 --date=short --no-merges --format='%ad %ae %s'
2017-12-22 mtklein Remove another stray pair of clamps.
2017-12-21 ethannicholas converted YUVEffect to SkSL

Created with:
  roll-dep src/third_party/skia


The AutoRoll server is located here: https://autoroll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=csmartdalton@chromium.org

Change-Id: Iae7150cf3c969368f2d667df5da1d81a4f835293
Reviewed-on: https://chromium-review.googlesource.com/843043Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#525998}

This CL fixes the omnibox perf tests. It was failing because the named
layout guide to position the omnibox popup wasn't added to the test.

Bug: 797282
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I9e1849b4c356379077dfc220a3acdd0e7f9cc789
Reviewed-on: https://chromium-review.googlesource.com/842865
Commit-Queue: Elodie Banel <lod@chromium.org>
Reviewed-by: Mark Cogan <marq@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#525997}

And fix a typo in a test along the way.

Bug: 732003, 796617
Change-Id: I9df6fb630139e1a2534c174c20804111a9189a31
Reviewed-on: https://chromium-review.googlesource.com/841264Reviewed-by: Ian Clelland <iclelland@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#525996}

Right now, if we sign in and click on a previously added address
and then click back instantly, we may have a crash.

The crash could be for two reasons:

1. An async task (subkey request) is calling the EditorDialog.show(),
while the activity is finishing.

2. The EditorDialog.onShow() is called, while the dialog is already
dismissed.

This fix, checks for these conditions before continuing.

Bug: 786962, 794892
Change-Id: I973860fece013c33759c95e178a667d63020ae68
Reviewed-on: https://chromium-review.googlesource.com/841263Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Commit-Queue: Parastoo Geranmayeh <parastoog@google.com>
Cr-Commit-Position: refs/heads/master@{#525995}