GitLab

The code checks both the runtime flag and the origin trial status, when
the origin trial check implicitly checks the runtime flag first.
Removing this antipattern will allow us to eventually mark the
inappropriate checks as private, and reduce inadvertent bugs.

Bug: 799949
Change-Id: I8af29c46a0f5eefacb366e87d378c86b7d4ac71f
Reviewed-on: https://chromium-review.googlesource.com/891573Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532975}

OnSurfaceDestroyed was unconditionally dereferencing
|target_surface_bundle_|, but this can be cleared once the codec
enters a terminal state.  A null target bundle should just be
ignored anyway.

Bug: 806630
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_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
Change-Id: I612c8a0c95ec0e3b918789384f44ad3a01fd2559
Reviewed-on: https://chromium-review.googlesource.com/893638Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Commit-Queue: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532974}

For some rare cases, service_ in CheckClientDownloadRequest is already
set to null when OnDownloadDestroyed is called, which as a result will
make item_ still dangle there, and causes crash at the DCHECK(item_==NULL).

Bug: 806785
Change-Id: I384f66ca94d6695d870a632d52b9be26a0f9a3a3
Reviewed-on: https://chromium-review.googlesource.com/891808Reviewed-by: Jialiu Lin <jialiul@chromium.org>
Reviewed-by: Luke Z <lpz@chromium.org>
Commit-Queue: Jialiu Lin <jialiul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532973}

This fixes the problem where Feature Policy can be independently
disabled through runtime flags (or chrome://flags), while the features
which depend on it, such as FeaturePolicyForPermissions, remain enabled.

Bug: 806362
Change-Id: Ied92dc74022a02f22a11c3ce68bd62c1aa98eaae
Reviewed-on: https://chromium-review.googlesource.com/893598Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532972}

It looks like we sometimes recompute the root scroller on an
HTMLFrameOwnerElement that's still connected to the DOM tree but doesn't
have a ContentFrame. One way this can happen is through layouts caused
from unload handlers.

This CL adds an early out in ApplyRootScrollerProperties since a
detached frame doesn't need it's properties reset.

Bug: 805317
Change-Id: I6d285b10a69f74f05604bda5f654fca96fc35238
Reviewed-on: https://chromium-review.googlesource.com/887733
Commit-Queue: David Bokan <bokan@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532971}

This CL:
- Replaces calls to
  ExternalNavigationDelegateImpl#isSpecializedHandlerAvailable() with calls to
  #countSpecializedHandlers() because #isSpecializedHandlerAvailable() has just
  one caller
- Replaces ChromeBrowserTestRule with NativeLibraryTestRule
- Uses a mock context for both ContextUtils#getApplicationContext() and for
  TabRedirectHandler instead of just for TabRedirectHandler
- Makes TestPackageManager#queryIntentActivities() and
  TestExternalNavigationDelegate#queryIntentActivities() use same
  implementation.
- Changes tests to register relevant http:// and https:// handlers at the
  beginning of the test. This is a change of behavior. Previously, most URLs
  would be handled by the "foo" external app
- Changes TestExternalNavigationDelegate#mQueryIntentOverride from tristate to
  boolean. The "true behavior" was nuked but the "false behavior" was
  preserved.

BUG=None

Change-Id: If212afde6e4c12d4f08576c4bd26392b8732bc09
Reviewed-on: https://chromium-review.googlesource.com/891744
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532970}

Our SQLite VFS [1] wrapper implementation unconditionally wraps
xCurrentTime(). However, when SQLite is compiled with the
SQLITE_OMIT_DEPRECATED option [2], xCurrentTime() is null for the VFS
implementations that ship with SQLite, because xCurrentTime() was
deprecated in favor of xCurrentTimeInt64().

This CL replaces the unconditional wrapper with a conditional wrapper.
This ensures that our VFS wrapper exposes an xCurrentTimeInt64()
implementation when used with a modern SQLite library (like the one
bundled with Chromium), and an xCurrentTime() implementation when using
an older SQLite version (which might be the case when using the library
bundled with the operating system).

