GitLab

This delegate serves two purposes:
1. It severs unwanted coupling between StartSurface/tasks and
LocationBar via ReturnToChromeExperimentsUtil
2. It enables us to supply the parent tab via glue code in
ToolbarManager rather than leaking this implementation detail
into FakeboxDelegate

Bug: 1119256

Change-Id: I0b161df67bd94e3f3fa9c9dbfda0cfcaf0ba05da
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533407
Commit-Queue: Patrick Noland <pnoland@chromium.org>
Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Reviewed-by: Yue Zhang <yuezhanggg@chromium.org>
Reviewed-by: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827039}

This CL aims to add polymorphism to the readable stream controller
classes. In particular, it adds a new base class called
ReadableStreamController, and the ReadableStreamDefaultController and
ReadableByteStreamController classes inherit PullSteps() and
CancelSteps() from the ReadableStreamController.

This is based on the approach discussed in
https://docs.google.com/document/d/1rvKpGjppeqRSWntokY-ft_hU2i2us8gis6zgrrLWSTI/view

Bug: 614302
Change-Id: I58439e52e43b525cadbb1a9d36dd950005a7c32c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531794
Commit-Queue: Nidhi Jaju <nidhijaju@google.com>
Reviewed-by: Adam Rice <ricea@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827038}

- move intent generation to the private repository.
- remove extra Lens Prime class dependencies from Context Menu.
- simplify the displaying and onClick handling logic for Lens chip.

Part I can be found at:
https://chromium-review.googlesource.com/c/chromium/src/+/2527724

Change-Id: I26516b1bc39d488922e963269da3114aaa5e998a
Bug: 1145748, b/171252338
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2503636
Commit-Queue: Yu Su <yusuyoutube@google.com>
Reviewed-by: Theresa  <twellington@chromium.org>
Reviewed-by: Ben Goldberger <benwgold@google.com>
Reviewed-by: Sinan Sahin <sinansahin@google.com>
Cr-Commit-Position: refs/heads/master@{#827037}

Text fragments are allowed only if a text fragment token has been
generated during loading. The token is effectively a navigation-time
user activation flag. To allow user initiated text fragment navigations
during a same document navigation, a token may be generated in
DocumentLoader::UpdateForSameDocumentNavigation

However, this method is called in a variety of other scenarios, such as
calls to history.replaceState or portal navigations. In those cases, we
would try to generate a new token, fail, and then clobber an existing
token, causing pages like in the linked bug to fail.

This CL makes it so that we only replace an existing token if the
navigation is a standard navigation.

Bug: 1147453
Change-Id: I2bad963884068d2de23833ffce37086fda9a034e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533688Reviewed-by: Nick Burris <nburris@chromium.org>
Commit-Queue: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827036}

https://chromium.googlesource.com/external/github.com/AOMediaCodec/libavif.git/+log/5fa7c4e22c1b..4eb2d0b58ecc

$ git log 5fa7c4e22..4eb2d0b58 --date=short --no-merges --format='%ad %ae %s'
2020-11-11 joedrago Fix comment typo and add parens around macro value
2020-11-11 jdrago Add avifDecoderNthImageReady() avifIO helper function
2020-11-11 jdrago Update changelog
2020-11-11 jdrago Expose maxThreads to avifDecoder, add --jobs to avifdec
2020-11-09 joedrago Minor rewording
2020-11-09 joedrago Create AVIF_VERSION_DEVEL, which allows for in-development changes to be ifdef'd in downstream projects
2020-11-09 joedrago v0.8.3
2020-11-07 wtc Reorder the offset checks in avifIOTestReaderRead
2020-11-09 joedrago Reword avifIOReadFunc comments for clarity (wantehchang)
2020-11-08 joedrago Allow checking the upsampling filter independent of subsampling choice to allow for a means for endusers to opt-out of libyuv, even if it is compiled in
2020-11-07 joedrago Remove misleading comment snippet about libyuv fast paths

