GitLab

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/79d4f9950680..89863242a428

git log 79d4f9950680..89863242a428 --date=short --no-merges --format='%ad %ae %s'
2019-11-12 yangguo@chromium.org Add fetch configs for devtools-frontend

Created with:
  gclient setdep -r src/third_party/depot_tools@89863242a428

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/+/master/autoroll/README.md

TBR=agable@chromium.org

Bug: None
Change-Id: Ide21ab380a0f3eb1031487e15de18a0450e5cc3d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911703Reviewed-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@{#714551}

Translation screenshots project requires Chrome devs to upload UI
screenshots along with string changes in .grd and .grdp files. These
are XML files containing all user visible strings in Chrome.

Translation screenshots has a presubmit checks that extracts the
list of modified strings, looks for image files associated with these
strings and warns the developer to add them if they are missing.

This presubmit parses .grds to extract the list of modified strings but
it skips <part> tags which are used to reference .grdp files. As a
result, the presubmit currently ignores string changes in .grdp files.
.grdp files contain the majority of UI strings in Chrome, so the lack of
presubmit checks for them results in low translation screenshot coverage.

This CL changes how .grdp files are loaded in the presubmit: Previously,
they were written in a temporary directory alongside a fake .grd file
that referenced the loaded .grdp file. The code loaded the fake .grd file
which resulted in loading the strings in the actual .grdp file. However,
test mocks override all os.path methods which in turn breaks temporary
directory creation. As a result, it was not possible to properly test
the .grdp loading code, so we skipped <part> tags as a workaround.

The new loading code writes a temporary copy of the .grdp file and wraps
its contents with proper tags so that it can be loaded as a .grd file
instead. This avoids the need to create a temporary directory and is
fully testable.

The end result is that Translation Screenshots presubmit will now ask
Chrome devs to upload screenshots for changes in .grdp files. This should
significantly increase the coverage of strings with translation screenshots,
leading to better quality in string localizations.

Bug: 924652
Change-Id: Iae7933e8147cefdabc15ef21d2b6daa43d5002bc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1875554Reviewed-by: anthonyvd <anthonyvd@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Mustafa Emre Acer <meacer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714550}

All RadioButtonWithDescriptions are hosted under the same layout now.
Logic that injected the additional checkbox option now needs to account
for that change. Also leaving some explinations in there to make the
file a bit more descriptive.

Bug: 1020890
Change-Id: Id46e4d0ef56aee168d55f85882a4a100d872fde0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898710
Commit-Queue: Brandon Wylie <wylieb@chromium.org>
Reviewed-by: Theresa  <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714549}

Currently, a multiline label will in many cases report a minimum size
that can be smaller than the appropriate line-height for the text style,
causing vertical layouts to short the size of the text by a few pixels.

This results in, for example, a visual up-and-down wobble in the
domain label in a hover card as it animates between tabs, due to the
changing hover card size. This CL fixes this issue.

Note that except in vertical flex (and possibly box) layouts minimum
size tends to not be factored in, so this should only affect a handlful
of situations (and likely produce a better visual result).

Change-Id: Icdc6006b8f51588009b0709773f1854ecf5a6a2e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910807
Commit-Queue: Dana Fried <dfried@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714548}

- Adds a flag to allow the AEQ UI toggle to be displayed
- UI only displays on devices that support AEQ

BUG=chromium:1021193
TEST=manual

Change-Id: Iaf3a4ef24ed2512cc7096adae40fcdacd7ac3cd5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894163
Commit-Queue: Zentaro Kavanagh <zentaro@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714547}

We were counting non-default suggestions with a relevance of 0, which
would later be removed anyways, in our logic to decide whether to
favor or squelch tail suggestions within a round of suggestions.
This CL now ignores them (unless they're the default, of course).

Bug: 1023821
Change-Id: If6a8a49cf6f2f53b487d2a18343af9bc038c33f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906963
Commit-Queue: Kevin Bailey <krb@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714546}

BrowserFocusTest.FocusOnReloadCrashedTab is currently flakey on
REL CHROMEOS builds as well. It generally seems off to be
disabling the test on DBG but not REL, so disable it for both.

TBR=sky@chromium.org

Bug: 50025
Change-Id: I94b172f4b472aa361ffc2ba6557ebc12b14691c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910843Reviewed-by: James MacLean <wjmaclean@chromium.org>
Commit-Queue: James MacLean <wjmaclean@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714545}

https://chromium.googlesource.com/breakpad/breakpad.git/+log/5915ea929c43..792e6b2143bf