[1] https://www.sqlite.org/c3ref/vfs.html
[2] https://www.sqlite.org/compile.html#omit_deprecated

Bug: 807093
Change-Id: Id26cc4517a1a23692e7860ed620348c027db990f
Reviewed-on: https://chromium-review.googlesource.com/892484Reviewed-by: Chris Mumford <cmumford@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532969}

The i18n custom bindings create v8 values from results for language
detection, but do so in an inefficient way (constructing a base::Value
and then converting to v8::Value). Update this to use
gin::DataObjectBuilder to construct the v8 value directly, and, while
we're at it, use some C++11 awesomeness in movable types.

Bug: None
Change-Id: I80597627b0b3803e50ecbe34009682161d178c7d
Reviewed-on: https://chromium-review.googlesource.com/892202Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532968}

Change-Id: I26c2a054b9f1e5eb076acd677e1223058825f6d6

Bug: 803416
Test: fast/loader/document-open-iframe-then-detach.html
Change-Id: I26c2a054b9f1e5eb076acd677e1223058825f6d6
Reviewed-on: https://chromium-review.googlesource.com/887298Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532967}

Change-Id: Idf6dd882d2d50b1ec349f245d76717553ee266b3
Reviewed-on: https://chromium-review.googlesource.com/882002Reviewed-by: Robert Flack <flackr@chromium.org>
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532966}

Bug: 724160
Change-Id: I9ede628f60f2ff32d09bcec9ef365db9246f4c98
Reviewed-on: https://chromium-review.googlesource.com/871046
Commit-Queue: Hidy Han <hidyhan@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Mustafa Emre Acer <meacer@chromium.org>
Reviewed-by: Eric Seckler <eseckler@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532965}

https://webrtc.googlesource.com/src.git/+log/79d331b09102..cf30d8b1ec22

$ git log 79d331b09..cf30d8b1e --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtc
BUG=NONE,None


The AutoRoll server is located here: https://webrtc-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: I30491d79540eafe636e555734277e417854399fb
Reviewed-on: https://chromium-review.googlesource.com/893525Reviewed-by: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#532964}

There is an edge case where SortNetworkList may destroy the default
network, causing a crash. This CL fixes that.

Bug: 805322
Change-Id: I3d880dacc567babea08d7b9f21c5c8759e290f83
Reviewed-on: https://chromium-review.googlesource.com/883971
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532963}

Bug: 807049
Change-Id: Ia223724629db4f04a676a023a4dbda365a0dbba3
Reviewed-on: https://chromium-review.googlesource.com/893719Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Commit-Queue: Robert Ogden <robertogden@google.com>
Cr-Commit-Position: refs/heads/master@{#532962}

https://pdfium.googlesource.com/pdfium.git/+log/af9f83656f98..1917cdd8c90b

$ git log af9f83656..1917cdd8c --date=short --no-merges --format='%ad %ae %s'
2018-01-30 dsinclair Cleanup some CPDFSDK_Widget XFA code.

Created with:
  roll-dep src/third_party/pdfium


The AutoRoll server is located here: https://pdfium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=dsinclair@chromium.org

Change-Id: Ic7870559e0596ac43919bf0732a649998e138926
Reviewed-on: https://chromium-review.googlesource.com/893599Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#532961}

The file is not named correctly. This CL uses the correct name suffix.

Bug: 721401
Change-Id: I9c0976bc802cee9f3c13141e6edb0b4ba81de97a
Reviewed-on: https://chromium-review.googlesource.com/891440Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532960}

This CL was uploaded by git cl split.

R=zhenw@chromium.org

Bug: 177475
Change-Id: I87d874d14490cd2476ef10f7b0aeaa058aa9273f
Reviewed-on: https://chromium-review.googlesource.com/889961Reviewed-by: Charlie Andrews <charliea@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532959}

The policy for test code in //ash is that it should live in the
directory of the code under test, which in this case is //ash/wm.

This fixes a TODO in //ash/BUILD.gn

Bug: none
Test: ash_unittests
Change-Id: Id40eaa137991efbb47b83c322eb6cf9c7bb405ec
Reviewed-on: https://chromium-review.googlesource.com/892198
Commit-Queue: James Cook <jamescook@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532958}