Created with:
  roll-dep src/third_party/libavif/src
R=dalecurtis@chromium.org

Bug: 1136922
Change-Id: I832341811c78d8698f75dafa1d3be8e76cb31e24
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2536034Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Wan-Teh Chang <wtc@google.com>
Cr-Commit-Position: refs/heads/master@{#827035}

This CL aims to add polymorphism to the readable stream reader
classes. In particular, it makes the ReadableStreamGenericReader
the base class, and the ReadableStreamDefaultReader and
ReadableStreamBYOBReader classes inherit from the GenericReader.

This is based on the approach discussed in
https://docs.google.com/document/d/1rvKpGjppeqRSWntokY-ft_hU2i2us8gis6zgrrLWSTI/view

Bug: 614302
Change-Id: I4dde2976938325c21aeb87c6980ac4ad1def483f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2528557Reviewed-by: Adam Rice <ricea@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Nidhi Jaju <nidhijaju@google.com>
Cr-Commit-Position: refs/heads/master@{#827034}

Previously we were only recording the time of the user's first pin to
holding space. Now we will record the time when the first item of any
type was added to holding space.

This should help determine how discoverable the feature is.

Bug: 1131266
Change-Id: Iacee3d00345fa2860b34d7695d92bf3e0bf91485
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2530751Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Toni Baržić <tbarzic@chromium.org>
Commit-Queue: David Black <dmblack@google.com>
Cr-Commit-Position: refs/heads/master@{#827033}

Add support for an allFrames boolean in the scripting.executeScript()
API. If specified, the code is executed in all frames within the tab
(that the extension has permission to access).

Bug: 1144839
Change-Id: Idecc69cf9599dfc821e2d309078324a6e34b0a59
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532901Reviewed-by: David Bertoni <dbertoni@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827032}

Update |hidden| argument calculations as follows:
 - Remove GetError() check, because same check was made earlier in the
   method and results in early return
 - Remove "is main frame" check as unnecessary, because
   DidFinishNavigation is not called for iframe
 - Remove navigation_context->GetResponseHeaders()->response_code() > 600
   check, as it looks like a mistake which overrides correct comparison
   to status code 400 (range of HTTP status codes is [100, 599]).

Bug: 924298
Change-Id: I0dc570d34ef6aed0654f7d0742190f425b2ac966
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535017
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Auto-Submit: Eugene But <eugenebut@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827031}

Currently, GUIDs are returned as and stored in std::string. A dedicated
base::GUID class would introduce greater type safety.

Mark existing GUID functions as deprecated.

Add tests for the new class.

Bug: 1026195
Change-Id: I15d0c8a5907291d48d2e1ddf029390170e81993b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2200456
Commit-Queue: Daniel Hosseinian <dhoss@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827030}

In this CL, we do two changes:
1. Add a start surface check in
ToolbarManager#ToolbarNtpDelegate#isLocationBarShown.

Before[1], this returns true even if it's in start surface v2 homepage,
that causes the ToolbarPhone[2] set background color transparent[3].
[1] Demo:
https://drive.google.com/file/d/1dPJwI4lK-9ch5Wwnx06oY9IIXenLIGzr/view?usp=sharing
[2] https://source.chromium.org/chromium/chromium/src/+/master:chrome/android/java/src/org/chromium/chrome/browser/toolbar/top/ToolbarPhone.java;drc=5b8933e94139a0ab5be46141666fdfcce0f624f6;l=2366
[3] https://source.chromium.org/chromium/chromium/src/+/master:chrome/android/java/src/org/chromium/chrome/browser/toolbar/top/ToolbarPhone.java;drc=5b8933e94139a0ab5be46141666fdfcce0f624f6;l=712

2. Add a shrink animation for omnibox when it's about to be scrolled up
and toolbar container is showing.

Please see demo (The left one is the fixed one):
https://drive.google.com/file/d/1jRfYGx9lD3UfxaTabhaNe9Cs_YWTJt2J/view?usp=sharing

