GitLab

BUG=631087

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

This is in preperation for CodeMirror-based console

BUG=607690
NOTRY=true

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

Integration between newly indroduced trace buffer writer
and proto generated stubs.

BUG=608719

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

BUG=637257

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

The following tests are flaky on all platforms:
- DumpAccessibilityTreeTest.AccessibilityIframeTransform
- DumpAccessibilityTreeTest.AccessibilityIframeCoordinates
- DumpAccessibilityTreeTest.AccessibilityIframeCoordinatesCrossProcess
- DumpAccessibilityTreeTest.AccessibilityIframeTransformCrossProcess

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=content_browsertests&tests=DumpAccessibilityTreeTest.AccessibilityIframe

NOTREECHECKS=true
NOTRY=true
BUG=622387
TBR=dmazzoni@chromium.org

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

It's confusing whether to add new accelerators to AcceleratorController vs.
AcceleratorControllerDelegateAura. Add some comments.

BUG=none
TEST=none

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

This CL reorganizes classes that derived from LocationArbitrator to instead
derive from LocationProvider. There was a quite a bit of overlap between the
two interfaces, and merging the two in favor of LocationProvider makes later
work with Blimp much easier.

Other work included:
*Addresses TODO to remove CreateArbitrator in favor of SetArbitratorForTest

BUG=614486, 637869

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

BUG=b/31003541

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

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

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

Change makes for more modular architecture. This allows the user of the webserver to build layers of ResponseProviders instead of creating a single monolithic provider.

BUG=none

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

Overview mode is probably more useful to people who are trying to use
a touch screen. Simply no-op on Alt+Tab when the virtual
keyboard's showing.

BUG=638269

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

Synchronize hit-testing code with rendering code with regards to how the
coordinate space is handled (for userSpaceOnUse <clipPath>s), and how
the reference boxes is computed.

Also "straighten" out code-flow a bit while here.

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

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

BUG=572740
R=davidben@chromium.org

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

Covered both unconditional firing and firing for captured pointer ids.

BUG=640637
TBR=isherman@chromium.org

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

R=chrisha@chromium.org
TBR=primiano@chromium.org
BUG=641242

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

This is a preparation to remove WTFLogAlwasy() from Node.cpp.
This CL has no behavior changes.

BUG=638849

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

In the renderer, we need to parse the extension schema string and pass
it to JS in order to create our bindings. However, right now, we first
parse this into a DictionaryValue, and then convert it into a
v8::Object. However, now that we fully generate all the schema code, we
should be able to parse it directly from a string into a v8::Object.

BUG=640247

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

BUG=638930

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

BUG=497340

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

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/f1a276cfabcd1f1cc950e36e66a2a50a664ce57f Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/7f070732c4b67f610e43d25dfd4ecd61788c15a9 remote_run: respect recipe build property (phajdan.jr@chromium.org)
depot_tools:
  https://crrev.com/5224ece27c41fab3633feefdec9153786624a66d git_bootstrap.py: add a switch to control client bitness (phajdan.jr@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG=641248,593999,635421,629679

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2284653002
Cr-Commit-Position: refs/heads/master@{#414722}

When precache manifest is fetched, the list of used and unused resources
corresponding to the top host are sent as request parameters in the
manifest URL. The resource URLs are hashed individually and sent as
base64 encoded string.

BUG=626463

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

This CL adds a a return value representing whether the element was found to the
TapWebViewElementWithId method.

BUG=none

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

glTexStorage* work incorrectly if the GL_TEXTURE_BASE_LEVEL is not 0.
This CL simply sets it to 0 beforehand and restores it afterward.

BUG=640506
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/2272823004
Cr-Commit-Position: refs/heads/master@{#414719}

Fixes IOSChromeContentSuggestionsServiceFactory::BuildServiceInstanceFor
return types by std::moving them, as the declared return type is a
superclass of the actually returned types.

BUG=None

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

HTC's mail apk calls evaluateJavaScript() inside
InputConnection#setComposingText(). This is problematic with ImeThread
feature since InputConnection methods are now running on its own thread
(IME thread). This causes a thread check assertion in Android's framework
(android.webkit.WebView).

As a workaround, we set sEnforceThreadChecking to false immediately after
it was set in WebView's constructor.

HTC is actively fixing their apk to avoid this crash, but as a last resort,
this workaround will help avoid such a crash.

BUG=636717, 622151

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

R=inferno@chromium.org, kcc@chromium.org, mbarbella@chromium.org, ochang@chromium.org
BUG=640627

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

Don't use properties of the above-the-fold view holder to
calculate how the peeking card should be, as it is sometimes
not already attached to the recycler view and thus not
accessible.

BUG=633511

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

It is not needed as 'generic_sensor' is build as a source set and it causes linker warnings on windows component build.

BUG=640769

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

Mark the policy as future, and add the "admin-sharing" tag.

BUG=639155

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

The return value is whether it's enforced by policy, the out param is whether it's enabled (not whether it's enforced, which made it an awfully confusing name).

R=ananta@chromium.org, jwd@chromium.org

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

BUG=640080

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

There is currently a single callback provided to PlatformScreen for when
new displays are added. We need to be able to notify not only when new
displays are added but also when displays are modified or removed.
Instead of having three callbacks, the PlatformScreenDelegate interface
is added.

The ui::ws::DisplayManager class implements PlatformScreenDelegate. This
is ultimately the class that needs to know about changes to displays so
it can update root windows accordingly.

There are some changes to ownership and constructor parameters to
facilitate having DisplayManager implement PlatformScreenDelegate. The
WindowServer instance is passed into DisplayManager, as it doesn't make
sense that Display objects owned by DisplayManager are allowed to know
about WindowServer but DisplayManager is not. The DisplayManagerDelegate
interface is simplified and changed to UserDisplayManagerDelegate as
it's no longer needed for DisplayManager.

The instantiation of SurfaceState is moved from Service to WindowServer.
There is no need to have Service know about SurfaceState anymore. Also,
SurfaceState will be moving to the GPU processes regardless.

BUG=641012

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

BUG=None

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

Previous: https://codereview.chromium.org/2241973002/

Original issue: https://codereview.chromium.org/2240573004/

Suppress all existing lint warnings so that builds will fail when new warnings
are generated. New lint warnings that are generated in files that are ignored
in suppressions.xml will not be caught. However, now we can remove lint
warnings on a file by file basis instead of having to fix all lint warnings
before making lint break builds.

BUG=607761

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

BUG=627517
TBR=

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

Encrypted media browser tests involving Widevine use the stub CDM when
branding != Chrome.  This change allows them to register the
preinstalled component containing the stub CDM.

BUG=622273

TEST=Widevine browser tests 'browser_tests --gtest_filter=*Widevine*' pass

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

Update expectations to match current flakiness:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=virtual%2Fthreaded%2Ftransitions%2F

BUG=248938
TBR=ajuma@chromium.org
NOTRY=true
NOTREECHECKS=true

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

BUG=None

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

The code stays around for now, so we can turn it on via a variation param if required.

BUG=640566

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

NOTREECHECKS=true
NOTRY=true
TBR=tapted@chromium.org,avi@chromium.org
BUG=640457

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