GitLab

https://skia.googlesource.com/skia.git/+log/1b63b4ac6933..5e6d789ce47c

2020-05-19 egdaniel@google.com Revert "Remove deprecated flush calls."
2020-05-19 egdaniel@google.com Revert "Fix flush change breaks."
2020-05-19 egdaniel@google.com Revert "Implement submit API to GrContext."
2020-05-19 egdaniel@google.com Revert "Another missed flush call fix."
2020-05-19 bungeman@google.com Avoid GDI measuring modes with CBDT fonts.
2020-05-19 bungeman@google.com Note typeface usage in layers.
2020-05-19 johnstiles@google.com Allow --disableDriverCorrectnessWorkarounds to enable MSAA on Intel GPUs
2020-05-19 egdaniel@google.com Another missed flush call fix.
2020-05-19 egdaniel@google.com Implement submit API to GrContext.
2020-05-19 bungeman@google.com Correct TestTypeface::onCharsToGlyphs signature.
2020-05-19 egdaniel@google.com Fix flush change breaks.
2020-05-19 mtklein@google.com remove support for mips64el/loongson3a
2020-05-19 mtklein@google.com port GrDataUtils to use generic transfer functions
2020-05-19 key.draw@gmail.com try adding some code to example.html
2020-05-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skcms from 97bfcc9529c9 to da218e6af725 (1 revision)
2020-05-19 csmartdalton@google.com Reland GrRectanizerPow2
2020-05-19 egdaniel@google.com Remove deprecated flush calls.
2020-05-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll skcms from 60d5d65325e6 to 97bfcc9529c9 (1 revision)
2020-05-19 bungeman@google.com Reland "Cache hb_face."
2020-05-19 jvanverth@google.com Add GrD3DPipelineStateDataManager.
2020-05-19 reed@google.com extract original paths at big size to improve precision

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/skia-autoroll
Please CC lovisolo@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-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
Bug: chromium:527565,chromium:983926
Tbr: lovisolo@google.com
Change-Id: I40d32e2b8166871849d2cf32a4033ae8667744c4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209488Reviewed-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@{#770354}

Previously this was needed because scrolling stacked elements under
a non-stacking-context scroller would otherwise be forced composited
and lose LCD-text if the elements didn't have a opaque background.

After crrev.com/c/2194791 we can squash such layers into the scrolling
contents layer if possible, so composited scrolling can be allowed
without sacrificing LCD text in most cases. In rare cases, a child
stacked element will be composited and lose LCD-text if it can't be
squashed, but this is the same situation as other indirectly composited
layers. It's a bigger win to use composited scrolling than blocking it
just for LCD text of the rare cases.

Bug: 1080056
Change-Id: I182aa2b9030becf684d9292b9246e4b81b90dd2b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2191313Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770353}

TBR=wangxianzhu@chromium.org

Bug: 1084509
Change-Id: If0ba708aeec31682f1852803930935ca24a5fddb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209672Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770352}

Using wpt-import in Chromium 3b8db088.
With Chromium commits locally applied on WPT:
f6984fd9 "De-flaky pointerlock/pointerevent_pointermove_in_pointerlock.html"
74ac20cc "Use original URL before redirects as blocked URL in CSP reporting"
ba055028 "Revert "Reorganize COEP + reporting tests""


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:
rego@igalia.com:
  external/wpt/css/css-grid

NOAUTOREVERT=true
TBR=raphael.kubo.da.costa

No-Export: true
Change-Id: I3bd05698907fdad411a152ae5c2a30b3093e2272
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209510Reviewed-by: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#770351}

Before this patch, only Java PaymentHandlerHost bridge would be able to
access the pointer to the cross-platform PaymentHandlerHost, which
prevented moving more of the payment app factories into C++.

This patch replaces Java
PaymentHandlerHost.getNativePaymentHandlerHost() call with C++
PaymentHandlerHost::FromJavaPaymentHandlerHost().

After this patch, C++ code has the ability to convert a Java
PaymentHandlerHost object into a cross-platform PaymentHandlerHost
object, so more logic can be moved to C++.

Design: https://bit.ly/cross-platform-pay-app-factory

Bug: 1083242
Change-Id: Id56770a205d5dee0941cdb26c61cad8d9ecf2cda
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2205876
Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: Danyao Wang <danyao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770350}