Bug: 1113852, 1136595
Change-Id: I549223c7a8b7c2e2f56ab62e0d0a090e1185fc96
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2517544
Commit-Queue: Hao Dong <spdonghao@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Reviewed-by: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Reviewed-by: Xi Han <hanxi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827029}

This is an attempt to fix the error where mac builder
is not able to find file
/opt/s/w/ir/cache/builder/src/out/Release/gen/chrome/updater/version_info.py.


Bug: 1148430
Change-Id: I671f32660913c03f0fa61c993e4fcb027c00f3c4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535220Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Anjali Doneria <adoneria@google.com>
Cr-Commit-Position: refs/heads/master@{#827028}

Refactor only change.

Move data conversions into FileHelper where path translation and file
sharing for VMs can be done.

The current code assumes only Arc is using this code, but we will soon
support crostini and pluginvm.

FileHelper::GetFilenames receives a list of files from the source window
and returns a list of ui::FileInfo.  To support this,
DragDropOperation::GetFilenames has been refactored to take FileHelper
and aura::Window*.

FileHelper::GetMimeTypeForUriList now takes target window since the mime
type will be different based on target.

FileHelper::GetUrlFromPath changed to SendFileInfo which takes a
callback to run when data is ready. The async callback allows files to
be shared with a VM before sending the data.

FileHelper::GetUrlsFromPickle changed to SendPickle which also takes a
callback to run when data is ready.

Bug: 1144138
Change-Id: I0356d4681705df5be174ac1debff51f4903eb742
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531087
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827027}

This is a reland of 74dbe648

This CL was reverted because of a flaky test in TabSearchAppTest.
The test 'Verify ShowUI' was originally flaky and is now disabled with:
https://chromium-review.googlesource.com/c/chromium/src/+/2533325
This is no longer an issue since this test is disabled.

Original change's description:
> Tab Search: Infinite list component
>
> A list component that renders DOM items on demand based on the viewable
> scroll area and user navigation and scrolling interactions. Provides a
> performance gain for use cases where users have several dozens of items
> that should be available while scrolling the list by deferring scripting
> and rendering costs until necessary to fill the current scroll view.
>
> Bug: 1099917
> Change-Id: Ib22c0edf5ebc5febe3d8d6d4f7d8ac319e9bd0a2
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2503401
> Commit-Queue: Roman Arora <romanarora@google.com>
> Reviewed-by: Thomas Lukaszewicz <tluk@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#826287}

Bug: 1099917
Change-Id: Ief4a032b3e66957576e5ce3677159345d1de9c52
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535013
Commit-Queue: Roman Arora <romanarora@chromium.org>
Reviewed-by: Thomas Lukaszewicz <tluk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827026}

Also removes "Allow copy" finch param.  Code now uses "enabled" path.

Bug: 1147960, 1022246
Change-Id: Ie3e7fcb85b2ed2a4ef67e41b0777aec137521bf5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533039
Commit-Queue: Ted Meyer <tmathmeyer@chromium.org>
Auto-Submit: Frank Liberato <liberato@chromium.org>
Reviewed-by: Ted Meyer <tmathmeyer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827025}

Change-Id: Ibb6f8213ef0892885cdcad3a3dd4d04e111a1edd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535797Reviewed-by: Alex Gough <ajgo@chromium.org>
Commit-Queue: Will Harris <wfh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827024}

The linked bug occurs, for both text and element fragments, because we
try to avoid scrolling to fragments when the scroll offset will be
restored by history. Manual scroll restoration is a web API that allows
the page to override the scroll restoration, we treat it separately but
want to avoid scrolling the fragment in this case too.

However, the special case for manual restoration was missing a check
that the navigation was actually a history navigation. Without this
check, an ordinary (non-history) page load that sets manual restoration
before the fragment is invoked will prevent it from scrolling.

