GitLab

This patch was generated by running the rewrite_scoped_refptr clang tool
on a ChromeOS build.

BUG=110610

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

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

GpuCommandBufferMsg_SetLatencyInfo is used to send current frame's latencyinfo
to gpu. We can piggyback the latencyinfo in GpuCommandBufferMsg_AsyncFlush so
eliminate the extra IPC.

BUG=404650
TEST=check trace that now GLRenderer::SwapBuffers only sends 2 IPC messages,
     i.e. AsyncFlush and Echo. And LatencyInfo tracking is still working.

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

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

BUG=None
TBR=jochen@chromium.org

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a ChromeOS build.

BUG=110610

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a ChromeOS build.

BUG=110610

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a ChromeOS build.

BUG=110610

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a ChromeOS build.

BUG=110610

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a ChromeOS build.

BUG=110610

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a ChromeOS build.

BUG=110610

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

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

Cleanup: Reorganize extensions renderer build files so we only have to list each file once, rather than two if they are excluded on Android.

Remove all native handlers from the Android build while we are at it.

BUG=402728

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

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

net: another batch of HSTS preloads.
Cr-Commit-Position: refs/heads/master@{#294886}

This change removes the signalling of a new media segment when an audio
frame is partially front-trimmed. This allows other muxed streams, like
video, to not necessarily continue with a keyframe as the next frame
processed (unless new media segment has otherwise been signalled).

BUG=408216
R=dalecurtis@chromium.org
TEST=no media_unittest or mediasource layout test regression

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

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

This prepares for the landing of
  https://codereview.chromium.org/522783002/

which now requires a three-sided change due to changes in how Blink accesses
private script resources.

Part 1: https://codereview.chromium.org/570863002/

BUG=364716

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

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

This change refactors the existing support for cross-process iframe
accessibility and extends it to make guest browser plugins, for example
the <webview> element in apps, accessible too.

In this change, a new class, FrameAccessibility, manages all links
from one frame's accessibility tree to either a child iframe's tree,
or a guest web contents' tree. This is cleaner than storing ids in
BrowserAccessibility nodes - now the accessibility tree just needs a
single bit indicating which nodes in one tree are acting as a host of
another tree.

FrameAccessibility doesn't try to be efficient yet, it scans its list
for every operation - the goal is for it to be simple and safe first.
Hash maps could make it much faster later if needed.

None of the APIs needed to test this are available outside of content
yet. Once http://crbug.com/396137 is finished we'll have a clean way
to test this. In the meantime, this can be tested by visiting
chrome://accessibility on Windows or Mac, and observing that the
accessibility tree for an app includes all nodes within <webview>
elements now. Also tested with VoiceOver on Mac; there are some bugs
but the <webview> is definitely part of the accessibility tree now.

BUG=330307,368298

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

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

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

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

https://skia.googlesource.com/skia/+log/a63d5df455e5fb32ac995f753709893221de6147..595aa05efcb504e85358b8d328ac4a9fa1c46e2e

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

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

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

The authentication is attempted when establishing a secure connection with the
remote device.

BUG=411043
TEST=none
NOTRY=true
R=jwd@chromium.org

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

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

Datatype is responsible for writing attachments to AttachmentStore.
GenericChangeProcessor doesn't need attachment data in SyncData, only
list of attachment ids.

This change removes AttachmentList from SyncData and fixes corresponding
dependencies.

BUG=
R=maniscalco@chromium.org

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

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

If a view has a ton of children (say > 5000) with the majority hidden
then we spend a lot of unecessary time in View::UpdateRootBounds on
every paint.

BUG=409126
TEST=none, see bug
R=luken@chromium.org

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

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

In order to componentize host_content_settings_map, one big obstacle
is the extension system. This CL changes the direction of the
dependency.
old: host_content_settings_map register extension system
new: host_content_settings_map accepts providers which allows
     extension system to register itself.

I believe this is a better design.

BUG=384869
R=yoz@chromium.org, markusheintz@chromium.org, bauerb@chromium.org
TBR=rlp@chromium.org, sky@chromium.org
TEST=no regression

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

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

Revert of Fix lsan failure with ManifestParserTest.* (patchset #1 id:1 of https://codereview.chromium.org/554043004/)

Reason for revert:
broke build

Original issue's description:
> Fix lsan failure with ManifestParserTest.*
>
> BUG=None
> TBR=rockot@chromium.org
> NOTRY=true
> NOPRESUBMIT=true
>
> Committed: https://crrev.com/36816846f1cf8ceea2f96d501ebc745c82349337
> Cr-Commit-Position: refs/heads/master@{#294870}

TBR=rockot@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=None

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

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

Revert of [Android] Mark posted UI thread tasks as asynchronous (patchset #4 id:60001 of https://codereview.chromium.org/512333002/)

Reason for revert:
This patch was purely experimental, and the experiment has expired.

Original issue's description:
> [Android] Mark posted UI thread tasks as asynchronous
>
> Chromium shares a message loop with Android on the browser UI thread.
> This can cause problems when the associated Looper has a sync barrier,
> preventing posted Chromium tasks from being dispatched until the
> barrier is removed. Make this sharing more fair by marking all Chromium
> Message tasks as asynchronous, avoiding stalls when there is a sync
> barrier.
>
> Note: This change is for gathering data about the perf impact and we'll
> revert it before cutting a release branch.
>
> BUG=407149,380781,407133
>
> Committed: https://crrev.com/feabeebb3ac5810f896ac8303a77ee695acaf9d4
> Cr-Commit-Position: refs/heads/master@{#292551}

TBR=aelias@chromium.org,epenner@chromium.org,nyquist@chromium.org,sievers@chromium.org,skyostil@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=407149,380781,407133

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

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

This adds key-system less support to isMediaMIMETypeSupported and mojo
surfaces bindings for YUV quads.

R=acolwell@chromium.org

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

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

The test was added for keyboard usability experiment which is no longer
needed.

BUG=413583
TEST=none

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

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

This allows dogfooders to get Easy unlock on consumer devices
(i.e. non-enrolled devices).

BUG=395154

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

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

This started out as simply converting NaCl's baseline policy to bpf_dsl,
but that led me down the rabbit hole of converting a bunch of remaining
uses of SandboxBPFPolicy within content too.  There are still a few
lingering dependencies on sandbox_bpf.h after this CL, but not many.

BUG=414363

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

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

BUG=None
TBR=rockot@chromium.org
NOTRY=true
NOPRESUBMIT=true

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

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

TTS initialiation can take around ~150ms to initialize on Android. Trace
this initialization to make it clear if the task is blocking critical
updates on the browser UI thread.

BUG=406597

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

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

BUG=

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

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

Since Write() takes two (or more) steps to finish, it's possible for
CdmFileIOImpl to be destructed after SetLength() is called but before
WriteFile() is called. In that case, since we SetLength(data_size), we
could end up with a corrupted file.

This CL update FileIOTest to explictly cover this case.

BUG=410630
TEST=ECKEncryptedMediaTest.FileIOTest still passes.

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

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

DisplayManager wasn't handling multiply nested things correctly and the
client library wasn't handling uploading to multiple views from the same
application. This fixes those.  There's still some z-ordering issues that
show up pretty visibly. This hacks up texture alpha so you can see
partially through views to try to debug.

BUG=

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

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

Creating a framework for suppressing pollution of the profiler data and applying for know cases of pollution.
See the bug.

The CL introduces TaskStopwatch that has to be used to measure run time for tasks. It takes care of double-counting
run time in the nested-tasks case by subtracting run time of nested tasks from their parents. TaskStopwatch can be
also used for subtracting other nested intervals, such as the time while a nested message pump runs. ThreadData::TallyADeath
now takes a stopwatch parameter instead of start_time and end_time. This helps avoid mistakes when the interval passed up to
the parent for exclusion is different from the interval for the current task duration.

BUG=401560

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

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

Annotate several NavigationController methods used only in testing, and
remove a helper test function that is unused even in tests.

BUG=398263
NOTRY=true

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

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

BUG=409926

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

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

-[NSView setFrame:display:] accepts BOOL as the second argument,
passing nil only worked by accident.

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

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

Changing in the intialization order of WeakPtrFactory such that all
member variables should appear before the WeakPtrFactory to ensure
that any WeakPtrs to Controller are invalidated before its members
variable's destructors are executed, rendering them invalid.

BUG=303818

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

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

We are going deprecate console.timeline/timelineEnd. As the first step they will result only in time/timeEnd events and won't start/stop tracing. This change removes the code that allows to change browser wide tracing recording from the render process, it mostly reverts part of r292578.

BUG=412782
TBR=dcheng

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

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

BUG=NONE

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

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

BUG=387410
R=qyearsley@chromium.org

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

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

Also update the name of the asan dynamic runtime thunk .lib since it was changed in LLVM r217673.
After this change, component ASan build should be able to report NULL derefs and other AVs the same way as the static build does.

BUG=345874

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

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