Add a function to the Switch Access end-to-end test base to wait until
a tree change happens, after which a specified predicate function
return true.

This function is not yet used, but will be in a follow-up change.

AX-Relnotes: n/a.
Bug: None
Change-Id: I12802fc50cae80463f1b28adfd573abd0dacfa5b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2203641
Commit-Queue: Anastasia Helfinstein <anastasi@google.com>
Reviewed-by: Katie Dektar <katie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770349}

The beta/stable only definitions were in master-only, and was not
present in commit-queue.cfg of beta. I'm moving these out of
master-only and into the root try.star, such that they are
propagated correctly for m84.

Bug: 1084655
Change-Id: I16dcf48a28cae7302ebdd1288d18b84439b058bc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209373Reviewed-by: Garrett Beaty <gbeaty@chromium.org>
Commit-Queue: Jeff Yoon <jeffyoon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770348}

This reverts commit 0f9b2fd4.

Reason for revert: 
Breaks download stream(clank) compile.
https://logs.chromium.org/logs/chrome/buildbucket/cr-buildbucket.appspot.com/8879834797612365360/+/steps/compile/0/stdout

Original change's description:
> [Web Payment] Cleanup PaymentRequestUpdateEventListener.
> 
> Move PaymentRequestUpdateEventListener from sub-interface in
> PaymentApp.java to its own file.
> 
> Bug: 1083242
> Change-Id: I0d8323b7dc2f75546ca3086897cb642e1e85111d
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2205873
> Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
> Reviewed-by: Sahel Sharify <sahel@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#770319}

TBR=rouslan@chromium.org,sahel@chromium.org

Change-Id: Ic0c26f934562a03768d88cd30692c194e5606816
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1083242
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209512Reviewed-by: Xing Liu <xingliu@chromium.org>
Commit-Queue: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770347}

This reverts commit eb686109.

Reason for revert: Suspecting as the cause for https://ci.chromium.org/p/chromium/builders/ci/linux-archive-rel/29115

Original change's description:
> Update fontconfig cache key handling
> 
> Since If6853404f1e87de4f95a24f6231282543814bd63, the "uuid" variable has
> been referring to the MD5 of a fixed string. Update the comment as such,
> and remove the now-unneeded .uuid file writing.
> 
> Also update test_fonts/README.chromium as the manual UUID change is no
> longer necessary or effective.
> 
> Bug: 767754
> Change-Id: I07a9b10a631e755877cbcb7df40a53359329f8a3
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209397
> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
> Auto-Submit: Timothy Gu <timothygu@chromium.org>
> Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#770287}

TBR=thomasanderson@chromium.org,timothygu@chromium.org

Change-Id: I7cc1c702d84a1c1a82933ae104d9d0e7fb4dcf5c
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 767754
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208322Reviewed-by: Darren Shen <shend@chromium.org>
Commit-Queue: Darren Shen <shend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770346}

StartSurfaceTest.testShow_TasksOnly and
StartSurfaceTest.testShow_TwoPanes are still flaky on Android M bots,
not only on Nexus5x(bullhead).

testScroll_OmniboxOnly_Pinned is failing on Android P as well.

TBR=wychen@chromium.org

Bug: 1081822,1084176,1083459
Change-Id: Ic52dc5527f2962773d15cc0ec7f88de891fa56aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209302
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770345}

This change generalizes the RootNodeWrapper.findChildren() function
so it can be used in tests (in a follow-up change).

It also changes all references to chrome.automation.AutomationNode to
use the abbreviated version of the name (AutomationNode), and moves the
declaration of that abbreviation into the constants file.

AX-Relnotes: n/a.
Bug: None
Change-Id: I8061ea8f2cec1587ae3b73e4f85ccc57b0f3de1b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208177
Commit-Queue: Anastasia Helfinstein <anastasi@google.com>
Reviewed-by: Akihiro Ota <akihiroota@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770344}

Add a check to ensure that DoH is enabled correctly, and not turned
off due to unhandled configurations (e.g. domain-joined Windows
machine, or VPN configuration). The existing navigation check passes
even when DoH is forced-disabled, and is therefore insufficient.