Bug: 1147568
Change-Id: I83e1a3cc12e9f40243c42ee21c52fef2acb36993
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533149Reviewed-by: Nick Burris <nburris@chromium.org>
Commit-Queue: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827023}

The current sharding algorithm calculates the expected shard time and stick to it. When a test has a long runtime, it could cause its sibling shard to be empty.

Updating the algorithm to always update the expect time for the remaining after each shard. Also will ensure each shard should have at least one test.

Bug: chromium:1130157
Change-Id: I3c335cfc58e35b48e5fd374c9d7546184cd68c56
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2481006Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: Wenbin Zhang <wenbinzhang@google.com>
Cr-Commit-Position: refs/heads/master@{#827022}

There seems to be a change in M87 unrelated to the CrOS MediaApp
which broke PDF loading. I think this was due to tighter CSP.

This change sets our CSP so we can load PDFs as blob urls in our
untrusted context and adds a test for coverage of the "object-src"
policy.

Further work to make our tests better & provide coverage for "frame-src"
is captured in crbug/1148090.

Bug: b/172881869, 1148090
Change-Id: I6e5ea0816c7f8c5a4eb382ed376e4fd98f9f671c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2531075Reviewed-by: dstockwell <dstockwell@chromium.org>
Reviewed-by: David Lei <dlei@google.com>
Reviewed-by: Patti <patricialor@chromium.org>
Commit-Queue: David Lei <dlei@google.com>
Commit-Queue: dstockwell <dstockwell@chromium.org>
Auto-Submit: David Lei <dlei@google.com>
Cr-Commit-Position: refs/heads/master@{#827021}

- Swap show and hide animation interpolators
- Delay the translation animation by 100ms

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

This is a reland of bce662eb

It was revert in crrev.com/c/2530204, to make room for other builders.
The host pool looks in a good shape today, so reland this CL
and prepare to fully enable the MM-x86-rel on CQ

Original change's description:
> luci: increase the experimental percentage to 100 from 70 for MM-x86-rel
>
> The pool has plenty of 4-cores for extra builds: http://shortn/_koR1dg8Vvu
> Also the host pool also has plenty of rooms: http://shortn/_xqhW5Z67ft
>
> Bug: 997376, 1127110
> Change-Id: I56b432d960c2f1821934667ac2f38d3fcb4705bf
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2450331
> Reviewed-by: Ben Pastene <bpastene@chromium.org>
> Commit-Queue: Haiyang Pan <hypan@google.com>
> Cr-Commit-Position: refs/heads/master@{#813888}

Bug: 997376
Bug: 1127110
Change-Id: I652429f328af4cb11fe22efc6286af832d77b43b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535641Reviewed-by: Garrett Beaty <gbeaty@chromium.org>
Commit-Queue: Haiyang Pan <hypan@google.com>
Cr-Commit-Position: refs/heads/master@{#827019}

This crash can occur if, e.g. there are no new tabs open and DWB
informs us that a site is suspended. This CL also adds a unit
test for this case and two instrumentation tests; one for this case
and one for a related one.

Bug: 1148363
Change-Id: I7fad393bd0369c4650fb7b8c2d8bd3c3cde40bed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535912Reviewed-by: Natalie Chouinard <chouinard@chromium.org>
Commit-Queue: Patrick Noland <pnoland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827018}

TBR=peconn@chromium.org

Bug: 1148493
Change-Id: I5ea9b0aab0cc0d069fea5f6f19a04712dd99eca8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2536080Reviewed-by: Brandon Wylie <wylieb@chromium.org>
Commit-Queue: Brandon Wylie <wylieb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827017}

This CL displays an error in chrome://extensions page when service
worker registrations fails, as opposed to calling DCHECK(false).

Bug: 1136582
Change-Id: Ie1619a922966d1b2e16ca316dc8f5dae8d448616
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2505602
Commit-Queue: Ghazale Hosseinabadi <ghazale@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827016}

The test can end before the video's first frames are painted. This CL
attempts to fix the issue by making sure we have a frame per video
before ending.

Bug: 963141
Change-Id: Iccc9f771f4e7e366914b00b3a6004a08cf9829b1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2225372
Commit-Queue: Thomas Guilbert <tguilbert@chromium.org>
Reviewed-by: Tommy Steimel <steimel@chromium.org>
Auto-Submit: Thomas Guilbert <tguilbert@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827015}

