GitLab

NOTE: This CL will land after https://chromium-review.googlesource.com/c/chromium/src/+/769330.

This is to expose MessagePort object in AudioWorkletNode and
AudioWorkletProcessor for the bi-directional communication.

Spec: https://webaudio.github.io/web-audio-api/#instantiation-of-AudioWorkletNode-and-AudioWorkletProcessor

Bug: 774564
Change-Id: I543735fc141460a8c356b52e2d461a82d75422fb
Reviewed-on: https://chromium-review.googlesource.com/747621
Commit-Queue: Hongchan Choi <hongchan@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517517}

Document test failures and re-enable passing tests. This brings us to
5200 tests.

Bug: 678687
Change-Id: I800f19093e4e614e69fa6d311d9bb767c9e3e759
Reviewed-on: https://chromium-review.googlesource.com/777546Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517516}

This CL makes ToolbarCoordinator conforms to QRScannerResultLoading and
VoiceSearchControllerDelegate protocols.

Bug: 784911
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I4383e36a97304c7e79b22ee8c5de968d19678744
Reviewed-on: https://chromium-review.googlesource.com/774282
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517515}

This CL adds download_objdump.py to make available binutil tools
that dump and analyze binary files. Currently the main user of
these tools is Supersize.

Bug: 723798
Change-Id: I9427f75d81538835ddb431ca7044398126ba9d11
Reviewed-on: https://chromium-review.googlesource.com/766728Reviewed-by: Peter Collingbourne <pcc@chromium.org>
Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517514}

It is possible that while an input channel is rebound to a new Widget
the old WidgetInputHandler had pending callbacks on it. Since we were
tossing out the mojo pipe we would discard any pending callbacks for it.
Make sure we reset the count and recreate the input router which has
state for pending inputs.

BUG=722928,615090

Change-Id: I34d3ab1af415c97f3ce5ba10ee5c250cf9e30dd1
Reviewed-on: https://chromium-review.googlesource.com/772321Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517513}

Bug: 785185
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I7b432dc42efca64870effc7ff819a8036bb649da
Reviewed-on: https://chromium-review.googlesource.com/775792Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Menglu Huang <huangml@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517512}

With the OOPIF implementation of webview, the embedder page is not
considered focused when the guest has focus (it does not see any
keyboard input). This means that upon reaching the end of the
<webview>'s content and focusing an element in the embedder,
<paper-icon-button> in this case, the page is first refocused.

This event listener only seemed to expect this to happen once at the
beginning.

This CL prevents the authenticator from focusing the webview if another
element is focused when the page itself is focused.

The alternative would be to call focus only once, when constructing the
Authenticator.

Bug: 781748
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Iac7020caad90c9e5e617f46bba392caeb09482b0
Reviewed-on: https://chromium-review.googlesource.com/775978
Commit-Queue: Alex Vallee <avallee@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517511}

When there's an uncaught exception from running our test, the test times out
which is annoying to wait for and it doesn't provide any intermediate output.

This executes our test script in an awaited try-catch block so we can print
the error and end the test early.

This updates our unit tests to follow the async IIFE pattern that the rest of
the new tests uses so it plays nicely with our new error-handling code.

R=dgozman@chromium.org,einbinder@chromium.org

Bug: 667560
Change-Id: I2f693c704ef2991a2970fa925e52422d633bc06c
Reviewed-on: https://chromium-review.googlesource.com/773565
Commit-Queue: Will Chen <chenwilliam@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Joel Einbinder <einbinder@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517510}

https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/8f5568bfc4fd..954710ddd756

