GitLab

This is mostly to prevent the high res avatars from being downloaded when not necessary, and will also prevent any other non-UI side effects which are undesired on non-desktop OSes.

BUG=404066, 411428

Committed: https://chromium.googlesource.com/chromium/src/+/8003a95bb88e1d25ad59251b5d7e8609d76e5669

Review URL: https://codereview.chromium.org/480453002

Cr-Commit-Position: refs/heads/master@{#293723}

Revert of Revert of Forward input and compositing tasks to the Blink scheduler (patchset #1 id:1 of https://codereview.chromium.org/548253002/)

Reason for revert:
this didn't reduce flakiness

Original issue's description:
> Revert of Forward input and compositing tasks to the Blink scheduler (patchset #21 id:400001 of https://codereview.chromium.org/363383002/)
>
> Reason for revert:
> suspecting it's causing the high rate of flakes in Instrumentation test ContentShellTest
>
> BUG=411756
>
> Original issue's description:
> > Forward input and compositing tasks to the Blink scheduler
> >
> > Instead of posting input and compositor tasks directly to the
> > main thread message loop, forward them to the Blink scheduler
> > so they can be prioritized over other tasks.
> >
> > BUG=391005
> >
> > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290845
> >
> > Committed: https://chromium.googlesource.com/chromium/src/+/68bab3290b45a8610ad875d3c92530477c16a948
>
> TBR=jamesr@chromium.org,alexclarke@chromium.org,picksi@chromium.org,eseidel@chromium.org,skyostil@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=391005
>
> Committed: https://chromium.googlesource.com/chromium/src/+/66f85d5405417e6ddd11e1b9be677062a2e3ae4f

TBR=jamesr@chromium.org,alexclarke@chromium.org,picksi@chromium.org,eseidel@chromium.org,skyostil@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=411756

Review URL: https://codereview.chromium.org/548333002

Cr-Commit-Position: refs/heads/master@{#293722}

Changing command dispatch from a "switch" dispatch to table
based dispatch makes DoCommand about 2 times faster on
Android, and also seems to be reducing times in other
command_buffer code due to code size reduction.

BUG=394570

Review URL: https://codereview.chromium.org/540053004

Cr-Commit-Position: refs/heads/master@{#293721}

Please see https://codereview.chromium.org/510323002 for reference.

NOTRY=true

Review URL: https://codereview.chromium.org/552553002

Cr-Commit-Position: refs/heads/master@{#293720}

Add content_editable_extractor_test to the gyp file and make it pass.

BUG=371692

Review URL: https://codereview.chromium.org/548523002

Cr-Commit-Position: refs/heads/master@{#293719}

Revert of Add test for ChromeVox keyboard commands. (patchset #8 id:140001 of https://codereview.chromium.org/490443002/)

Reason for revert:
Broke Linux ChromiumOS Tests (dbg)(3)

http://build.chromium.org/p/chromium.webkit/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%283%29/builds/1199

Original issue's description:
> Add test for ChromeVox keyboard commands.
>
> This is just a couple of quick sanity checks to prevent future regressions
> like the one in the linked bug. It tests both a Search+Shift shortcut and
> the corresponding Prefix Key shortcut.
>
> Adds support for sending the meta modifier key in
> aura/x11 tests.
>
> BUG=404470
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=291547
>
> Committed: https://chromium.googlesource.com/chromium/src/+/8cab2737f1cb6e6011b0946f6bef5027868602c0
>
> Committed: https://chromium.googlesource.com/chromium/src/+/ad123c3ac76fadc625a9dff66960743d0df253ba

TBR=dtseng@chromium.org,sky@chromium.org,dmazzoni@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=404470

Review URL: https://codereview.chromium.org/550183002

Cr-Commit-Position: refs/heads/master@{#293718}

Goal is to:

1) Reduce amount of log overhead for WebRTC clients;
2) Reduce risk for false-alarm for "no data" on initial short streams.

BUG=405449

Review URL: https://codereview.chromium.org/551763003

Cr-Commit-Position: refs/heads/master@{#293717}

BUG=410915
TEST=Start Chrome with --enable-new-avatar-menu. Sign in a profile. In
the avatar menu, the descenders in the "@gmail.com" string should not
be cut off.

Review URL: https://codereview.chromium.org/546053003

Cr-Commit-Position: refs/heads/master@{#293716}

BUG=407972

Review URL: https://codereview.chromium.org/544123003

Cr-Commit-Position: refs/heads/master@{#293715}

BUG=347425
TEST=unit_tests --gtest_filter="MetadataDatabase*"

Review URL: https://codereview.chromium.org/542233003

Cr-Commit-Position: refs/heads/master@{#293714}

Revert of Changes exception for gyp files to only gyp files in build/ (patchset #1 id:1 of https://codereview.chromium.org/512693004/)

Reason for revert:
gyp change this depended on was reverted

Original issue's description:
> Changes exception for gyp files to only gyp files in build/
>
> I changed analyzer such that if a gyp file (or an included gypi file)
> changes all targets in the gyp file are marked as changed. This means
> I should no longer have to special case gyp files. I'm leaving the
> exception in place for any gyp files in build as any change to those
> really needs to recompile the world.
>
> BUG=109173
> TEST=none
> R=thakis@chromium.org
>
> Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/5110e8b
>
> Committed: https://chromium.googlesource.com/chromium/src/+/e53e29fc161f96871160f6db39919752a0376333
>
> Committed: https://chromium.googlesource.com/chromium/src/+/5c361bce306a985bcb65cd9db6060ccbe139f66d

TBR=thakis@chromium.org,sky@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=109173

Review URL: https://codereview.chromium.org/553933002

Cr-Commit-Position: refs/heads/master@{#293713}

Cr-Commit-Position: refs/heads/master@{#293712}

Unit tests are now named *.unitjs instead of just *.js.
This is consistent with the e2e tests having a separate extension.
This avoids the files being read by the deps graph tools and avoids
regenerating/recompiling all tests when changing just one (because the
edpendency of the test_deps.js file on each test is now removed).

This also renames a test base class to use test_base.js to be consistent.

BUG=371692

Review URL: https://codereview.chromium.org/541403002

Cr-Commit-Position: refs/heads/master@{#293711}

Create HpackStaticTable singleton class, so that every HPACK context can share
a constant instance of the static table.

This lands server change 74862865 by bnc.

BUG=400336

Review URL: https://codereview.chromium.org/549583003

Cr-Commit-Position: refs/heads/master@{#293710}

BUG=330528
TBR=hpayer@chromium.org

Review URL: https://codereview.chromium.org/548273002

Cr-Commit-Position: refs/heads/master@{#293709}

BUG=399533
TEST=n/a (no behavioral change)
NOTRY=true

Review URL: https://codereview.chromium.org/552693002

Cr-Commit-Position: refs/heads/master@{#293708}

BUG=410726
TEST=manual

Review URL: https://codereview.chromium.org/542643003

Cr-Commit-Position: refs/heads/master@{#293707}

BUG=None

Review URL: https://codereview.chromium.org/550903002

Cr-Commit-Position: refs/heads/master@{#293706}

BUG=405860
TEST=manual

Review URL: https://codereview.chromium.org/545233002

Cr-Commit-Position: refs/heads/master@{#293705}

- Removed the warning for the call of util.entriesToURLs.
- Made sure that all private-API call using entriesToURLs have TODO comments.
- Added issue ID for the TODO comments.

BUG=411790
TEST=manually confirmed

Review URL: https://codereview.chromium.org/546413002

Cr-Commit-Position: refs/heads/master@{#293704}

This adds API to convert handle to and from int to allow interaction
with native code.

R=ppi@chromium.org

Review URL: https://codereview.chromium.org/544103002

Cr-Commit-Position: refs/heads/master@{#293703}

This CL provides touch-operation users with feedback earlier.
Menu items will be colored blue on touchstart.
Scrollable items (in directory tree and file list) will be colored gray on touchstart.
We don't use blue for them because touching them doesn't mean selecting them. (Users may have started scrolling)

BUG=404540
TEST=manually

Review URL: https://codereview.chromium.org/540873002

Cr-Commit-Position: refs/heads/master@{#293702}

TBR=blundell@chromium.org

Review URL: https://codereview.chromium.org/549123002

Cr-Commit-Position: refs/heads/master@{#293701}

BUG=374321

Committed: https://chromium.googlesource.com/chromium/src/+/77daf4022522fd889f31fe8cb6abb33c45a54857

Review URL: https://codereview.chromium.org/517193003

Cr-Commit-Position: refs/heads/master@{#293700}

The corresponding Blink side change is
https://codereview.chromium.org/531693002/.

BUG=

Review URL: https://codereview.chromium.org/531713002

Cr-Commit-Position: refs/heads/master@{#293699}

On some window managers the Maximize hints are not respected when they
are set on an unmapped window. There was an existing workaround for this
in the code, but the workaround failed whenever ShowWindowWithState
didn't happen to be called with the parameters needed to activate the
workaround. This meant that if a window was first made visible and later
activated, it would not get maximized at all.

This fix saves whether the maximize hints need to be sent to the window,
and makes sure it gets done after mapping the window.

BUG=

Review URL: https://codereview.chromium.org/543663003

Cr-Commit-Position: refs/heads/master@{#293698}

This is work around to avoid hijacking CORS preflight.
In current implementation, CORS preflight request is also sent to the ServiceWorker.
This is a bug in the implementation of HTTP fetch in DocumentThreadableLoader.
But it takes time to fix this.
So I want to introduce this work around because this is a security bug.

BUG=409454

Review URL: https://codereview.chromium.org/548773002

Cr-Commit-Position: refs/heads/master@{#293697}

ServiceWorker: Insert TRACE_EVENT to watch a breakdown of the ServiceWorker's performance on chome://tracing

Add TRACE_EVENT into each IPC message receiver and some important procedure (register, unregister and fetch).

BUG=401389
TEST=N/A

Review URL: https://codereview.chromium.org/538913002

Cr-Commit-Position: refs/heads/master@{#293696}

Revert of [telemetry] Add WinRing0/MSR power monitor. (patchset #3 id:40001 of https://codereview.chromium.org/536243002/)

Reason for revert:
Not sure but this CL was the only possible reason I could find why we see so many issues on XP.

See http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%282%29 and
http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%282%29/builds/35196/steps/telemetry_unittests/logs/platform_backend_unittest

Original issue's description:
> [telemetry] Add WinRing0/MSR power monitor.
>
> This is a power monitor that reads the Intel RAPL MSRs to get energy measurements.
> It uses a defunct open source DLL and kernel driver, WinRing0, to access the MSRs.
> WinRing0 is a little wonky, and we have to install the kernel driver to the Python executable's directory.
> Python must be run as Administrator for this to work.
>
>
> BUG=336558
> TEST=tools/telemetry/run_tests winring0
>
> Committed: https://chromium.googlesource.com/chromium/src/+/1f745bba63c18483a8f3c58464bcb01940adb04b

TBR=tonyg@chromium.org,dtu@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=336558

Review URL: https://codereview.chromium.org/547343003

Cr-Commit-Position: refs/heads/master@{#293695}

Speculative revert by Chrome sheriff.

NOTRY=TRUE
BUG=411827
TBR=davidben

Review URL: https://codereview.chromium.org/549103002

Cr-Commit-Position: refs/heads/master@{#293694}

BUG=None

Review URL: https://codereview.chromium.org/546213003

Cr-Commit-Position: refs/heads/master@{#293693}

HTML Tags: del, dfn, img, menu, ol, option, q, source
Aria role: alert, img, timer
Aria state/property: aria-readonly="false", aria-hidden="false"

BUG=170582

Review URL: https://codereview.chromium.org/496313003

Cr-Commit-Position: refs/heads/master@{#293692}

Previously we didn't have a call to gtk_window_present(), which would
mean that the widgets would become visible, but the necessary actions to
focus weren't always taken. This resulted in the active window not being
correct after opening the file or print dialogs.

BUG=399514

Review URL: https://codereview.chromium.org/543093002

Cr-Commit-Position: refs/heads/master@{#293691}

https://skia.googlesource.com/skia/+log/9f8cab153878f6e88190c5e2ef572ee708791177..6dcccc476bddd4ccba69491f6a87eed68e581414

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=fmalita@google.com

Review URL: https://codereview.chromium.org/548243002

Cr-Commit-Position: refs/heads/master@{#293690}

Reason for revert:
outputs invalid phony rules for freetype2 and others

Original issue's description:
> Roll gyp r1972:1973
>
> r1973: Changes the ninja generator to output phony targets for
> uninteresting targets
>
> BUG=410410
> TEST=none
> R=thakis@chromium.org
>
> Committed: https://chromium.googlesource.com/chromium/src/+/82f612b91e7d16880b26922f8b0b2cdc1490141b

TBR=thakis@chromium.org,sky@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=410410

Review URL: https://codereview.chromium.org/547303002

Cr-Commit-Position: refs/heads/master@{#293689}

Revert of More System Dialog printing code disabled on Windows. (patchset #3 id:40001 of https://codereview.chromium.org/517193003/)

Reason for revert:
Need also to check if !ENABLE_FULL_PRINTING

Original issue's description:
> More System Dialog printing code disabled on Windows.
>
> BUG=374321
>
> Committed: https://chromium.googlesource.com/chromium/src/+/77daf4022522fd889f31fe8cb6abb33c45a54857

TBR=thestig@chromium.org,jschuh@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=374321

Review URL: https://codereview.chromium.org/546373002

Cr-Commit-Position: refs/heads/master@{#293688}

This is just a couple of quick sanity checks to prevent future regressions
like the one in the linked bug. It tests both a Search+Shift shortcut and
the corresponding Prefix Key shortcut.

Adds support for sending the meta modifier key in
aura/x11 tests.

BUG=404470

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=291547

Committed: https://chromium.googlesource.com/chromium/src/+/8cab2737f1cb6e6011b0946f6bef5027868602c0

Review URL: https://codereview.chromium.org/490443002

Cr-Commit-Position: refs/heads/master@{#293687}

Revert of Forward input and compositing tasks to the Blink scheduler (patchset #21 id:400001 of https://codereview.chromium.org/363383002/)

Reason for revert:
suspecting it's causing the high rate of flakes in Instrumentation test ContentShellTest

BUG=411756

Original issue's description:
> Forward input and compositing tasks to the Blink scheduler
>
> Instead of posting input and compositor tasks directly to the
> main thread message loop, forward them to the Blink scheduler
> so they can be prioritized over other tasks.
>
> BUG=391005
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290845
>
> Committed: https://chromium.googlesource.com/chromium/src/+/68bab3290b45a8610ad875d3c92530477c16a948

TBR=jamesr@chromium.org,alexclarke@chromium.org,picksi@chromium.org,eseidel@chromium.org,skyostil@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=391005

Review URL: https://codereview.chromium.org/548253002

Cr-Commit-Position: refs/heads/master@{#293686}

Revert of cc: Single-threaded impl-side painting for unit tests (patchset #5 id:80001 of https://codereview.chromium.org/508373002/)

Reason for revert:
LayerTreeHostContextTestDontUseLostResources.RunSingleThread_DirectRenderer_MainThreadPaint Fails on Win7 bot after this change.

Speculative revert by sheriff.

http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/30957/steps/cc_unittests/logs/RunSingleThread_DirectRenderer_MainThreadPaint

Original issue's description:
> cc: Single-threaded impl-side painting for unit tests
>
> This adds methods to SingleThreadProxy to support impl-side painting
> when using the scheduler.  This allows most unit tests to test impl-side
> painting classes through both the single and threaded paths.
>
> The biggest caveat to this approach is the "commit waits for activation"
> logic that mailboxes want to use.  Because there is no easy way to block
> the only thread there is while activation happens, activation always
> happens immediately during commit, even if the tree is not ready.  To
> cause the tree to wait to draw until it is ready, drawing is prevented
> via SetRequiresHighResToDraw.  This prepares the way for the future
> where the SingleThreadProxy will likely commit directly to the active
> tree.
>
> The synchronous CompositeImmediately function does not support impl-side
> painting yet.  This is the path used by everything but unit tests at
> this point.
>
> R=danakj@chromium.org
> BUG=329553
>
> Committed: https://chromium.googlesource.com/chromium/src/+/7eee8e03582772716b820001af75343f6341e6e8

TBR=danakj@chromium.org,enne@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=329553

Review URL: https://codereview.chromium.org/550073002

Cr-Commit-Position: refs/heads/master@{#293685}

When a receive error occurs, the SerialConnection pauses receives, which
triggers a call to CancelRead(). This should be a no-op as this occurs
after the previous read completed with an error, but
pending_read_buffer_ isn't cleared until after the error is reported to
SerialConnection. Thus, on posix, a call to ReadDone is enqueued, but
the pending read is cleared before it runs, leading to a crash. This
change clears pending_read_buffer_ and pending_write_buffer_ before
calling Done() or DoneWithError().

BUG=410331

Review URL: https://codereview.chromium.org/551713002

Cr-Commit-Position: refs/heads/master@{#293684}