https://quiche.googlesource.com/quiche.git/+log/442f894c74b5..54fc9abd737d

$ git log 442f894c7..54fc9abd7 --date=short --no-merges --format='%ad %ae %s'
2020-11-12 nharper Add out_alert to ProofVerifier::VerifyCertChain
2020-11-12 bnc Fix QuicSpdySessionTestServer.SendHttp3GoAway.

Created with:
  roll-dep src/net/third_party/quiche/src src/third_party/quic_trace/src

Change-Id: If4b7a3415b433cc4e648b49d0c966c98b3f2cef9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535690
Commit-Queue: Nick Harper <nharper@chromium.org>
Commit-Queue: David Schinazi <dschinazi@chromium.org>
Auto-Submit: Nick Harper <nharper@chromium.org>
Reviewed-by: David Schinazi <dschinazi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827014}

Bug: 1126778
Change-Id: Icbf0745c97febdb4a640cc401f7fd2d625fe207a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2536074Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Shimi Zhang <ctzsm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827013}

Enabling for the default channel only to avoid including the change in beta when branch point is close. After branch, both enable_jdk_library_desugaring and enable_feedv2_modern should be unconditionally true.

Bug: 1115463
Change-Id: I693a3f79f94280887f4d7e85711e6fe7e50ee8d8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2519931Reviewed-by: Justin DeWitt <dewittj@chromium.org>
Commit-Queue: Ian Wells <iwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827012}

If the module visibility state is controlled by a policy the user is not
able to set module visibility. We communicate that by graying out the
toggle and showing a business icon explain the situation.

Toggle off: screenshot/8dUeEADe6jyvyLa
Toggle on: screenshot/K4WGhYvGnESmLYF

Bug: 1142878
Change-Id: Id3af801552c6c01a2bf3611233c08de892393469
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533581
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Reviewed-by: Esmael Elmoslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827011}

The Lighting Estimation API has stabilized a little bit for the time
being; this change updates the test page with the API changes.

Change-Id: I770e0bb0e70e28e7373dbc650c5332247f6bb30d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532772Reviewed-by: Brandon Jones <bajones@chromium.org>
Commit-Queue: Alexander Cooper <alcooper@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827010}

TBR=wychen@chromium.org

Bug: 1148365
Change-Id: I853864863384398c02832d415e2e0c4affa8fb84
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2536079Reviewed-by: Brandon Wylie <wylieb@chromium.org>
Commit-Queue: Brandon Wylie <wylieb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827009}

BUG=chrome:1148422
TEST=fuzzer test case now passes

Change-Id: I1d466d470b827249f5884cca8db9e61bdd291563
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2534153
Auto-Submit: Jeffrey Kardatzke <jkardatzke@google.com>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Sergey Volk <servolk@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Sergey Volk <servolk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827008}

This CL removed the periods in the strings of reverse
gesture toasts

Bug: 1107183
Change-Id: Ia2196d08443a4ecc08a59e1680388eb74b2c3543
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535913Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Xiaodan Zhu <zxdan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827007}

This allows for the use of portals with the
--enable-experimental-web-platform-features flag. Note that this is the
same configuration as the M85 origin trial: same-origin, Android only
(see issue 1040212). We'll hold off using this flag for enabling for
desktop and/or cross origin cases until those are further developed.

Bug: None
Change-Id: I0579c241c58aec3e37e5dd7fba4e8375d43d91a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2518613Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827006}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/f86e9704c66b..696f7382ade0

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll
Please CC zmin@google.com,dbertoni@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.chrome.try:linux-chromeos-chrome
Bug: chromium:1147077
Tbr: zmin@google.com,dbertoni@google.com
Change-Id: I4d7799190c714c00c863c21c5a0d6d482e3009f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535991Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#827005}