$ git log 8f5568bfc..954710ddd --date=short --no-merges --format='%ad %ae %s'
2017-11-15 vtsyrklevich * include/freetype/ftrender.h: Fix `FT_Renderer_RenderFunc' type.
2017-11-14 madigens Use Adobe hinting engine for `light' hinting of both CFF and Type 1.
2017-11-09 yuri_levchenko * CMakeLists.txt: Add `DISABLE_FORCE_DEBUG_PREFIX' option.

Created with:
  roll-dep src/third_party/freetype/src
R=bungeman@chromium.org,drott@chromium.org

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_msan_rel_ng

Bug: 785442
Change-Id: Ib465d6300eb54d0d944e082d8b14b45da8f4bc5f
Reviewed-on: https://chromium-review.googlesource.com/775778Reviewed-by: Ben Wagner <bungeman@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517509}

Bug: 784527
Change-Id: I6f57b53db6f3f898238efa4617e5d8e0bbbfae36
Reviewed-on: https://chromium-review.googlesource.com/776014Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Zhiling Huang <hzl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517508}

BUG=783427

Change-Id: Id1a43e33e83c7cfbfa2a86f82adc286e3c2b0c29
Reviewed-on: https://chromium-review.googlesource.com/775995Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Maria Khomenko <mariakhomenko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517507}

- uncaught error to Cursor::move. Closure should have caught this as a function
  that takes two args; only passed one.

- remove deep equivalent changes in cursor line movement. This causes us to get stuck in various circumstances.

- add AutomationUtil.getLastNode. It computes the last node visited by a predicate. The assumption is that we prefer ancestors that match the predicate. Elsewhere, this is expressed as skipping initial subtree hwne moving forward.

Bug: 784699
Test: existing chromevox_tests.
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ib9713acf4e749cef31c5f816407df0aa45d5f87e
Reviewed-on: https://chromium-review.googlesource.com/775083Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517506}

SetFocus for non-proxy frames are sent input message over mojo.
RenderFrameProxyHost based messages are still sent via
InputMsg_SetFocus chrome IPC since there is no widget input handler
for these situations.

In order to grab the mojo input message we are required to subclass
the RenderWidgetHostImpl to be able to provide a mock interface.

BUG=722928

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_site_isolation
Change-Id: Ieb34b0e1948583c433891edb9bfc5618b03952ad
Reviewed-on: https://chromium-review.googlesource.com/762156
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Nick Carter <nick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517505}

I did about 800 run on Nexus 5x on Android N and 200 run on Pixel on
Android master, all of them passed, so try to reenable it.

Bug: 767180
Change-Id: Ie373a37271aabb5008ac700ff29779f134e5faea
Reviewed-on: https://chromium-review.googlesource.com/776206Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Shimi Zhang <ctzsm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517504}

BUG=783973

Change-Id: I74cdc80d3cd4187ce0741fa722e2527329347ec5
Reviewed-on: https://chromium-review.googlesource.com/770991Reviewed-by: Robert Liao <robliao@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517503}

The "AutomaticTabDiscarding" field trial has expired. Not having
to read the "AllowMultipleDiscards" param of this field trial will
simplify the TabManager refactor.

Bug: 775644
Change-Id: Idaded6bbc74c8ceb02379d5b7ddf43715b552a6e
Reviewed-on: https://chromium-review.googlesource.com/772090
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: Chris Hamilton <chrisha@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517502}

Change "Not now" to "Not yet".

BUG=783829

Change-Id: I9699116ea54d3b6f55b344463328c8b07ff2fcb7
Reviewed-on: https://chromium-review.googlesource.com/777020Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517501}

https://chromium.googlesource.com/chromium/third_party/ffmpeg.git/+log/c99db0ab53a4..1e816bccb5ff

$ git log c99db0ab5..1e816bccb --date=short --no-merges --format='%ad %ae %s'
2017-11-16 dalecurtis Re-add MIPS configuration. It's still used after all.
2017-11-16 gordana.cmiljanovic [MIPS] Put mipsel Linux configuration options

Created with:
  roll-dep src/third_party/ffmpeg

Bug: None
Test: None
Tbr: chcunningham
Change-Id: I2cf10050a3abc16f183982b8e16042fb03e463cc
Reviewed-on: https://chromium-review.googlesource.com/776036
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517500}

Don't need to store the previous_url, NavigationHandle does it for us.

This removes 1 copy on navigation start and 4 copies on commit. It
also simplifies the code a reasonable amount.

Bug: None
Change-Id: I1f19f87a254c6b6fb5147a38da23f66b5db55c01
Reviewed-on: https://chromium-review.googlesource.com/774985Reviewed-by: Martin Šrámek <msramek@chromium.org>
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517499}

Similar adapter observer methods have already been marked deprecated.
GattServiceChanged is not all too different since it is fired when
essentially the device changes shape (attributes are added/removed).

This change also removes a platform-specific use of
gatt_service_changed_count() and gatt_characteristic_added_count().

BUG=710352

Change-Id: Iffe8118083ec10ba762eacb18886bca3acda2d58
Reviewed-on: https://chromium-review.googlesource.com/776354
Commit-Queue: Conley Owens <cco3@chromium.org>
Commit-Queue: Vincent Scheib <scheib@chromium.org>
Reviewed-by: Vincent Scheib <scheib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517498}

This patch fixes PaintInvalidator to use NGPaint code path not only
for LayoutNGBlockFlow but also for LayoutNGMixin.

This fixes inline children in table cells not be painted at all.
~700 tests turn to green.

Bug: 714962
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I36fec7263b2a1e1d06271c9f672f9e0e12b25230
Reviewed-on: https://chromium-review.googlesource.com/776177
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517497}

R=etienneb@chromium.org

Bug: 783296
Change-Id: Ie096d3d407249844a546ec9e17863854fe666c8c
Reviewed-on: https://chromium-review.googlesource.com/776239
Commit-Queue: Abhishek Arya <inferno@chromium.org>
Reviewed-by: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517496}

https://aomedia.googlesource.com/aom.git/+log/08b26a8a257e..535ecf6b31fe

$ git log 08b26a8a2..535ecf6b3 --date=short --no-merges --format='%ad %ae %s'
2017-11-15 jzern av1_txfm,round_shift: remove implicit conv warning
2017-11-13 huisu Fix integer overflows in av1_iidentity*_c()
2017-11-13 huisu Fix int overflow in decode_coefs()

Created with:
  roll-dep src/third_party/libaom/source/libaom

R=jzern@chromium.org

Change-Id: I872329cccab87a81683a8359e2dfde63f2e49bf5
Reviewed-on: https://chromium-review.googlesource.com/776240Reviewed-by: Tom Finegan <tomfinegan@chromium.org>
Commit-Queue: Johann Koenig <johannkoenig@google.com>
Cr-Commit-Position: refs/heads/master@{#517495}

https://webrtc.googlesource.com/src.git/+log/feee08f4da81..12ab00b4d856

$ git log feee08f4d..12ab00b4d --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtc


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
TBR=webrtc-chromium-sheriffs-robots@google.com

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

CMA backends might choose to implement GetCurrentPts in such a way that
it reports the current position to be kStartPts immediately after
Start(kStartPts) is called, rather then after playback actually starts,
and that should be ok.

Bug: none
Test: cast_media_unittests
Change-Id: I4e232504c0a664fb0e715eed96982c2a6d62b2e2
Reviewed-on: https://chromium-review.googlesource.com/775873Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Commit-Queue: Sergey Volk <servolk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517493}

Use TabPersistentStoreObserver.onMetadataSavedAsynchronously to background the Browser Actions tab creation service

Change the logic to background the Browser Actions tab creation service
based on TabPersistentStoreObserver.onMetadataSavedAsynchronously. The
callback indicates that tab model lists are backuped so the saved tabs
can be restored succesfully.

Also for the notification when a new tab has not been created (tabId is
not available), the Intent to bind the notification will be a VIEW
intent. This notification is necessary to make the service foreground.

Bug: 766349
Change-Id: I73d49e2a1e26410b2dd6e51a07f35e9965b9a38a
Reviewed-on: https://chromium-review.googlesource.com/769852Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Lei Tian <ltian@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517492}

This reverts commit 02bddfd9.

Reason for revert: NavigationCallbacksTest.FailedNavigation is failing on devices, and only devices, both iPhones and iPads with iOS 10 and iOS 11.

Original change's description:
> Fixed Navigation callbacks for failed provisional navigation.
> 
> Verified callbacks correctness with
> NavigationCallbacksTest.FailedNavigation integration test.
> 
> Before this CL DidStartNavigation was called twice (second time
> for presenting error in Native Content). DidFinishNavigation was
> called once (also for presenting NativeContent). First
> DidStartNavigation did not have a matching DidStartNavigation
> call.
> 
> This CL removes DidStartNavigation/DidFinishNavigation callbacks
> for for presenting error in Native Content and adds balanced
> DidFinishNavigation callback in handleLoadingError:.
> 
> Bug: 725241
> Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
> Change-Id: Id15ef8c9422e571f74f9c00bf88ea98ccbf6607c
> Reviewed-on: https://chromium-review.googlesource.com/758718
> Commit-Queue: Eugene But <eugenebut@chromium.org>
> Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#517420}

TBR=eugenebut@chromium.org,kkhorimoto@chromium.org

Change-Id: I76bc2987780795fd72284b22203e945387f04c53
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 725241
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/777279Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517491}

Also refactors our proguard wrapper a bit by having it:
 * Ensure all outputs exist before short-circuiting
 * Only touch missing files after building (avoids creating 0-length
   files for a failed run).

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

This fixes a crash where the user could still request a profile reset while
one was already in progress, by closing and re-opening the dialog.

Bug: 616089
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ia99f6ceea4d6f3c1ba797ba47affc42804c3fd5f
Reviewed-on: https://chromium-review.googlesource.com/772939Reviewed-by: Dominic Battré <battre@chromium.org>
Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517489}

TBR=dtrainor@chromium.org

Bug: 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I8e277d2ab135195a2848b07f6bd922fe5dc392c3
Reviewed-on: https://chromium-review.googlesource.com/777032Reviewed-by: Min Qin <qinmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517488}

https://chromium.googlesource.com/angle/angle.git/+log/2c7f34c833b4..3d70ca9cc194

$ git log 2c7f34c83..3d70ca9cc --date=short --no-merges --format='%ad %ae %s'
2017-11-10 oetuaho Remove unreferenced variables from the AST

Created with:
  roll-dep src/third_party/angle


The AutoRoll server is located here: https://angle-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.android:android_optional_gpu_tests_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=ynovikov@chromium.org

Change-Id: I2fe389880ad4ff0c131fff6b21cd668f8d9d8e16
Reviewed-on: https://chromium-review.googlesource.com/776648Reviewed-by: angle-deps-roller . <angle-deps-roller@chromium.org>
Commit-Queue: angle-deps-roller . <angle-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517487}

This fixes a regression where just clicking on text in some
live regions caused it to announce. The problem was that
we rewrote live region event firing (crrev.com/2957973003)
and made it fire an event it any time a node in a live region
was updated, but we need to limit the scope to only:

* A node with text is added
* A node with text is removed
* A node's text changes

Added some new tests and more Windows test expectations
to cover this, and also manually tested this page to
make sure everything there continues to work:

https://minorninth.github.io/aria-live-region-tests/live_region_tests.html

Bug: 785347
Change-Id: Id6fea0f32695234edbbf1493b7dfa40111e5f6a3
Reviewed-on: https://chromium-review.googlesource.com/772290
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517486}

This reverts commit 220c3a64.

Reason for revert: DownloadTaskImplTest.* are failing again on iOS 11 devices:

For example:

DownloadTaskImplTest.FailureInTheBeginning:
../../ios/web/download/download_task_impl_unittest.mm:321: Failure
Value of: session_task
  Actual: false
Expected: true
../../ios/web/download/download_task_impl_unittest.mm:319: Failure
Actual function call count doesn't match EXPECT_CALL(task_observer_, OnDownloadUpdated(task_.get()))...
         Expected: to be called once
           Actual: never called - unsatisfied and active
unknown file: Failure
Uninteresting mock function call - returning directly.
    Function call: OnTaskDestroyed(0x101752170)

Original change's description:
> Reland DownloadTaskImpl implementation for //ios/web Download API.
> 
> This CL implements DownloadTask public interface.
> 
> DownloadControllerImpl CL: crrev.com/c/758525
> Design doc: http://go/ios-web-download-api
> 
> Bug: 780646
> Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
> Change-Id: Iffea44bfb64cc945f7d899dd7181556a730394f1
> Reviewed-on: https://chromium-review.googlesource.com/775889
> Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
> Commit-Queue: Eugene But <eugenebut@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#517424}

TBR=sdefresne@chromium.org,eugenebut@chromium.org

Change-Id: I254a9d8856c29bbe228a2dc96838536dd23e5a35
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 780646
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/777063Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517485}

This is a fix for a regression introduced by this jumbo fix:
    commit b1622826
    Author: Mostyn Bramley-Moore <mostynb@vewd.com>
    Date:   Wed Nov 15 07:05:13 2017 +0000

    [jumbo] avoid ToBrowserAccessibilityAuraLinux redefinition

The change reuses the BrowserAccessibilityAuraLinux AtkAction
casting for BrowserAccessibilityAtkHyperlink. But it is not the
same object, and it will break the hyperlink action support. So
let's provide again a specific casting, but with a name that will
not clash with jumbo build.

Bug: 783447
Change-Id: I319154aadd5aa040a2222d533ca94cb83877068e
Reviewed-on: https://chromium-review.googlesource.com/776778Reviewed-by: Mostyn Bramley-Moore <mostynb@vewd.com>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: José Dapena Paz <jose.dapena@lge.com>
Cr-Commit-Position: refs/heads/master@{#517484}

Original: https://chromium-review.googlesource.com/c/chromium/src/+/642056
Reverted: https://chromium-review.googlesource.com/c/chromium/src/+/688354

Fix just required rebaselining some tests.

For historical reasons, the ARIA role "combobox" is misused
and misunderstood. It can be one of three things:

1. A fully editable text field that also has a drop-down list
   of possible options
2. A button that opens to reveal a mutually-exclusive list of
   choices (but with no text field anywhere)
3. A grouping container with a text field and a drop-down list
   inside

The only platform where this mostly worked was on Windows,
where "combo box" is a native control. On Mac and Android,
we need to treat #1 as a text field primarily, and #2 more
like a button, since there's no equivalent that acts as
both.

Bug: 516869, 666049, 546051
TBR: dtseng, aleventhal, michaelpg
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Idd47a98b396cf8862bde000af43c447d2b9fa921
Reviewed-on: https://chromium-review.googlesource.com/692704Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517483}

Change-Id: Iad4faae02b7a8939b8ede8488bc3239bbec27774
Reviewed-on: https://chromium-review.googlesource.com/763812Reviewed-by: Tom Finegan <tomfinegan@chromium.org>
Commit-Queue: Johann Koenig <johannkoenig@google.com>
Cr-Commit-Position: refs/heads/master@{#517482}

Bitstream conversion must be enabled for the media playback of AAC,
H.264 and HEVC media streams. Mojo renderer does this by default.

Bug: none
Test: cast_media_unittest on Luther now plays AAC files
Change-Id: I4e5cd522a52dc7954a5d187251366ff6ac360d18
Reviewed-on: https://chromium-review.googlesource.com/776217Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Commit-Queue: Sergey Volk <servolk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517481}

BUG=783897

Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Id5ef2b72cd73f2fbdbb49ab2f3dab590a9e3b3d2
Reviewed-on: https://chromium-review.googlesource.com/767153
Commit-Queue: Ryan Hamilton <rch@chromium.org>
Reviewed-by: Buck Krasic <ckrasic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517480}

Check if the sheet is open, layout manager is null, or overview is
visible inside the tracker initialization callback in case state has
changed during tracker intialization.

BUG=786251

Change-Id: I5cbfa00a98f06a02e9145f569c096595243f35cd
Reviewed-on: https://chromium-review.googlesource.com/777119Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517479}

This reverts commit 02bddfd9.

Reason for revert: Breaks device bots tests

Original change's description:
> Fixed Navigation callbacks for failed provisional navigation.
> 
> Verified callbacks correctness with
> NavigationCallbacksTest.FailedNavigation integration test.
> 
> Before this CL DidStartNavigation was called twice (second time
> for presenting error in Native Content). DidFinishNavigation was
> called once (also for presenting NativeContent). First
> DidStartNavigation did not have a matching DidStartNavigation
> call.
> 
> This CL removes DidStartNavigation/DidFinishNavigation callbacks
> for for presenting error in Native Content and adds balanced
> DidFinishNavigation callback in handleLoadingError:.
> 
> Bug: 725241
> Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
> Change-Id: Id15ef8c9422e571f74f9c00bf88ea98ccbf6607c
> Reviewed-on: https://chromium-review.googlesource.com/758718
> Commit-Queue: Eugene But <eugenebut@chromium.org>
> Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#517420}

TBR=eugenebut@chromium.org,kkhorimoto@chromium.org

Change-Id: I6c21310e2f1893fead293ca3808b680a4ae910b0
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 725241
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/777240Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517478}