$ git log 5915ea929..792e6b214 --date=short --no-merges --format='%ad %ae %s'
2019-11-12 sdefresne Remove dependency of uploader.mm on GTMLogger

Created with:
  roll-dep src/third_party/breakpad/breakpad

Bug: 1012115
Change-Id: I33c4f2b7a762be15df97ae7fe1d84ecffd6b317d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912239
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Mark Mentovai <mark@chromium.org>
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714544}

This reverts commit 1d267524.

Reason for revert: Suespect that this broke the build on Windows.

Original change's description:
> Added access operator to color mixers.
> 
> Added the access operator to the color mixer class, updated existing
> code and tests. Changed AddMixer() to return a mixer by reference.
> 
> Bug: 1887666
> Change-Id: I2ae180f77d73e3ce4567d061b5505d920677c450
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910521
> Commit-Queue: Peter Kasting <pkasting@chromium.org>
> Reviewed-by: Peter Kasting <pkasting@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#714528}

TBR=pkasting@chromium.org,tluk@chromium.org

Change-Id: I7eae3b7e1824b31adaa04c5e998c883da05eea96
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1887666
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911709Reviewed-by: James MacLean <wjmaclean@chromium.org>
Commit-Queue: James MacLean <wjmaclean@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714543}

https://swiftshader.googlesource.com/SwiftShader.git/+log/371316798c01..197a5725f2fe

git log 371316798c01..197a5725f2fe --date=short --no-merges --format='%ad %ae %s'
2019-11-12 swiftshader.regress@gmail.com Regres: Update test lists @ 9da287fd

Created with:
  gclient setdep -r src/third_party/swiftshader@197a5725f2fe

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/swiftshader-chromium-autoroll
Please CC swiftshader-team+autoroll@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/+/master/autoroll/README.md

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_chromium_msan_rel_ng;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
TBR=swiftshader-team+autoroll@google.com

Bug: None
Change-Id: Ieb054cde46164cdeb1c27291b11db110644139a6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912159Reviewed-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@{#714542}

Goes with spec CL https://github.com/w3c/csswg-drafts/pull/4437 (merged)
Eliminated the use of CSSPseudoElement from WebAnimations to coincide with
recent spec changes. JeyframeEffect targets now use an Element reference
and pseudo-element selector like getComputedStyle and css-animations events.

Change-Id: Ib6957a7b764eac3b4372ad35c1b79b139c356a19
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894477Reviewed-by: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Kevin Ellis <kevers@chromium.org>
Commit-Queue: George Steel <gtsteel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714541}

Overdraw feedback has been broken for two reasons:
1. When we added explicit fences, we inserted the fence before the
  overdraw colors were drawn
2. When we changed how we do partial swap, we forgot to expand the rect
  where the overdraw feedback is drawn to.

This CL fixes those two issues moving the drawing of the overdraw
feedback at the end of DrawRenderPass.

Test: Run Chrome on nocture with the flag to enable overdraw feedback
Change-Id: I57069f8596d0bdb2bc819c8de5c70cf3901d7199
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904681
Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
Reviewed-by: kylechar <kylechar@chromium.org>
Reviewed-by: Andres Calderon Jaramillo <andrescj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714540}

Since caret browsing mode is turned off by default in Chrome and the
detailed report does not show "enable-caret-browsing" among the command
line switches, the null reference read must be from the dereference of
frame.GetSettings() at the start of the method
MoveCommands::UpdateFocusForCaretBrowsing. Note that in earlier
patchsets in:

https://chromium-review.googlesource.com/c/chromium/src/+/1666567

there was a null check for frame.GetSettings():

https://chromium-review.googlesource.com/c/chromium/src/+/1666567/9

These were removed because of reviewer feedback that they were
unnecessary:

https://chromium-review.googlesource.com/c/chromium/src/+/1666567/9/third_party/blink/renderer/core/editing/commands/editor_command.cc#1057

However I see several places under blink\renderer\core\editing where a
null check on return value of Frame::GetSettings is performed. And
Frame::GetSettings has comment:

Settings* GetSettings() const;  // can be null

So the first part of my fix is to move the call to
GetCaretBrowsingEnabled to new method
LocalFrame::IsCaretBrowsingEnabled, which includes a null check on
Frame::GetSettings.

However, this simple initial attempt at a fix that was made before I had
a repro did not answer why an editing command was being executed in
response to an input event when the frame is detached. I finally did
find an easy repro, and used it as the basis for a new webtest.