This reverts commit 53b1b7da.

Reason for revert: The new test webaudio/BiquadFilter/no-dezippering.html is failing under Mac 10.10.

Original change's description:
> Remove Dezippering from BiquadFilter
> 
> Remove dezippering from all of the attributes.  The value will now 
> change immediately instead of gradually changing from the old value
> to the new value.
> 
> Chromium Feature: https://www.chromestatus.com/features/5287995770929152
> Intent to ship: https://groups.google.com/a/chromium.org/d/msg/blink-dev/YKYRrh0nWMo/aGzd3049AgAJ
> 
> Bug: 752986
> Test: BiquadFilter/dezipper.html
> Change-Id: I86840159709158fd52a69a7c3c5f279267277d44
> Reviewed-on: https://chromium-review.googlesource.com/612104
> Commit-Queue: Raymond Toy <rtoy@chromium.org>
> Reviewed-by: Hongchan Choi <hongchan@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#532910}

TBR=rtoy@chromium.org,hongchan@chromium.org

Change-Id: I30c98f8aad3c8a1f7df22e55912a4f86d94d7dfa
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 752986
Reviewed-on: https://chromium-review.googlesource.com/893531Reviewed-by: Samuel Huang <huangs@chromium.org>
Commit-Queue: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532957}

Bug: 743173
Change-Id: Ie31e92bff4fda5907d45b931dd79caeac79b36ff
Reviewed-on: https://chromium-review.googlesource.com/669319Reviewed-by: anthonyvd <anthonyvd@chromium.org>
Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532956}

Bug: None
Change-Id: I47fc0e4a15e178072f65358ad8044def01ba1909
Reviewed-on: https://chromium-review.googlesource.com/889618Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Cait Phillips <caitkp@chromium.org>
Commit-Queue: Gayane Petrosyan <gayane@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532955}

This reverts commit 5ef6e548.

Reason for revert: net_unittests failing on android_cronet.

Original change's description:
> Reland "Make PUSH_PROMISE headers URL-parsing more robust and conform to HTTP/2 spec"
> 
> Reland of https://chromium-review.googlesource.com/c/chromium/src/+/887970.
> 
> MSAN use-of-uninitialized-value should be fixed.
> 
> Change-Id: I392ad1a83a269b47ff0c8583f6c72e18a4a0e5b9
> Reviewed-on: https://chromium-review.googlesource.com/891302
> Commit-Queue: Yixin Wang <wangyix@chromium.org>
> Reviewed-by: Ryan Hamilton <rch@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#532669}

TBR=rch@chromium.org,wangyix@chromium.org

Change-Id: I19d5f50c17b052f72cfa6025bcea2b4770363666
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/893864Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Commit-Queue: Yixin Wang <wangyix@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532954}

https://skia.googlesource.com/skia.git/+log/ab2621d3e2d2..c4e31092b13e

$ git log ab2621d3e..c4e31092b --date=short --no-merges --format='%ad %ae %s'
2018-01-30 reed don't draw images with nonfinite dst bounds
2018-01-30 angle-skia-autoroll Roll skia/third_party/externals/angle2/ efb5a5c8f..6969132c1 (1 commit)
2018-01-30 reed handle clipping large triangles

Created with:
  roll-dep src/third_party/skia


The AutoRoll server is located here: https://autoroll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


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=bungeman@chromium.org

Change-Id: I2027c9d264d8b24f3058c671f38dedde548edf45
Reviewed-on: https://chromium-review.googlesource.com/893642
Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#532953}

