GitLab

BUG=403645

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

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

This CL removes most usages of the guava library.

TBR=thakis@chromium.org,brettw@chromium.org,yfriedman@chromium.org
BUG=412673

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

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

This reverts commit aefdf297.

There was a logic bug.

BUG=408911
TBR= bruening

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

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

Revert of Non-plafrom-specific part of an OwnerSettingsService is moved to components/ownership/*. (patchset #4 id:60001 of https://codereview.chromium.org/548323003/)

Reason for revert:
Breaks compile on multiple bots:

http://build.chromium.org/p/chromium.win/builders/Win%20Builder%20%28dbg%29/builds/3134
http://build.chromium.org/p/chromium.mac/builders/Mac%20Builder%20%28dbg%29/builds/73424
http://build.chromium.org/p/chromium.webkit/builders/GPU%20Win%20Builder%20%28dbg%29/builds/13365

Original issue's description:
> Non-plafrom-specific part of an OwnerSettingsService is moved to components/ownership/*.
>
> BUG=398856
> TEST=existing browser_tests and unit_tests
> R=erg@chromium.org, jochen@chromium.org, nkostylev@chromium.org, pastarmovj@chromium.org
>
> Committed: https://chromium.googlesource.com/chromium/src/+/46a58bffbe303d3484d7cc288b47bfae5388109c

TBR=jochen@chromium.org,erg@chromium.org,nkostylev@chromium.org,pastarmovj@chromium.org,ygorshenin@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=398856

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

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

BUG=

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

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

Make sure this flag can be used for out of sandbox plugin setup.

BUG=

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

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

https://skia.googlesource.com/skia/+log/1f777e8580a67f2f975151bc414a1695696aa635..83658305a1bda2d862f50145ecf5cb697b575e86

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

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

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

Maintain the current hash associated each Page's state and use that rather than
querying location.hash directly. This fixes several long-standing issues with
the settings search page and the back/forward list:

- Opening an overlay from a search page no longer keeps the fragment in the
  URL.
- Closing an overlay no longer loses the search in the URL.
- Navigating back from one search page to another actually works.
- Navigating back from the search page doesn't clear the search box.

Add tests for this behavior.

BUG=410204

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

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

This binds mojo's html_viewer to a compositor implementation that uses surfaces
to present. It reuses the bindings for the WebLayer* types in cc/blink and adds
a new implementation of blink::WebLayerTreeView that shares some things with
content/renderer/gpu/render_widget_compositor.cc but is distinct enough to be
its own class. Whenever the compositor resizes (and thus needs a new surface id)
it passes the surface id to the view manager client lib.

This also adds mojo bindings for a few new quad types that are used on the web.
We don't yet have support for video quad types, but we don't have the media
system hooked up to html_viewer yet either so these will not be generated.

Committed: https://crrev.com/2abb46fdc18d2110b443a9e570ea393e3b0a42c5
Cr-Commit-Position: refs/heads/master@{#294546}

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

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

This CL includes the following changes:
1. Reduces the interval of media time update msg to 16 ms
2. Use interpolation to calculate the current playback time, and guarantee that time is always incrementing

BUG=327425

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

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

Revert of Generalize scroll parent work in CalculateDrawProperties (patchset #6 id:160001 of https://codereview.chromium.org/548963002/)

Reason for revert:
This patch seems to be exposing some bugs. They bugs appear not to be directly related to this patch, but I'm going to revert for now until I can investigate.

Original issue's description:
> Generalize scroll parent work in CalculateDrawProperties
>
> Previously, CDP would fail if the scroll child wasn't the immediate
> descendant of the lowest common ancestor of the scroll parent and child.
>
> With this patch I've fixed up two things so that CDP can handle these
> cases.
>
> 1. Dumber sorting.
>   I've made sorting simpler by removing assumptions about the tree
>   topology. Instead, I assigned "sort weights" to the layers in
>   PreCalculateMetaInformation. The approach is pretty naive. As I
>   walk the tree I assign a monotonically increasing weight. If I ever
>   hit a scroll child before its parent, I jump up to the lowest common
>   ancestor and make sure that the child of the LCA is assigned the
>   smallest weight in the tree so far (the negative of the next
>   monotonically increasing weight). This ensures that the scroll parent
>   will be visited first. Here's a picture of a layer tree and the
>   weights that would be assigned.
>
>   lca(0)
>   + filler(1)
>   | + scroll child(2)
>   + more-filler(-3)
>     + scroll parent(4)
>
>   It would be a huge bummer if we had to sort all the time, though, so
>   I've taken care to flag the LCA if I ever have to assign a negative
>   weight so that we only sort in that case.
>
>   Here's my argument for the correctness of the algorithm. Clearly,
>   we force correct ordering of a particular scroll child/parent pair,
>   but the worry is that by setting a negative weight, we'll force an
>   illegal ordering (i.e., we'll "undo" an ordering we've forced
>   earlier in the algorithm). Here's a diagram.
>
>   lca
>   + subtree with scroll parent #1
>   + subtree with scroll child #1 and scroll_parent #2
>   + subtree with scroll child #2.
>
>   Now, say we're processing scroll child #2. We want to be sure that
>   we don't go reordering subtrees and mess up the relative ordering of
>   scroll child #1 and scroll parent #1's subtrees.
>
>   But we know that this is impossible. Since we've already visited the
>   subtree with scroll child #1, scroll child #2 will necessarily get a
>   higher sort weight, so we'll never need to force any reordering in
>   this case; we won't assign any negative weights here.
>
>   Say the ordering were changed such that when processing scroll child
>   #2, we haven't seen scroll child #1's subtree. We're also fine in
>   this case because we will do any ordering fixups required when we do
>   eventually visit scroll child #1.
>
>   The only case where we get into trouble is when we have a cycle. For
>   example,
>
>   lca
>    + scroll parent 1
>    | + scroll child 2
>    + scroll parent 2
>      + scroll child 1
>
>   No ordering could satisfy a situation with a cycle, but this is
>   an impossible situation as CSS stacking defines a well defined, non
>   circular order.
>
> 2. Proper clip transformations.
>   We sometimes need to put the scroll parent's clip into the space of
>   the scroll child's immediate parent. This needs to go through the
>   LCA. Previously, we'd assumed that the LCA was going to be the scroll
>   child's immediate parent, but we no longer make that assumption.
>
> BUG=403866
>
> Committed: https://crrev.com/7faedca87705b714845842e595bca262c4090fa4
> Cr-Commit-Position: refs/heads/master@{#294521}

TBR=danakj@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=403866

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

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

BUG=412954

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

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

BUG=398856
TEST=existing browser_tests and unit_tests
R=erg@chromium.org, jochen@chromium.org, nkostylev@chromium.org, pastarmovj@chromium.org

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

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

The shared memory support functions weren't getting exported properly, and the right targets didn't depend on it. This is the last component build failure in the GN build as of now.

R=dpranke@chromium.org

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

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

third_party/lss includes headers that are broken on Android.
Don't use LSS on Android and disable the corresponding test.

BUG=413635

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

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

GN handles adding platform-appropriate prefixes to library
entries. As such, doing it here leads to stuff like '-l-lpython'
winding up in generated build files.

BUG=388412
TEST=use GN to generate build files for something that depends on cython
R=qsr@chromium.org

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

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

Keep the wrench menu open for 300ms after dropping an extension action in it
to provide the user with visual feedback that the extension was properly moved.

BUG=412861

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

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

The linux builder is failing to checkout from github.

No trybots because this only affects the SDK waterfall builder (trybots don't
build the appengine demos).

BUG=412969
R=bradnelson@google.com, sbc@chromium.org
NOTRY=true

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

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

BUG=331299

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

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

This patch adds a flag indiciating whether we were OOM the last time
we did an assign.

R=reveman, nduca
BUG=408758, 410398

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

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

DeviceMotionAndOrientation.java has been renamed to DeviceSensors.java
(see https://codereview.chromium.org/236833003/) but the owners file has
not been updated.

BUG=

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

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

This reverts commit 2abb46fd.

The patch was causing flaky compile failures on the GN try servers.

TBR=jamesr@chromium.org
BUG=

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

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

The "conversion" from NativeEvent to ui::Event logically belongs in the
PlatformWindow, just prior to dispatch into aura. This is where the
analogous process happens for other platforms such as X11.

So do it there, using a helper function, and write a blurb about why we
need to do this (short answer: for compatibility with existing code,
while we working on fixing that code).

BUG=none
TEST=chrome --ozone-platform=caca, dri
NOTRY=true

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

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

R=miguelg@chromium.org, bauerb@chromium.org
BUG=413665

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

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

BUG=none
R=
TEST=none
NOTRY=true
(documentation only change)

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

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

We should invalidate the input buffer index, or otherwise the new MediaCodec will call queueInputBuffer on an unitialized buffer

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

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

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

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

These classes do not need to be classes and expecially don't need to be
abstract classes as this leads to a complicated implementation and
complicated tests. All USB devices no matter the platform will have the
same descriptor data.

This change follows the model of device::HidDeviceInfo.

BUG=

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

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

When cretaing an InterfaceRequest for an interface that have a client,
there must be a way to provide the client implementation.

R=ppi@chromium.org

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

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

Changes the Courgette vector implementation to slightly
over-reserve the size of the vector, which makes many future
reserve operations no-ops (as they're quite expensive otherwise,
since they copy the full contents of the previous buffer).

When applying the patch from 35.0.1916.114 to 37.0.2062.120,
before and after this change, the runtime goes from 1m10s to 21s
on my z620.

Slightly higher multipliers produce even better results (e.g. 19s),
but this seemed like a reasonable value to chose so that it doesn't
result in significant additional memory use by Courgette.

BUG=167622
TEST=Build courgette target in Release mode and run it as
courgette.exe -apply chrome.7z chrome_patch.diff out.7z
(where chrome.7z was from un7zipping a 37.0.2062.94 chrome
installer and chrome_patch.diff was from un7zipping a
37.0.2062.94 -> 37.0.2062.120_37 chrome_updater_3stage.exe).
With the patch, the operation should run ~5x faster.

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

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

Move Cronet Java tests from Cronet Sample App to Cronet Test App.

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

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

BUG=413618
TBR=asargent@chromium.org

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

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

This is a reland of dd7edfa6 which was reverted in 80a9a88f for build
failures. It also includes a BoringSSL roll and build changes to avoid adding a
static initializer.

This is a much much larger change than the diff suggests. If it breaks
something, please revert first and ask questions later.

BUG=338885
TBR=brettw

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

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

Accessibility improvement.

BUG=326793

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

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

Instead of opening a popup in the wrench menu, extension actions that are
overflowed should close the wrench menu and then use the wrench menu button as
a reference view for the popup.

BUG=413003

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

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

The feature is always on regardless. The corresponding CL for removing the
runtime flag in blink is https://codereview.chromium.org/562403002

R=jochen@chromium.org

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

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

* Move CrOSInterface from backends into platform (CrOSPlatformBackend)
* Introduce Device/CrOSDevice.
* Replace IsRunningOnCrosDevice with GetHostPlatform().GetOSName()

BUG=413637

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

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

This pulls in the following Native Client changes:

r13731: (bradnelson) Turn on goma for nacl and switch windows to ninja, fix tls_edit weirdness.
r13732: (mcgrathr) Update GCC toolchains to r13729
r13733: (dschuff) Build x86-32 IRT with PNaCl

BUG=none
TEST=browser_tests and nacl_integration
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_cross_compile,linux_nacl_sdk,linux_chromium_dbg

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

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

This namespace was used for transition between old apps v1 and v2, which is
completed. There is no need to keep this redundant layer.

TEST=Manually tested Files app, Gallery and a Video player.
BUG=412653

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

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

This makes MediaStreamTrack::getSources go through UserMediaClient instead, needed to know which frame the call came from.

Blink CL: https://codereview.chromium.org/560843002/

BUG=406094

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

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