With a repro in hand. even with the null check debug builds were
hitting a NOTREACHED crash in FrameSelection::DidSetSelectionDeprecated.
And this was a clue to the root cause. The call to SetFocusedNodeIfNeeded
dispatches focus events, and a script handler for these events can
remove the frame. In other words, it is the editing code itself that
is causing the frame to be detached. The fix here is to remove the
NOTREACHED macro and have DidSetSelectionDeprecated return early.
This follows the pattern of other methods in FrameSelection that call
IsAvailable to assure that the frame is still attached.

For trivial changes to local_frame.cc and local_frame.h:
TBR=dcheng@chromium.org

Bug: 1015710
Change-Id: If72e51eadbebbcf8879caccc5bd2c3480d9fbf39
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1869298
Commit-Queue: Bruce Long <brlong@microsoft.com>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714539}

- enable verbose logging.
- try skipping webdriver tests, which seem related to the timeouts.

Also proposing to add per-file owners for testing/scripts/run_wpt_tests.py
to enable quicker poking at these flags in the near term.

Bug: 1023835
Change-Id: I1d7b7b7f08adea6a8f33ff722458faf3bacacbde
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906897
Commit-Queue: Luke Z <lpz@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714538}

Following bot results are included.
253 254 255 261 262

TBR=yosin@chromium.org
NOTRY=true

Bug: 982194
Change-Id: Ib1d1a756c4c0d828d76781a35c5f7e00f3b08f3e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911339Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714537}

Idiomatically, the dawn headers are included with angle-bracket style,
using dawn, dawn_native, etc. as the root. This is the include style
used in existing webgpu code in Chromium, eg. webgpu_decoder_impl.cc.
Refactor the DawnContextProvider and SkiaOutputSurfaceDawn code to match
this style.

Bug: 1021566
Change-Id: I3d462b432d763189350a31050ae334dd2bc56b84
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911600Reviewed-by: Jonathan Backer <backer@chromium.org>
Commit-Queue: Sean Gilhuly <sgilhuly@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714536}

We have some special-code for tall unbreakable lines; if a line in
itself is taller than the fragmentainer, don't break before it like we
normally do when out of space. This piece of code was here for
compatibility reasons (or maybe just to work around another bug), but we
need to avoid this special behavior if the line is the one to break
before in order to honor a widows requirement. Otherwise we'd get
nowhere, stuck in a loop enternally re-laying out, attempting to break
before this line but refusing to do so.

Bug: 1022348
Change-Id: I2b9f702194dc0b7e6cbf8f37dffa95ea5722d045
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911205Reviewed-by: Koji Ishii <kojii@chromium.org>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714535}

https://chromium.googlesource.com/angle/angle.git/+log/b9b1bae4effb..3f647b1bf9a4

git log b9b1bae4effb..3f647b1bf9a4 --date=short --no-merges --format='%ad %ae %s'
2019-11-12 jmadill@chromium.org Vulkan: Improve Bresenham line emulation.

Created with:
  gclient setdep -r src/third_party/angle@3f647b1bf9a4

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/angle-chromium-autoroll
Please CC cnorthrop@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/+/master/autoroll/README.md

CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
TBR=cnorthrop@google.com

Bug: None
Change-Id: Ia807404e27e41ae8b7ec74db796be455a17ca2e4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910797Reviewed-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@{#714534}

Prior to this CL, the month picker would continue to highlight the same month even after
the year is changed. The new behavior is to always highlight the first month when a year
is changed.

This change is validated by adding: date-picker-month-appearance-after-year-change.html.
The expected results for existing tests are also updated.

Bug: 1021331
Change-Id: Ib661e9b3607fd190a7753060a06a108ff53e8be0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1907487
Commit-Queue: Ionel Popescu <iopopesc@microsoft.com>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Mason Freed <masonfreed@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714533}

Bug: 1023397
Change-Id: Iaecffb245ca0da14a8a048966690534eb92f3046
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912122Reviewed-by: Roman Sorokin [CET] <rsorokin@chromium.org>
Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714532}

We'll limit testing on the new builder to devtools-specific tests in a follow up CL.

Bug: 1018204
Change-Id: I45e7442258f4de47999fc1d0b26cd1b31b96e5ca
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899502
Auto-Submit: Liviu Rau <liviurau@chromium.org>
Commit-Queue: Liviu Rau <liviurau@chromium.org>
Reviewed-by: Michael Achenbach <machenbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714531}