Bug: 1032710
Change-Id: Idf20f8c4ce1b20c5017ce8693ab5bc57b77b7bac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2200156
Commit-Queue: Kaustubha Govind <kaustubhag@chromium.org>
Reviewed-by: Eric Orth <ericorth@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770343}

https://webrtc.googlesource.com/src.git/+log/2671dac29c27..4ab61cb9b449

2020-05-19 sprang@webrtc.org Optionally allows TaskQueuePacedSender to coalesce send events.

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/webrtc-chromium-autoroll
Please CC webrtc-chromium-sheriffs-robots@google.com on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

Bug: None
Tbr: webrtc-chromium-sheriffs-robots@google.com
Change-Id: Ib9b110ca8f1263d4133d3eca60edb8d787d850fb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209552Reviewed-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@{#770342}

The a11y label changes between "Hide the 'X' section" and
"Show the 'X' section" depending on the suggestion group visibility where
X is the suggestion group header (e.g., 'Recommended for you').

The tooltip changes between "Hide Suggestions" and "Show Suggestions"
depending on the suggestion group visibility.

Bug: 1052514
Change-Id: Ib7bce19b8d203e57b9b916fbcc089a58593bdd68
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2206222
Commit-Queue: Moe Ahmadi <mahmadi@chromium.org>
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: manuk hovanesian <manukh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770341}

No change to logic. Remove an extra 's' from the template and reword
webview package field.

Bug: 1077783
Test: bin/run_webview_instrumentation_test_apk -f "*CrashBugUrlFactoryTest*"
Change-Id: I3ab221d822611d230a8d91e9af22d0b68b1ba507
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207222
Commit-Queue: Nate Fischer <ntfschr@chromium.org>
Reviewed-by: Nate Fischer <ntfschr@chromium.org>
Auto-Submit: Hazem Ashmawy <hazems@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770340}

TopIconAnimationView

Bug: 1044687
Change-Id: Ifdaf646043e5e86d593ddf23aed49fbc6d60f3ea
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207733
Commit-Queue: Allen Bauer <kylixrd@chromium.org>
Reviewed-by: Robert Liao <robliao@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770339}

Generated by //tools/gn/roll_gn.py

https://gn.googlesource.com/gn/+log/ab32747a..d0a6f072

    d0a6f072 Add weak_frameworks linking option
    3cbdb628 Add xcasset_compiler_flags variable to create_bundle target

Bug: chromium:1082896
Change-Id: I62ba729a2aaf53206fd6f3cb450dadd44745d1d0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209333
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770338}

Prelude to removal of WithSandboxType API. See linked bug for
rationale.  Does not change behavior.

Bug: 1065087
Change-Id: Idf559fdb44076b11219f17f233c81ae6966fe2aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2203140
Commit-Queue: Alex Gough <ajgo@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770337}

The Shill algorithm for identifying metered networks is slightly more
accurate than the Chrome logic, and has the potential to improve
further in the future.

There is also the possibility of allowing users to override this
property, in which case we will want to use the override value.

Bug: 1078427
Change-Id: I85ad477a27e981be45ec77e8d060eb299bc78065
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2186112
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Azeem Arshad <azeemarshad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770336}

See (future) Skia CL https://skia-review.googlesource.com/c/skia/+/290831

