GitLab

Moved typedefs out from anonymous function because typedefs can
only be defined at the top level, not inside functions.

R=thestig@chromium.org
BUG=487804

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

Upstream some scripts used to simplify the setup of a new Chromium on iOS
checkout and update the instruction to remove obsolete informations, and
add recommendation to use the script.

BUG=None

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

BUG=636710

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

This ensures that there'll be a compile error if we add a new category but don't update the metrics code.
Also, add a histogram suffix for PHYSICAL_WEB_PAGES which was missing.

BUG=none

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

1. Disabled msa by default
2. Removed compiler flags

R=dpranke@chromium.org
BUG=

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

Use modified Read and delegate methods from the following
CL - https://codereview.chromium.org/2262653003/

BUG=423484

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

Add existing interfaces with zcr prefix and mark old zwp interfaces
as deprecated.

No change in behavior.

BUG=631960
TBR=piman@chromium.org

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

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

This change uses the ready promise on a composited transform animation
to determine whether the compositor has synchronised with the main thread.

This is the same as https://codereview.chromium.org/2309413002 but for
similar tests.

BUG=647776

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

BUG=none

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

This CL modifies clients that use URLRequest::Read() method
or implement net::Delegate/net::NetworkDelegate methods
and makes them to use returned networking errors instead,
which are returned in term of int.

Old implementation of URLRequest::Read() used to set its
status inside the URLRequest object itself and clients
used to check the status by calling request->status().

The main CL that changes URLRequest::Read and delegate
methods is here - https://codereview.chromium.org/2262653003

BUG=423484

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

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

This is in preparation for the new prompt that is planned for plugins. This
adds all the boilerplate needed for the new permission.

BUG=641615

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

In mus, the client should get the ICCProfile from the mus server,
instead of from the native platform. So get the ICCProfile from
a virtual function so that the platform can override it.

BUG=647510

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

This fixes a typo introduced r419404, where the GN label for
webapk_shell_apk_junit_tests had one too many leading slashes.

TBR=brettw@chromium.org
NOTRY=true
BUG=555273

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

BUG=

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

dom-if and similar stamp their templates asynchronously, because it's
not a great idea to block execution to stamp out DOM as a result of
changes to properties. cr-lazy-render does not really share the same
concerns: it will only stamp when explicitly told to, and the caller
should be aware of how long this could take. As such, there is no
particular reason to prefer async stamping.

This CL changes all existing usage of cr-lazy-render to be synchronous,
which simplifies both application code and tests. An asynchronous API
could still be added in the future, if required.

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

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

If a native app and a PWA are installed which handle the same URLs,
a navigation within the PWA may open the intent picker to ask the user
if they wish to browse in the native app.

This CL stops the picker from appearing if the user is browsing within
the fullscreen PWA. It does not do so if the user is browsing the PWA
outside of the fullscreen standalone (i.e. launched from homescreen)
mode.

BUG=647569

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

Removed --enable/disable-experimental-app-list and
--enable-centered-app-list flags. Now always behaves as if experimental
(which was the default). This effectively removes the old
lower-left-corner tall app launcher (although all the code is still
there; it will be removed in a follow-up CL).

This cleans up the tests that set those switches, and removes tests that
test the old app launcher behaviour.

BUG=531059

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