Creates a new binary, subresource_indexing_tool, that takes an unindexed
ruleset as input and writes an indexed ruleset as output.

Bug: 793025
Change-Id: Ieeb10aab4ba80c83baf8058fe202e4d4fcd770e1
Reviewed-on: https://chromium-review.googlesource.com/891140
Commit-Queue: Josh Karlin <jkarlin@chromium.org>
Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532952}

This change cleans up the logic in OnResolveProxy and its two helper
functions (one of which is removed) so that it is clearer what logic is
run only when a request is eligible for the data reduction proxy or only
when the request will be made through the data reduction proxy.
Duplicate checks are removed or the later ones turned into DCHECKs to
clarify assumptions.

Bug: 721403
Change-Id: I271a553e0c1406921c6a2a23ab21241ba3095df5
Reviewed-on: https://chromium-review.googlesource.com/887650
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532951}

Instead of a custom ClampInt() function.

Change-Id: I7dc852b0c681053e7b90ce3d5ef05a699d0f12f3
Reviewed-on: https://chromium-review.googlesource.com/891943Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532950}

This is also allowing navigations with gesture to carry the flag such as
middle click on links and CTRL+click. A side effect is to be able to
prevent non-renderer initiated navigations to carry the flag such as
reloads.

Bug: 799136, 759448
Change-Id: I25836f8a4e9aca5a60dc095c6d0a81d480027124
Reviewed-on: https://chromium-review.googlesource.com/873450Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Mounir Lamouri (slow) <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532949}

Currently, the extensions can go in an incorrect
hover state when they're being reordered via drag
and drop. This bug fixes it by preventing the buttons
to enter the hover state when a button is dragged.

Bug: 793572
Change-Id: I1275882b68bbafd901f5edeab4e5c9b93286e699
Reviewed-on: https://chromium-review.googlesource.com/874523Reviewed-by: Robert Sesek <rsesek@chromium.org>
Commit-Queue: Sarah Chan <spqchan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532948}

The mul/div operators are tricky. operator/ works when |a| is small
enough (in absolute value) to deterministically not risk overflow.

operator*() on the other hand doesn't because of a limitation in
__builtin_mul_overflow. We could do some template hacking to make
operator*(a) == operator/(1.0/a) when |a| is a constant expression.

Bug: 761570
Change-Id: I9906edfc049017ad19872e4a586b2ec675404850
Reviewed-on: https://chromium-review.googlesource.com/886344Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Yuri Wiitala <miu@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532947}

On android, adb is used to forward connections from the device to the
host, but there isn't a way to reserve the source port before deciding
which destination port on the host to forward it to. Change
RemoteTestServer reserve a port for AIA & OCSP, and forward it to another
port, like it already does on Fuchsia. Unlike Fuchsia, the second adb
forwarding step is still required to forward the connection from device to
host.

Change-Id: I739543fd0036f189928b5c7ce31c4d9a520e1cf0
Reviewed-on: https://chromium-review.googlesource.com/887171Reviewed-by: Scott Graham <scottmg@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Commit-Queue: Matt Mueller <mattm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532946}

Geolocation stops owning GeoNotifiers in some cases (e.g.
Geolocation::clearWatch), so it's possible that, when
GeoNotifier::TimerFired gets invoked, the owner geolocation
had already stopped owning the notifier.  In this scenario,
no one is performing wrapper-tracing for the notifier, thus
the underlying V8 functions might have already been
collected by V8 GC.  GeoNotifier must not invoke any
callback in such a case.

This patch adds a check whether the geolocation still owns
the notifier or not.

Bug: 792604
Tbr: haraken@chromium.org
Change-Id: I46f2d34f62bac2073b75eb36b935172cb1c10465
Reviewed-on: https://chromium-review.googlesource.com/893240
Commit-Queue: Adam Klein <adamk@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Adam Klein <adamk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532945}