Made `RecordParsed::rdata_` nullable but missed a deref in
`RecordParsed::IsEqual()`.

Original change's description:
> Revert "Ignore unknown record types in DNS responses"
>
> This reverts commit e4620991.
>
> Reason for revert: Suspected for causing crashes on Windows/Intel.
> For more info see bug.
>
> Bug: 1148269
>
> Original change's description:
> > Ignore unknown record types in DNS responses
> >
> > Previously worked only for address requests. Other request types (e.g.
> > HTTPS), on seeing any unrecognized (and therefore unparsable) record
> > type in the response, would fail the whole response as malformed.
> >
> > Bug: 1147247
> > Change-Id: Icd9e5ab2e39cf09ffe7c3945b011ebf1d4b45778
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2529619
> > Commit-Queue: Eric Orth <ericorth@chromium.org>
> > Auto-Submit: Eric Orth <ericorth@chromium.org>
> > Reviewed-by: Matt Menke <mmenke@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#826462}
>
> TBR=mmenke@chromium.org,ericorth@chromium.org
>
> Change-Id: I03ea912d1a9c8a01be3d84a3f4d417c385f97c2d
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 1147247
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533548
> Reviewed-by: Jamie Madill <jmadill@chromium.org>
> Commit-Queue: Jamie Madill <jmadill@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#826873}


Bug: 1148269,1147247
Fixed: 1147247
Change-Id: I312c095e519c62d730e5fb6165833727f5d913be
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2534629
Commit-Queue: Eric Orth <ericorth@chromium.org>
Commit-Queue: Dan McArdle <dmcardle@chromium.org>
Auto-Submit: Eric Orth <ericorth@chromium.org>
Reviewed-by: Dan McArdle <dmcardle@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827004}

There is no reason for this file to be part of the shared
webui_resources.grd, as it is only used by NTP as follows:

 1) chrome-search://local-ntp/ (deprecated)
 2) chrome://new-tab-page

The former directly inlines the image by using flattenhtml=true, whereas
the latter consumes it from a chrome:// URL.

This is part of the effort of auto-genearting the contents of
webui_resources.grd.

Bug: 1132403
Change-Id: I20642b4c845393daf72597f156f7a934c9f3ea93
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533646
Auto-Submit: dpapad <dpapad@chromium.org>
Commit-Queue: Esmael Elmoslimany <aee@chromium.org>
Reviewed-by: Esmael Elmoslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827003}

These defaults are no longer necessary now that downstream is using its
own suppressions and baseline files. This makes sure that all targets
use lint the way that these updated docs say.

Bug: 1139957
Change-Id: I1ce9d79e990d472c719cdde688f2ca34a8100ac9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533575
Commit-Queue: Peter Wen <wnwen@chromium.org>
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Mohamed Heikal <mheikal@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Auto-Submit: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827002}

If a string destined for a menu cannot be turned into a CFString,
base64 it to allow for diagnosis.

Bug: 1140620
Change-Id: I78447813c8cfa5cae92e3e02f693313faf5336ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2535553
Commit-Queue: Avi Drissman <avi@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Reviewed-by: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827001}

design doc: https://docs.google.com/document/d/19dGklalQTRtRrG3PKrVbDRmYPLHGLnsGEwUaed7sAFQ/edit?usp=sharing

overview cl: https://chromium-review.googlesource.com/c/chromium/src/+/2393252

Chrome status: https://www.chromestatus.com/feature/5739732661174272

Bug: 1072058
Change-Id: I8a73e070d0121635be015239dc8f6c34a55fad97
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461219
Commit-Queue: Lu Huang <luhua@microsoft.com>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Alan Cutter <alancutter@chromium.org>
Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: Mandy Chen <mandy.chen@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#827000}