Change-Id: I43ec4dcc468a5bb9b21cd5022d04b9fb84e2e85b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209492Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Mike Reed <reed@google.com>
Cr-Commit-Position: refs/heads/master@{#770335}

Bug: 1068730
Change-Id: I1459d2ec69738b4c6dfca661fe6fd6a38295706b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2202392Reviewed-by: Theresa  <twellington@chromium.org>
Commit-Queue: Sinan Sahin <sinansahin@google.com>
Cr-Commit-Position: refs/heads/master@{#770334}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/002f97b35a72..620c56e90e03

2020-05-19 zhin@google.com Fix git cl completion

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/depot-tools-chromium-autoroll
Please CC agable@chromium.org on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

Bug: None
Tbr: agable@chromium.org
Change-Id: I0fe72de07d2ce0ac3b5f2f471b7f20fd359b9f29
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209554Reviewed-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@{#770333}

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll
Please CC cr-fuchsia+bot@chromium.org on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

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

Tbr: cr-fuchsia+bot@chromium.org
Change-Id: Ia65f820a02a6eec7c01e0d4bfc05ccb1b81ef7c3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209489Reviewed-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@{#770332}

NewTabPage.LinkOpenedFromContextMenu.WebUI is the WebUI equivalent of
NTP_LinkOpenedFromContextMenu and MostVisited_ClickedFromContextMenu on
the local NTP.

+ Update ownership of the three aforementioned user actions.

Bug: 1074060
Change-Id: I59e412ad48894cefc1d7bc3dd599ee45c072d62d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208183
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Commit-Queue: Ilya Sherman <isherman@chromium.org>
Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770331}

Webdriver standard has updated handling for contenteditable,
specifically, only for element that was not focuses would
we move the caret to the end of children.

Modified test to reflect

Bug: chromedriver:3214
Change-Id: I123a7cb3569ad82c3c30d180abfdeaa3b0c8204c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2203288Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: Shengfa Lin <shengfa@google.com>
Cr-Commit-Position: refs/heads/master@{#770330}

Bug: 1079177
Change-Id: I99fa1ae741d13ea808e0a92c119de042a72e5cb6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2207453Reviewed-by: Sergey Poromov <poromov@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Nikita Podguzov <nikitapodguzov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770329}

This test was flaky, especially on the Mac bots, where timers can often fire
with varying precision.

SensorProxy uses an ObserverSet to store Sensor instances that need to be
notified of a reading, so in some cases we could end up with the following
sequence of notifications in C++ and JavaScript:
1. OnSensorReadingChanged() is called for the fast sensor (running at either
   10Hz or 60Hz).
2. A "reading" event is delivered to the fast sensor's event handler in
   script.
3. OnSensorReadingChanged() is called for the slow sensor (running at 1/4th
   of the fast sensor's frequency).
4. A "reading" event is delivered to the slow sensor's event handler in
   script.
5. OnSensorReadingChanged() is called for the slow sensor again, but it is
   too early so it posts a delayed task to call NotifyReading().
6. OnSensorReadingChanged() is called for the fast sensor.
7. A "reading" event is delivered to the fast sensor's event handler in
   script.
8. The delayed task is executed, and a "reading" event is delivered to the
   slow sensor's event handler in script.

At this point, both |slowSensorNotifierCounter| and
|fastSensorNotifierCounter| are set to 2, and the assertion in the test
fails.

Fix it by waiting for another "reading" event to be delivered to the slow
sensor. Given the frequencies we choose, at this point a working
implementation's |fastSensorNotifierCounter| will definitely be higher than
2.

The downside is that we slow down the test by the value of another sensor
period -- for regular sensors, that is around 0.016s, while for slower
sensors such as AmbientLightSensor and Magnetometer (capped at 10Hz) this is
a 0.4s slowdown.

Bug: 731018
Change-Id: I249ab88eccf316888177a93614c2955b516bc835
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2204023
Auto-Submit: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770328}

Delete BaseCustomTabActivityTabController now that
CustomTabActivityTabController is the only subclass.

BUG=985328

Change-Id: I80599887f9f5cb1d04ce8272d5f1812e8bf907c4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2205593
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: Peter Conn <peconn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770327}

Ensures that if we change decoders in the future this feature still
works.

R=tguilbert

Bug: 1083840
Test: It's all tests!
Change-Id: Icd388953554fa167887718b76469a5003244ca10
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209401Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770326}

The old search tag was "Privacy verified access", but it really should
have just been "Verified access". The setting was in the privacy section
of settings, which led this this confusing name.

Bug: b/156024668
Change-Id: I684cd966f1dd4b01abde81b1da66289de13ad69a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209530
Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
Commit-Queue: Regan Hsu <hsuregan@chromium.org>
Auto-Submit: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Regan Hsu <hsuregan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770325}

This CL cleans up many calls to
  EmbeddedTestServer::AddDefaultHandlers(const base::FilePath&)
in //net/url_request/url_request_unittest.cc, where the test only needed
to add the default request handlers and did not need to serve files from
a given directory.

These are replaced with calls to
  test_server::RegisterDefaultHandlers(EmbeddedTestServer*)

Bug: 833625
Change-Id: I6d9128ce58510d5d5674dc5e4d084dd768828ac4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209409Reviewed-by: Nick Harper <nharper@chromium.org>
Commit-Queue: Lily Chen <chlily@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770324}