This change re-lands r411530, with a bunch of additional targets
added to gn_isolate_map.pyl to handle all of the other compile and
test targets (including mappings for apk's and junit tests).

TBR=brettw@chromium.org
BUG=555273

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

ContainerViewObserver is used by ExternalVideoSurfaceContainer
only, which in turn used by Chromecast/WebView.

Since Chromecast never updates container view in its lifecycle,
it can go without using ContainerViewObserver. And WebView
disabled support for video hole since M52.

With that, ContainerViewObserver is not used any more. This CL
deletes the class.

BUG=624977

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

https://chromium.googlesource.com/skia.git/+log/5d1110ed4679..e7a7812744c2

$ git log 5d1110ed4..e7a781274 --date=short --no-merges --format='%ad %ae %s'
2016-09-18 rmistry Whitespace change

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
TBR=borenet@google.com

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

Currently DeleteWindow() is (mostly) ignored if the client doesn't own
the root. This change allows the client to call delete on a root and
it's treated as unembedded the client from that root.

BUG=647509
TEST=covered by tests
R=sadrul@chromium.org

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

After discussion concluded the best way was to delegate the laser pointer out of ash/common. Moved all the laser related files into ash/laser.

LaserPointerController.* replaces LaserPointerMode.* and it inherits from EventHandler to grab the events and propagate them to LaserPointerView/LaserPointerPoints, which remain the same. LaserPointerMode just inherits common palette tool and calls the delegate.

BUG=644804
TEST=ash_unittests --gtest_filter="LaserPointer*"

https://screenshot.googleplex.com/BtwX1GTCpcW

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

Avoid mash null dcheck/deref with --login-manager.

BUG=647781,647039
TEST=No crash running chrome --mash --login-manager
R=alemate@chromium.org,xiyuan@chromium.org

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

https://chromium.googlesource.com/skia.git/+log/bdf3e5c03418..5d1110ed4679

$ git log bdf3e5c03..5d1110ed4 --date=short --no-merges --format='%ad %ae %s'
2016-09-18 update-skps Update SKP version

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
TBR=borenet@google.com

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

BUG=631960,648028
TBR=piman@chromium.org

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

In Desktop OGLs whose version < 4.4, they maybe don't enable srgb conversion
between srgb color format and linear color format. This change can emulate
srgb and do the conversion for machines with these OGL profiles.

BUG=634525
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/2286593002
Cr-Commit-Position: refs/heads/master@{#419396}

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

This cl splits the big CL with changes to net/ folder and
has changes only to net/cert_net according to the
changes to URLRequest API.

What the big cl does:
It modifies net/ clients that use URLRequest API as long as
URLRequest::Read returns int net errors and
URLRequest::Delegate and NetworkDelegate methods
(for example, OnResponseStarted or OnCompleted) have int
net_error in the arguments now.

The reason behind splitting the CL into small one is that
an android bot started to be unstable and unittests became
flaky. It was not possible to locate the problem and the
decision was to split the CL and upload small parts with a
6+ hours interval in order to make it possible to locate
the problematic code.

The big CL is located here -
https://codereview.chromium.org/2265873002/

BUG=423484

Committed: https://crrev.com/a74d3cbc572fa1c85cc2c0434a9f8e2f9cba0cd9
Review-Url: https://codereview.chromium.org/2330863002
Cr-Original-Commit-Position: refs/heads/master@{#418014}
Cr-Commit-Position: refs/heads/master@{#419394}

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

This patch adds vrdisplayconnect and vrdisplaydisconnect event support
of google vr sdk for android. It defines 'connect' status as
nativeContext is available and defines 'disconnect' status as
nativeContext is unavailable.

Note that connect event may dispatch before user get vr devices
because devices in connection status could already deliever user some device
info and user could use this event to do some preparing work.

BUG=389343
R=bajones@chromium.org, kenrd@chromium.org, ochang@chromium.org

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

BUG=647820,641261

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

Fold uses of:

  CSSPrimitiveValue::create(..., const ComputedStyle&)
  CSSValuePair::create(..., const ComputedStyle&)

into the (few) users. Handle the fall-out.

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

The new way is simpler and uses the same pattern as PaintLayerCompositor.

BUG=646188
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

TBR=rtoy@chromium.org,foolip@chromium.org
BUG=626449,647693

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

This is a partial revert of the WebViewPlugin changes in
https://crrev.com/8489f1bf78c2c6f296e0df6c529751d29e8a0188

BUG=641028

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

This cl splits the big CL with changes to net/ folder and
has changes only to net/cert_net according to the
changes to URLRequest API.

What the big cl does:
It modifies net/ clients that use URLRequest API as long as
URLRequest::Read returns int net errors and
URLRequest::Delegate and NetworkDelegate methods
(for example, OnResponseStarted or OnCompleted) have int
net_error in the arguments now.

The reason behind splitting the CL into small one is that
an android bot started to be unstable and unittests became
flaky. It was not possible to locate the problem and the
decision was to split the CL and upload small parts with a
6+ hours interval in order to make it possible to locate
the problematic code.

The big CL is located here -
https://codereview.chromium.org/2265873002/

BUG=423484

Committed: https://crrev.com/a74d3cbc572fa1c85cc2c0434a9f8e2f9cba0cd9
Review-Url: https://codereview.chromium.org/2335513003
Cr-Original-Commit-Position: refs/heads/master@{#418014}
Cr-Commit-Position: refs/heads/master@{#419386}

Revert of Evacuate ComputedStyle references from the CSS*Value hierarchy (patchset #1 id:1 of https://codereview.chromium.org/2345893004/ )

Reason for revert:
Tree is red; this patch fails to compile:

../../third_party/WebKit/Source/core/css/CSSValuePair.h:42:12: error: incomplete result type 'WTF::String' in function definition
    String customCSSText() const

Original issue's description:
> Evacuate ComputedStyle references from the CSS*Value hierarchy
>
> Fold uses of:
>
>   CSSPrimitiveValue::create(..., const ComputedStyle&)
>   CSSValuePair::create(..., const ComputedStyle&)
>
> into the (few) users. Handle the fall-out.
>
> Committed: https://crrev.com/ba9df880383908b052d5a7d00abb32deec94cfc5
> Cr-Commit-Position: refs/heads/master@{#419382}

TBR=sashab@chromium.org,timloh@chromium.org,fs@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

This drops includes from CSSValue.h that are not directly used by it.
Downstream dependents are fixed up as needed. Also strip some includes
from StyleImage.h and other places that includes CSSValue.h while
there.

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