GitLab

Pass the Connector* provided by the service:media's ServiceContext into
MojoMediaClient::Initialize(), allowing MojoMediaClient implementations
to connect to interfaces exposed by other services. Also makes the
InterfaceProvider passed in CreateInterfaceFactory optional to eliminate
unecessary complexity, and corrects an error in media_manifest.json.

BUG=660736

Review-Url: https://codereview.chromium.org/2544523002
Cr-Commit-Position: refs/heads/master@{#436123}

This change is a part of larger effort of propagating D3D VSync
signal to the compositor. Since the current implementation in
BrowserCompositorOutputSurface explicitly depends on a time based
SyntheticBeginFrameSource, enne@ suggested that a good first step
would be to try to decouple it from a specific BeginFrameSource
type.

Instead of passing SyntheticBeginFrameSource and CompositorVSyncManager
in every constructor of BrowserCompositorOutputSurface and classes
derived from it, this change replaces the SyntheticBFS / VSyncManager
pair with a callback to update VSync parameters. The callback is
handled at GpuProcessTransportFactory.

BUG=467617

Review-Url: https://codereview.chromium.org/2511273002
Cr-Commit-Position: refs/heads/master@{#436122}

Fix PathBuilderKeyRolloverTest.TestMultipleRootMatchesOnlyOneWorks unordered_multimap ordering dependency.

BUG=670889

Review-Url: https://codereview.chromium.org/2550893002
Cr-Commit-Position: refs/heads/master@{#436121}

The JS Injection verification times out on some devices when used to
tap a form element. This CL thus increases the timeout.

BUG=670380

Review-Url: https://codereview.chromium.org/2543213003
Cr-Commit-Position: refs/heads/master@{#436120}

The current structure make it impossible to support hosting different
media services in different processes. For example, on desktop we may
want to host a video decoder service in the GPU process, but host a CDM
service in a utility process. For another example, on Android, we want
to host the audio decoder and CDM service in the GPU process, but host
a Renderer service in the browser process.

This CL adds a MediaInterfaceProxy which will serve as a proxy (and
central place) to dispatch media interface creation calls and decide
where to forward the interface request to support the above use cases.

In this CL, MediaInterfaceProxy will always forward calls to the
existing media service we have. In future CLs, we'll add more logic to
support the more complicated use cases.

This also moves a lot of media logic into a helper class, which helps
make RenderFrameHostImpl cleaner.

BUG=664364
TEST=Maually tested.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2521133002
Cr-Commit-Position: refs/heads/master@{#436119}

Original CL: https://codereview.chromium.org/2418173002/

Revert CL: https://codereview.chromium.org/2463103002/
https://codereview.chromium.org/2474073004

Changes: Skip failed gpu pixel tests until baseline is created.
Fix following failures in Win7 dbg
RendererPixelTest/2.PremultipliedTextureWithBackground
GLRendererPixelTest.TextureQuadBatching
RendererPixelTest/0.PremultipliedTextureWithBackground
GLRendererPixelTest.NonPremultipliedTextureWithBackground
IntersectingQuadGLPixelTest/0.Y16VideoQuads
VideoGLRendererPixelTest.TwoColorY16Rect
IntersectingQuadGLPixelTest/1.Y16VideoQuads

TBR=dalecurtis@chromium.org,ccameron@chromium.org,dpranke@chromium.org,junov@chromium.org
BUG=615325
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2547683003
Cr-Commit-Position: refs/heads/master@{#436118}

This is a first step to separate exo::SurfaceFactoryOwner into
exo::CompositorFrameSink and exo::CompositorFrameSinkHolder
(WIP CL: https://codereview.chromium.org/2493223002)

BUG=659601
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2544103002
Cr-Commit-Position: refs/heads/master@{#436117}

Since kInvalidOfflineId is a static data member, it needs explicitly
defined storage as soon as it becomes 'odr-used'.  Replace a temporary
variable in RecentTabHelper with storage in offline_page_model.cc both
for clarity and to allow further references to kInvalidOfflineId to be
taken.

BUG=

Review-Url: https://codereview.chromium.org/2548033002
Cr-Commit-Position: refs/heads/master@{#436116}

To reduce IPC calls between window_tree_client.cc and window_tree.cc.

BUG=632049

Review-Url: https://codereview.chromium.org/2520093003
Cr-Commit-Position: refs/heads/master@{#436115}

Normally, regenerating this file is deterministic. I can't find a discrepancy between the JSON and the behaviour of the generated data, but this updates the .h so we can debug without blocking further changes right now.

BUG=670139
TBR=palmer@chromium.org

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

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

Instead of immediately tearing down the image_decoder service
once it's idle, wait 5 seconds in case there are going to be
more requests soon.

This mirrors the behavior of utility process image decoding
prior to servicification, and is intended to address latency
issues with multiple image decodes happening in quick (but not
quite immediate) succession.

BUG=669835

Committed: https://crrev.com/0798a4585e6cb5df5e5596b8692656fca669cc28
Review-Url: https://codereview.chromium.org/2538833004
Cr-Original-Commit-Position: refs/heads/master@{#435815}
Cr-Commit-Position: refs/heads/master@{#436113}

This brings it in sync with components/precache/OWNERS.

BUG=

Review-Url: https://codereview.chromium.org/2541163002
Cr-Commit-Position: refs/heads/master@{#436112}

Doing TBR so that we can get some information from the minidumps over the weekend.

BUG=660293
TBR=csharrison

Review-Url: https://codereview.chromium.org/2542373002
Cr-Commit-Position: refs/heads/master@{#436111}

If WebContentsImpl::IsNeverShown() is true, like for an extension background
page, then there's never a reason to enable accessibility. Update
WebContentsImpl::SetAccessibilityMode to check for this, and also
update the UI page for chrome://accessibility to avoid exposing
such views.

BUG=365370
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2544993002
Cr-Commit-Position: refs/heads/master@{#436110}

BUG=666732

Review-Url: https://codereview.chromium.org/2550813002
Cr-Commit-Position: refs/heads/master@{#436109}

https://skia.googlesource.com/skia.git/+log/51d77ffdc635..d87fbee77524

$ git log 51d77ffdc..d87fbee77 --date=short --no-merges --format='%ad %ae %s'
2016-12-02 scroggo Do not create an SkRawCodec with zero dimensions

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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
TBR=brianosman@google.com

Review-Url: https://codereview.chromium.org/2549943002
Cr-Commit-Position: refs/heads/master@{#436108}

BUG=665074

Review-Url: https://codereview.chromium.org/2525563003
Cr-Commit-Position: refs/heads/master@{#436107}

R=dschuyler@chromium.org
BUG=614265
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2544453003
Cr-Commit-Position: refs/heads/master@{#436106}

These tests were failing around 17% of the time on Windows.

BUG=670846
R=qyearsley@chromium.org

Review-Url: https://codereview.chromium.org/2550733002
Cr-Commit-Position: refs/heads/master@{#436105}

This matcher will be used to check that interstitial was dismissed after
going back or proceeding.

BUG=531721

Review-Url: https://codereview.chromium.org/2546193002
Cr-Commit-Position: refs/heads/master@{#436104}

R=dschuyler@chromium.org
BUG=614265
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2538373002
Cr-Commit-Position: refs/heads/master@{#436103}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/8d05c456e60b..ca719be61d32

$ git log 8d05c456e..ca719be61 --date=short --no-merges --format='%ad %ae %s'
2016-12-02 aiolos Make telemetry use the same chrome stable build as tracing/dashboard.
2016-12-02 aiolos Update Chrome Cannary/Dev channel reference builds.

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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=catapult-sheriff@chromium.org

Review-Url: https://codereview.chromium.org/2550783002
Cr-Commit-Position: refs/heads/master@{#436102}

Initialize data reduction proxy (DRP) bypass stats on IO thread, and
register as a network change notifier on the IO thread. This ensures
that network change notifier does not notify DRP bypass stats on IO
thread after DRP bypass stats has been destroyed. This prevents the
use after null crash.

Also, add thread checker to some of the other DRP classes.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

BUG=669929

Review-Url: https://codereview.chromium.org/2546023002
Cr-Commit-Position: refs/heads/master@{#436101}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/372ee1fb..6e549d18

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

TBR=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org

Review-Url: https://codereview.chromium.org/2544423002
Cr-Commit-Position: refs/heads/master@{#436100}

NOTRY=true
BUG=na

Review-Url: https://codereview.chromium.org/2549763002
Cr-Commit-Position: refs/heads/master@{#436099}

This deletes things that are very VDA specific and will not be reusable.

BUG=660942
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2550523003
Cr-Commit-Position: refs/heads/master@{#436098}

This grows standalone WebView APKs by ~1MB. Ameliorating the growth is
tracked in the same bug.

BUG=662166

Review-Url: https://codereview.chromium.org/2541993002
Cr-Commit-Position: refs/heads/master@{#436097}

* Small refactoring to ChangeRequestsStateTask to use a more efficient
  method to query requests (which was not available at the time it was
  originally created).

BUG=651238

Review-Url: https://codereview.chromium.org/2541423002
Cr-Commit-Position: refs/heads/master@{#436096}

RenderMessageFilter interface (it has to be part of the associated interface for
now because of ordering issue with other CC messages).

As part of that, unified the bitmap allocation behavior in
ChildSharedBitmapManager on all platforms, by allocating shared memory
with the generic shared memory allocation code path and then
registering it as a bitmap.
Also moved ChildSharedBitmapManager from the ChildThreadImpl to RenderThreadImpl
as it is not common to all child processes.

BUG=612500
TEST=content_shell should run normally with the --dissable-gpu flag.

Review-Url: https://codereview.chromium.org/2488913003
Cr-Commit-Position: refs/heads/master@{#436095}

This CL cleans up the test setup and adds a test for the toggle control
in settings.

BUG=None

Review-Url: https://codereview.chromium.org/2517753002
Cr-Commit-Position: refs/heads/master@{#436094}

Also add a comment about the current state of ubsan.

BUG=669642

Review-Url: https://codereview.chromium.org/2550593004
Cr-Commit-Position: refs/heads/master@{#436093}

SameDomainOrHost does not require Origins. In fact, we just convert the
origins back to GURLs to do the check, which just looks at the host
piece of the url.

This conversion is not free in either direction, as we need to do
allocations, copies, and in some cases, url-related policies
(e.g. in constructing the underlying SchemeHostPort of Origin).

BUG=664174

Review-Url: https://codereview.chromium.org/2542843009
Cr-Commit-Position: refs/heads/master@{#436092}

BUG=669261
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2544723002
Cr-Commit-Position: refs/heads/master@{#436091}

BUG=662644
TEST=GPU FYI bots
NOTRY=true
R=kbr@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2549513003
Cr-Commit-Position: refs/heads/master@{#436090}

The test HandsOffNetworkScreenTest.RequiresNoInput was flaky.
Made some changes to fix that.

BUG=668054
TEST=browser test

Review-Url: https://codereview.chromium.org/2526423002
Cr-Commit-Position: refs/heads/master@{#436089}

Now that we don't support Ash on Windows we don't need all the conditional
code in this test.

The reason I'm cleaning up this one first is because it contains the string
"ash_unittests.exe" and I'm eliminating all references to Windows-specific
binaries.

BUG=670351
TEST=ash_unittests

Review-Url: https://codereview.chromium.org/2529293014
Cr-Commit-Position: refs/heads/master@{#436088}

This CL replaces base::WorkerPool::PostTask() with
base::PostTaskWithTraits(). The following traits are used:

Priority: Inherited (default)
  The priority is inherited from the calling context (i.e. TaskTraits
  are initialized with the priority of the current task).

Shutdown behavior: CONTINUE_ON_SHUTDOWN
  Tasks posted with this mode which have not started executing before
  shutdown is initiated will never run. Tasks with this mode running at
  shutdown will be ignored (the worker will not be joined).

With File IO:
  The task waits on synchronous file IO operations.

With Wait:
  The task waits on things other than synchronous file IO
  operations (e.g. WaitableEvent, ConditionVariable, join a thread,
  join a process, blocking system call that doesn't involve
  interactions with the file system).

BUG=659191

Review-Url: https://codereview.chromium.org/2533473002
Cr-Commit-Position: refs/heads/master@{#436087}

If a second offline snapshot starts for the same page load where a
previous one was saved, it should not immediately delete it: as it might
fail the user would end up with no offline copy of the page. This change
fixes this issue.

Three new tests are added: one to cover this precise case and a couple
more to check for related situations. Also many comments were added or
updated.

BUG=655697

Review-Url: https://codereview.chromium.org/2542833003
Cr-Commit-Position: refs/heads/master@{#436086}

Added e9ff58f7acde29c4e53c761da775293e776bcc23 to internal repo.

BUG=665305

Review-Url: https://codereview.chromium.org/2547203002
Cr-Commit-Position: refs/heads/master@{#436085}

- WTF::TimeDelta aliased base::TimeDelta
- WTF::{Time,TimeTicks} are a thin wrapper around base::{Time,TimeTicks} this is
  mainly because we want to control the API surface we expose in particular we
  want to avoid exposing string parsing functions at this time.

Once we have these types we can start replacing usage of double and int to
represent time in blink with more accurate and type safe representations.

Relevant discussion: https://groups.google.com/a/chromium.org/d/msg/platform-architecture-dev/7kg1FAsXkng/l19cEeBPBAAJ

BUG=625680, 402027
TEST=Critical functionality is already covered in base/time/time_unittest.cc but
added a smoke test in TimeTest.cpp

Review-Url: https://codereview.chromium.org/2543663002
Cr-Commit-Position: refs/heads/master@{#436084}