Currently the parsing step of TestExpectations takes ~11 seconds on a
development machine, with the majority of that time in the
_collect_matching_tests() step, building the list of |matching_tests|.

This can be sped up considerably by changing _all_tests from a flat
list of test names to a dictionary tree of path components. Then
matching a directory of tests is a search in the tree + iterating the
subtree below the matching directory node, instead of a search through
the full list with substring matching.

This reduces the time locally for parsing TestExpectations from ~11 to
~3 seconds, saving ~72% of the time on this step.

With logging in web_tests/controllers/manager.py:

+        _log.debug("> build TestExpectations");
+        start = time.time()
         self._expectations = test_expectations.TestExpectations(self._port, test_names)
+        _log.debug("< build TestExpectations %fs", time.time()-start);

Before:
15:28:22.789 167015 < build TestExpectations 11.192820s

After:
11:23:38.824 11280 < build TestExpectations 3.105809s

On the windows try bot:

Before: https://chromium-swarm.appspot.com/task?id=4874d0c111ee7a10
11:48:32.359 11980 Parsing expectations ...
11:49:15.140 11980 Found 3676 tests (total 86570); running 3439, skipping 237.
= 43 seconds

After: https://chromium-swarm.appspot.com/task?id=4874f28bff322210
12:24:46.461 11648 Parsing expectations ...
12:25:22.805 11648 Found 3676 tests (total 86570); running 3439, skipping 237.
= 36 seconds

R=robertma@chromium.org

Bug: 982208
Change-Id: I04f5c3c1b09cf4f1b42ad501d624f83f6f378a97
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906472Reviewed-by: Robert Ma <robertma@chromium.org>
Commit-Queue: danakj <danakj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714530}

This tap gesture was preventing the TableView from recognizing some
delegate callback like didSelectRow. Setting cancelsTouchesInView
to NO "forwards" the touch to the TableView successfully.

Bug: 1018341
Change-Id: I249e6795b47d67c168eaf574dfdb765ee1bd6d38
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911480Reviewed-by: Chris Lu <thegreenfrog@chromium.org>
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714529}

Added the access operator to the color mixer class, updated existing
code and tests. Changed AddMixer() to return a mixer by reference.

Bug: 1887666
Change-Id: I2ae180f77d73e3ce4567d061b5505d920677c450
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910521
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714528}

The reported crash can happen when navigation sheet's request to show
fails. It should have always called |hideContent| to remove itself
from the queue but it was done only when the current sheet state is
hidden. This CL fixes it to perform the hiding operation
unconditionally.

Bug: 1021932
Change-Id: I3fa3c08ff814d9aa2aa149196b9f5e91b262cb73
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910819Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714527}

Use the tab height constant instead of the height of the view. The former should match the desired height anyway, since group headers are TabSlotViews. The latter can sometimes be 0, e.g. when dragging a group header into a new window.

Bug: 1005333
Change-Id: I6000af475cb25f0635fbacc1f0ef8acd6aa9fe67
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906329Reviewed-by: Charlene Yan <cyan@chromium.org>
Reviewed-by: Taylor Bergquist <tbergquist@chromium.org>
Commit-Queue: Connie Wan <connily@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714526}

When you drag a window from clamshell overview, there shall be split
view drag indicators on every display. As of the present CL, all labels
will be based on whether the dragged window can be snapped in split view
on the display where the drag started. I will fix that in a separate CL.

Bug: 961171
Change-Id: Iabef96cece3dcc5e7d45050c3218aba54993d593
Fixed: 995413
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903614Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Avery Musbach <amusbach@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714525}

This documentation-only CL modifies user_types.md to mention
LoggedInUserMixin as a convenient way to log in as child user for
browser tests.

Bug: None
Change-Id: Ib6d3833600bb1fcd74e1fee61e0d68446dbee7c5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911219Reviewed-by: Aga Wronska <agawronska@chromium.org>
Commit-Queue: Toby Huang <tobyhuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714524}

-The Trace method was missing EventTargetWithInlineData::Trace call
-Using ActiveScriptWrappable now so EditContext wrapper object remains alive
till the execution context is valid and when the composition events arrive,
they can be processed properly by the event listeners for the EditContext.
- Also removed an invalid test case that had a global reference to EditContext
which was causing memory leaks

Test: run_web_tests web_tests/editing/input/edit-context.html

Bug: 1023351, 1023242

Change-Id: If7e9d7ebea195095b0ac5048cd0864d3e0b730f2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911280Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Anupam Snigdha <snianu@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#714523}