The UIChromeOSTestSuite will fail some of the unit tests based on
views::ViewsTestBase. In the retries, the tests will pass.

By comparing UIChromeOSTestSuite and ViewsTestSuite, the only difference
is that UIChromeOSTestSuite does not init/clear up aura env.

This cl properly sets up and cleans up the aura env for UIChromeOSTestSuite.

Bug: 807148
Test: ui_chromeos_unittests
Change-Id: Ife3bae23c2003a6e219764f7483cdff7eb785c72
Reviewed-on: https://chromium-review.googlesource.com/892495
Commit-Queue: Tao Wu <wutao@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532944}

Change-Id: I3465c7c43e9f699e13eb1b4dddfa924ff4cb54a4
Reviewed-on: https://chromium-review.googlesource.com/881957Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532943}

services/network/ needs that helper class, and it can't depend on content/. While content/ can depend on services/network, this helper isn't related to the network service so it's moved to live beside the test server code.

Bug: 753658
Change-Id: Ied49b9e5dd45549d00e5684a9cb696155edacf13
Reviewed-on: https://chromium-review.googlesource.com/892292Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532942}

It's sometimes quite useful when debugging build failures to be able to
see the classpath.

Bug: 771347
Change-Id: Ic378b969e25f93d1325663c832fc76583ca15744
Reviewed-on: https://chromium-review.googlesource.com/893518Reviewed-by: Eric Stevenson <estevenson@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532941}

This works around an issue where DOMUtils injects its input through the
rootView (view.getRootView()), which causes the input to be swallowed
by the GvrLayout in VR (this is because in VR, screen touches are used
for headset alignment rather than user input).

DOMUtils clicks will optionally get injected directly into r.id.content
rather than the root of the view hierarchy.

We need to inject into r.id.content, rather that cvc.getContainerView()
because the CompositorViewHolder needs to be able to intercept the
events to correctly set the EventForwarder offsets. Gross.

Bug: 804808
Change-Id: I6b246262d1e7b64437382b67db0a0d10518ee839
Reviewed-on: https://chromium-review.googlesource.com/884010
Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532940}

Add the recent destinations list and search box, and enable selection
of printers using the dialog.

Bug: 773928
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I157461a6a2ec5223e88b97067a4ddc8933f08ea8
Reviewed-on: https://chromium-review.googlesource.com/885502
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532939}

Use GLSurfacePresentationHelper to support PresentationCallback in
DirectCompositionSurfaceWin.

Bug: 804970
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_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
Change-Id: I50931d5b59dfea70d5e333abf0398144c8823bb3
Reviewed-on: https://chromium-review.googlesource.com/881710
Commit-Queue: Peng Huang <penghuang@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532938}

The flag is still disabled by default.

Bug: 791806
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ibc438f98e359e7f4c1c86ce10c175e5b94344cef
Reviewed-on: https://chromium-review.googlesource.com/889642
Commit-Queue: Eugene But <eugenebut@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532937}

This CL adds dcastagna@ and reveman@ as OWNERS for //media
*gpu_memory_buffer* files, namely:

$ find media | grep gpu_memory_buffer
media/capture/video/chromeos/local_gpu_memory_buffer_manager.h
media/capture/video/chromeos/mock_gpu_memory_buffer_manager.h
media/capture/video/chromeos/local_gpu_memory_buffer_manager.cc
media/capture/video/chromeos/mock_gpu_memory_buffer_manager.cc
media/video/gpu_memory_buffer_video_frame_pool.cc
media/video/gpu_memory_buffer_video_frame_pool.h
media/video/mock_gpu_memory_buffer_video_frame_pool.cc
media/video/mock_gpu_memory_buffer_video_frame_pool.h
media/video/gpu_memory_buffer_video_frame_pool_unittest.cc

(Since those files are spread, I added the rule in the top-level file).

Change-Id: I469f5177db8c170a6b79d483e4a812610be1b74b
Reviewed-on: https://chromium-review.googlesource.com/892159Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532936}