The HasBluetoothStack() method checks if bthprops.cpl can be loaded.
This can take the dynamic library loader lock while on a background
thread and so an annotation is needed to temporarily boost thread
priority to avoid janking the main thread.

Noticed in crash/e8feafd49cc404d4.

Bug: 973868
Change-Id: I036ffacd5f22afa9a5493d746b2b1ada600ecfc6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209133
Auto-Submit: Reilly Grant <reillyg@chromium.org>
Commit-Queue: Vincent Scheib <scheib@chromium.org>
Reviewed-by: Vincent Scheib <scheib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770323}

This is a reland of 2a23cb68

The build failure was due to an ash header being included in a
non-ChromeOS build. This managed to build correctly until my CL.
The reland makes the include conditional on building for ChromeOS.

TBR=oshima@chromium.org

Original change's description:
> Scale source window during tab drag-and-drop
>
> This behavior is very similar to tablet mode tab drags from the native
> tab strip. Some of this code is copied, and some is shared (through
> TabletModeBrowserWindowDragSessionWindowsHider).
>
> Bug: 1069869
> Change-Id: If5c6f163f7d8af9918be3bc97ca8a91b1df063c1
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2202507
> Commit-Queue: Collin Baker <collinbaker@chromium.org>
> Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
> Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#770272}

Bug: 1069869
Change-Id: I02f6becf72fac663a744d2033e16e2fd14dce843
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209384Reviewed-by: Collin Baker <collinbaker@chromium.org>
Commit-Queue: Collin Baker <collinbaker@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770322}

Syncs media_app.externs.js with cl/312200889.

Annoyingly closure doesn't let fromClipboard remain undefined, so make
it explicitly false.

Bug: b/134975059
Change-Id: I6c0fb8fbc91c5dfce52a4ec3cce92467869c9296
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2206771Reviewed-by: Patti <patricialor@chromium.org>
Commit-Queue: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770321}

Bug: 1000989
Change-Id: Ia607dc32e46ab88d88f889feef873189bad5ea3e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209290
Commit-Queue: John Lee <johntlee@chromium.org>
Auto-Submit: dpapad <dpapad@chromium.org>
Reviewed-by: John Lee <johntlee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770320}

Move PaymentRequestUpdateEventListener from sub-interface in
PaymentApp.java to its own file.

Bug: 1083242
Change-Id: I0d8323b7dc2f75546ca3086897cb642e1e85111d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2205873
Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: Sahel Sharify <sahel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770319}

Bug: 1044196
Change-Id: Ie4a96b06ee8169bee76e07c79334cbc6d34a78aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2209345Reviewed-by: Dan H <harringtond@chromium.org>
Commit-Queue: Alexandre Frechette <frechette@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770318}

Adds a method that allows for setting a new port value directly from
a string. Also add missing validation for the old SetPort(int) method.

BUG=none
TEST=on my laptop, a dedicated unit test was added

Change-Id: I3adc98928ef7e136619cbce4184dff1d1dc11e47
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2208650
Commit-Queue: Piotr Pawliczek <pawliczek@chromium.org>
Reviewed-by: Sean Kau <skau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770317}

Makes it so we only jetify libraries that specifically
request it. Previously we jetified all libraries and let them opt out.
Now everything is opted out unless requested to be jetified.
This will decrease build time.

before change building chrome_public_apk:
user	118m19.720s
sys	12m29.984s

after change building chrome_public_apk:
user	112m5.589s
sys	11m25.513s


Bug: 1069613
Change-Id: I20bbef5cb1bad9c84fc53272af6a0835692f586c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2172133
Commit-Queue: benjamin joyce <bjoyce@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: benjamin joyce <bjoyce@chromium.org>
Cr-Commit-Position: refs/heads/master@{#770316}

There are a lot of occurrences of c-style casts and other minor warnings
that clutter my error-jumping keybinding. This CL removes them, and
additionally introduces an implementation of operator<<(std::ostream&)
for media::HTMLMediaElement that just outputs its memory address, since
that already appears to be the de facto way of printing it.

Change-Id: Idaa763ceed878835fa3e7661c0c85355ba3ef42d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2183289Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Will Cassella <cassew@google.com>
Cr-Commit-Position: refs/heads/master@{#770315}