Preliminary step for removing gaia-page template in saml-interstitial
template.

Bug: 981472
Change-Id: I8a17b079a6b45caa376a54a124fd1833cc414fa1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1897699Reviewed-by: Roman Sorokin [CET] <rsorokin@chromium.org>
Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714522}

Parsing a single CSV row inside CSVPassword::Parse can fail for two
reasons: bad syntax, or incorrect semantics. So far, both cases have
been reported simply as an error. Soon, the distinction will be needed
to correctly report from CSVPasswordSequence.

Therefore this CL changes the return type of CSVPassword::Parse from
Boolean (whether parsing failed) to a new status enum (how it
failed): OK, Syntax error or Semantic error. This enum describes
a subset of ParserState::Result, but ParserState::Result was not
directly used here, because the CSV logic should not know about
its own client code.

Bug: 934326
Change-Id: I621e2faa38241541d684357b2f54d9f5fb8ed923
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1908528
Commit-Queue: Vaclav Brozek <vabr@chromium.org>
Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714521}

Revert "Add dominant/latest version index to PaintTiming.NavigationToFirstContentfulPaint UKM event."

This reverts commit 83bbeb87.

Reason for revert: impact on pipeline

Original change's description:
> Add dominant/latest version index to PaintTiming.NavigationToFirstContentfulPaint UKM event.
> 
> Change-Id: Ib63a04cc663011872f60bd04be420b3207dffb29
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1869451
> Auto-Submit: Brian White <bcwhite@chromium.org>
> Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
> Commit-Queue: Robert Kaplow <rkaplow@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#708206}

TBR=rkaplow@chromium.org,bcwhite@chromium.org,sdoyon@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Change-Id: I7ef5bc2f83fdbf3e69fd92f053b80632c12712e8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904687Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Brian White <bcwhite@chromium.org>
Commit-Queue: Robert Kaplow <rkaplow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714520}

Bug: 981349
Change-Id: I85bd9eea5a612cb5aaa6c519c8750a79505007c0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911785
Commit-Queue: Mikhail Khokhlov <khokhlov@google.com>
Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714519}

Bug: 1022937
Change-Id: Iacbda3b8a13f41760c74c252fb71bccc0ac66e1c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906939Reviewed-by: Peter Kvitek <kvitekp@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Andrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714518}

* Add DiffSizeInfo, with logic for matching up symbols from two
  loaded SizeInfos. This doesn't yet consider the new native and
  Java canonical name processing code, but the boilerplate is there.
* Add scaffolding in caspian_web.cc to support diffing.
* Add async waits in .js to avoid attempts to call WebAssembly
  code before module is loaded.

Bug: 1011921
Change-Id: Ia121e78fd5c8c2d7e07c53de345e53e9fa80d88d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906109
Commit-Queue: Jasper Chapman-Black <jaspercb@chromium.org>
Reviewed-by: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714517}

Stop checking for aggregated trace from telemetry, and stop uploading
diagnostics artifact.

Bug: 981349
Change-Id: I045e97ce875751b3621177bac1a38949cb1272c5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911745
Commit-Queue: Mikhail Khokhlov <khokhlov@google.com>
Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714516}

BrowserDMToken contains a string and an enum. It should be able to be
copied and moved as a normal string. Explicitly declare copy and move
constructor and operator.

Also move the factory function declaration to the top of the class.

Bug: 1020296
Change-Id: I401812b07db1bd2801a4133cb67592f9ef4bc6d2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906477Reviewed-by: Roger Tawa <rogerta@chromium.org>
Commit-Queue: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714515}

This change fixes multiple issues with the process-internals page when
viewed in high contrast. Specifically, the header background and tree
highlight colors are fixed. See the bug for screenshots.

Bug: 1023585
Change-Id: I6405e8b02c5020eb64d3aff93c564ab20a613ef0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911482Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Cliff Smolinsky <cliffsmo@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#714514}

Bug: 1022625
Change-Id: Idd906b79dda4ee54df7d743ceb52ddfe4f2447ba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1907313
Auto-Submit: Michael Crouse <mcrouse@chromium.org>
Commit-Queue: Michael Crouse <mcrouse@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Sophie Chang <sophiechang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714513}

Change-Id: Id1b567b87d88d3f3c29403ad9ec0c794ab9c9fd9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911794Reviewed-by: Alexander Hendrich <hendrich@chromium.org>
Commit-Queue: Jit Yao Yap <jityao@google.com>
Cr-Commit-Position: refs/heads/master@{#714512}