GitLab

H264 is most affected since VP9 gets 4K portrait support by virtue
of supporting 8K decoding. This cl also adds 8K portrait mode
support for VP9.

BUG=806568
TEST=manually verified video in bug gets GpuVideoDecoder.

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: I4c622e3d3e54c43f85c6d3af0d7624f66a0639b6
Reviewed-on: https://chromium-review.googlesource.com/891670Reviewed-by: Frank Liberato <liberato@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532710}

This folder contains "library CDM" related code and will soon be
unrelated to ppapi.

This CL only renames the folder. More clean-up will follow in later CLs.

TBR=thestig@chromium.org
BUG=769833

Change-Id: I346a59d23838d5d4cd93bfea5afd3f28827e147f
Reviewed-on: https://chromium-review.googlesource.com/889693
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: John Rummell <jrummell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532709}

This reverts commit 3d6ada5e.

Reason for revert: wjmaclean@ landed potential fix in https://crrev.com/532661.
Let's see if it worked.

Original change's description:
> Disable flaky BasicSelectionIsolatedScrollMainframe
> 
> TouchSelectionForCrossProcessFramesTests/TouchSelectionControllerClientAuraSiteIsolationTest.BasicSelectionIsolatedScrollMainframe/1 fails on all the platforms.
> 
> TBR=tdresser@chromium.org
> 
> Bug: 805017
> Change-Id: I1ea7c82a4c8fc2fa8e83c83ac47ef65f98cd985f
> Reviewed-on: https://chromium-review.googlesource.com/888753
> Reviewed-by: Vasilii Sukhanov <vasilii@chromium.org>
> Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#532122}

TBR=vasilii@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 805017
Change-Id: I59c74f106b2c04632e0826512e75c938587eccbe
Reviewed-on: https://chromium-review.googlesource.com/892203Reviewed-by: Charlie Reis <creis@chromium.org>
Commit-Queue: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532708}

Use the proper BUILDFLAG() macro


Change-Id: I358fd008b5eb081d18782c66e0df904589435140
Reviewed-on: https://chromium-review.googlesource.com/889422
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532707}

The modifier 0 means linear layout (DRM_FORMAT_MOD_LINEAR) which is
different from the legacy behavior of not specifying a modifier.  When
a modifier is not specified, the exact layout (tiling, compression
etc) is communicated through some kind of driver specific back channel.

The few places that build a NativePixmapHandle from scratch want the
implicit legacy behavior, but passing 0 explicitly requests a linear
layout. We introduce gfx::NativePixmapPlane::kNoModifier (equal
to DRM_FORMAT_MOD_INVALID) to indicate that the caller wants the
legacy semantics.

Bug: b:70546200, b:71416883, chromium:789292, b:69238712, b:68003124
Change-Id: Ic7eb4838f53ffb30ede39709abd0bb4639223754
Reviewed-on: https://chromium-review.googlesource.com/885905Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Pawel Osciak <posciak@chromium.org>
Reviewed-by: Miguel Casas <mcasas@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
Commit-Queue: Kristian H. Kristensen <hoegsberg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532706}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/7a4ced2773b2..6fe29419beee

$ git log 7a4ced277..6fe29419b --date=short --no-merges --format='%ad %ae %s'
2018-01-23 bpastene depot_tools: Add a checkout_chromeos conditional var to gclient.

Created with:
  roll-dep src/third_party/depot_tools


The AutoRoll server is located here: https://depot-tools-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.


TBR=phajdan.jr@chromium.org

Change-Id: Iaabeafeddc47c8525e09a5f6bcfe990855ae7a14
Reviewed-on: https://chromium-review.googlesource.com/892025Reviewed-by: depot-tools-roller . <depot-tools-roller@chromium.org>
Commit-Queue: depot-tools-roller . <depot-tools-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532705}

Due to unspecified function argument evaluation order in C++ some
compilers might choose to evaluate base::Passed earlier than subsequent
smart pointer access, so we need to be extra careful and save raw
pointer values before using them. Otherwise they might no longer be
valid by the time we call ptr.get due to the pointer being already
modified by the base::Passed.

BUG=734325

Change-Id: Icb5a06c42c14018c98ab8ddaffcc1fd41f3111ab
Reviewed-on: https://chromium-review.googlesource.com/885091Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>
Commit-Queue: Sergey Volk <servolk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532704}

TBR=bsalomon@chromium.org

Bug: 807055
Change-Id: Iaefabbef9b09da1a03bdcb739688056095d0259f
Reviewed-on: https://chromium-review.googlesource.com/891649Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532703}

It's used in Chrome.

Bug: 733662
Change-Id: I538cccaf1282b8de2ded93c3c93cb52bb789e250
Reviewed-on: https://chromium-review.googlesource.com/882136Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Jiaquan He <hejq@google.com>
Cr-Commit-Position: refs/heads/master@{#532702}

This CL adds new elements to create self-contained sync-account element.
When a profile is DICE-enabled, the original avatar row is hidden,
and this new row will be shown instead.

Despite the new element and layout, the new avatar row will look
similar to the existing one after this CL. The intention is to make it
possible for follow-up CLs to actually change the account sync flow to
achieve the DICE design without affecting the existing avatar row for
non-DICE-enabled profiles.

Bug: 800970
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I9325a8d5986d0513e9d63bdac2ee9055823f7eba
Reviewed-on: https://chromium-review.googlesource.com/862982
Commit-Queue: Scott Chen <scottchen@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532701}

This CL sets -limited-coverage-experimental=true to allow building
large unit test target on Mac.

Bug: 796290
Change-Id: Ief6277ae2ea3d4e1372c4fd796e97c2a1ae0bfba
Reviewed-on: https://chromium-review.googlesource.com/891799
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Abhishek Arya <inferno@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532700}

These Instrumentation Test Runners will be removed soon

Bug: 711517
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I7af9ee4dc09e7c6e5b40548252c94c9869af6cbf
Reviewed-on: https://chromium-review.googlesource.com/887790Reviewed-by: Bo <boliu@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Paul Jensen <pauljensen@chromium.org>
Reviewed-by: Lambros Lambrou <lambroslambrou@chromium.org>
Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Commit-Queue: Yoland Yan <yolandyan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532699}

Bug: skia:7533
Change-Id: If19339a4a0aaa8ea655184db054058ea2697c6f3
Reviewed-on: https://chromium-review.googlesource.com/887381
Commit-Queue: Brian Salomon <bsalomon@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Florin Malita <fmalita@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532698}

The original failure reason is not included in the bug.
If the test was only  failing on device then it was not
disabled correctly.

The test has passed twice on the bots, so enabling it
to diagnose the problem.

Bug: 764691
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ie6787ca7124171c75d885322f77b124276723886
Reviewed-on: https://chromium-review.googlesource.com/892083Reviewed-by: Yuke Liao <liaoyuke@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532697}

https://pdfium.googlesource.com/pdfium.git/+log/bc4869293d78..95b232f79123

$ git log bc4869293..95b232f79 --date=short --no-merges --format='%ad %ae %s'
2018-01-29 dsinclair Cleanup some ChoiceList code
2018-01-29 dsinclair Add common base class between CXFA_FFComboBox and CXFA_FFListBox
2018-01-29 dsinclair Cleanup CXFA_Node visibility
2018-01-29 dsinclair Fold single caller methods into callsite
2018-01-29 dsinclair Move barcode code to CXFA_Barcode
2018-01-29 xlou Change pageNums from uint16_t to uint32_t
2018-01-29 tsepez Kill some bare |new|s in cjs_field.cpp
2018-01-29 dsinclair Cleanup the FFWidget determination code.
2018-01-29 thestig Clean up FPDF_GetFormType().
2018-01-29 thestig Remove redundant checks in CPVT_WordRange::Intersect().

Created with:
  roll-dep src/third_party/pdfium
BUG=805881


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: I0e74766457e7612a54d88724144921f64c640699
Reviewed-on: https://chromium-review.googlesource.com/892044
Commit-Queue: <pdfium-deps-roller@chromium.org>
Reviewed-by: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532696}

content_shell previously was failing to find V8 natives file and resources
when packaged on Fuchsia. Update paths used to locate these assets to make
content_shell usable again.

Bug: 796779, 778467
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: I257124223f953bbf048ca6ab649322e8f726bf5d
Reviewed-on: https://chromium-review.googlesource.com/865466
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532695}

This CL was uploaded by git cl split.

R=michaelpg@chromium.org

Bug: 177475
Change-Id: Ia5cea2af0f72d24b611ae85a09051e5a7e41cb70
Reviewed-on: https://chromium-review.googlesource.com/891500Reviewed-by: Michael Giuffrida <michaelpg@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532694}

In some corner case, the application language isn't blocked due to its
legacy state, so we set app language blocked always.

Bug: 783049
Change-Id: I13924b12bf2353217db902bc48796ea74cfafd7f
Reviewed-on: https://chromium-review.googlesource.com/890147Reviewed-by: Theresa <twellington@chromium.org>
Commit-Queue: Leo Zhang <googleo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532693}

'ninja' isn't always in PATH on luci builders, so use the copy checked
into third_party/depot_tools

Bug: 731558
Change-Id: I3f57375dc768d5266e7b80a0324abb3acf48fd49
Reviewed-on: https://chromium-review.googlesource.com/889994
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532692}

TBR=xjz@chromium.org

Bug: 754872, 806715
Change-Id: Iea4cdc3fd17315ad688b48312a0185c114cc8908
Reviewed-on: https://chromium-review.googlesource.com/892053Reviewed-by: Yuri Wiitala <miu@chromium.org>
Commit-Queue: Yuri Wiitala <miu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532691}

Chrome doesn't compute descendant name on focus in this case.

Bug: 795275
Test: manually on twitter.com using j/k. Verify all text gets read.
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I3bf48846aac498655cae8488331df09b65ff9b53
Reviewed-on: https://chromium-review.googlesource.com/881884
Commit-Queue: David Tseng <dtseng@chromium.org>
Reviewed-by: Aaron Leventhal <aleventhal@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532690}

R=pfeldman@chromium.org

Bug: none
Change-Id: I101d69a1b9efdb399a1ced589e3e7f74602ff576
Reviewed-on: https://chromium-review.googlesource.com/889919Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532689}

This CL implements URLLoader::OnCertificateRequested for network
service. It adds a mojo interface OnCertificateRequested for
NetworkServiceClient, and also adds a SSLPrivateKey mojo interface
to be implemented by browser side and used on the network side.

The design doc is at:
https://docs.google.com/document/d/1H6tdoFgx1JP0fjnVUWxmhwidKz1_DUemTxzteztOhaQ/edit

Bug: 783996
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I29ac6ab766d18d3ffbbaf2ca9512039b778d03b0
Reviewed-on: https://chromium-review.googlesource.com/848303Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Matt Mueller <mattm@chromium.org>
Commit-Queue: Jun Cai <juncai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532688}

Bug: 807038
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I499028acf6ccfc8fbeb07fc32bc703593c030325
Reviewed-on: https://chromium-review.googlesource.com/892046Reviewed-by: Yash Malik <ymalik@chromium.org>
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532687}

* Allow --mus on all channels (canary, dev, beta, stable).
* Allow --mash on everything but stable.
* Ensure only one copy of the --mus flag if the user specifies
  both --mus and --mash. The build will run in --mash mode.

Bug: none
Test: about:flags has both options separately
Change-Id: Ic111189e288a8784718f4df0a99bae865734fe6f
Reviewed-on: https://chromium-review.googlesource.com/891624Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532686}

If a [Root]CompositorFrameSinkImpl in FrameSinkManagerImpls
|compositor_frame_sinks_| wasn't deleted before FrameSinkManagerImpl
is destroyed then we can end up running the CompositorFrameSinkSupport
destructor twice. It looks like flat_map::clear() runs the
CompositorFrameSinkSupport destructor before removing the items from the
map, so when CompositorFrameSinkSupport calls back into
FrameSinkManagerImpl::UnregisterCompositorFrameSinkSupport() the entry
is still in the map and gets removed, running the
CompositorFrameSinkSupport destructor a second time.

Separate the map that contains SinkAndSupport into two maps. This will
avoid future errors using iterators after they've been invalidated.
|sink_map_| can be removed with OOP-D when all CompositorFrameSinks are
owned inside FrameSinkManagerImpl.

This is a different approach to https://crrev.com/c/874813 which was
reverted.

Bug: 803405
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I92c230e57dc471f4b16125b0edb3a0618c01930f
Reviewed-on: https://chromium-review.googlesource.com/891424Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Commit-Queue: kylechar <kylechar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532685}

r531270 refactored HeaderChecker to use an atomic counter to track the number of
header-checking tasks.  The main thread waits on the task count becoming zero,
and the task counter is decremented whenever a task completes.  However, the
main thread didn't increment the task counter when it created a task, so the
counter would count into negative values and never reach zero, hanging gn
indefinitely.  The fix is to increment the counter when posting a task.

R=dpranke
CC=fdoray

Change-Id: Id55f943cf0e2af3e9e5af5f3dddf8a7b7671eaac
Reviewed-on: https://chromium-review.googlesource.com/889930Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532684}

Simplifies logging by always appending the current location
(i.e. function name) when LOG() or LOG_ERRNO() are called.

This makes client code smaller and easier to write / read at
the same time.

BUG=NONE
R=pasko@chromium.org,agrieve@chromium.org,rmcilroy@chromium.org

Change-Id: I820c2584eee4cd476bda7e6fb99bbd27782858d7
Reviewed-on: https://chromium-review.googlesource.com/891233Reviewed-by: agrieve <agrieve@chromium.org>
Reviewed-by: Ross McIlroy <rmcilroy@chromium.org>
Commit-Queue: David Turner <digit@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532683}

Remove dezippering from delayTime.  The value will now change
immediately instead of gradually changing from the old to 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: 550533
Test: Delay/dezipper.html
Change-Id: Id3ce76401d4342205c5d67079f6684514ae7e2a2
Reviewed-on: https://chromium-review.googlesource.com/609443
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532682}

This CL was uploaded by git cl split.

R=patricialor@chromium.org

Bug: 177475
Change-Id: Idae699da611562f3d75fdd07c915265c204b74c7
Reviewed-on: https://chromium-review.googlesource.com/891884Reviewed-by: Patti <patricialor@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532681}

https://skia.googlesource.com/skia.git/+log/caaf211e8dd0..b0c427b1f286

$ git log caaf211e8..b0c427b1f --date=short --no-merges --format='%ad %ae %s'
2018-01-29 halcanary SkQP: error image now opaque
2018-01-29 fmalita [skottie] Layer clip support
2018-01-29 angle-skia-autoroll Roll skia/third_party/externals/angle2/ e4741fd0f..37b697edf (1 commit)
2018-01-29 bungeman Don't add lines that don't fit on the page.
2018-01-29 halcanary SkQP: ResourceFactory
2018-01-29 reed check that rect is finite before jamming our bounds
2018-01-29 fmalita [skottie] Time start, time stretch
2018-01-29 reed handle conics in 1D patheffect

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: I67956b1063bc8f706e37d84eba1950c98ba7c37d
Reviewed-on: https://chromium-review.googlesource.com/891625
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@{#532680}

This CL was uploaded by git cl split.

R=pkotwicz@chromium.org

Bug: 177475
Change-Id: I0b25a266bda563f3ad25524cb337f8b6b6a30564
Reviewed-on: https://chromium-review.googlesource.com/891501Reviewed-by: Peter Kotwicz <pkotwicz@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532679}

See BUG for more details. With this CL, for a given key system, the
first registered CDM will always be used for handling the capability
query, and for creating the CDM instance. Hence, there's no way for any
inconsistency to happen (as described in the BUG).

This means that a later registered CDM will not be used until browser
restart (e.g. component updated CDM), which should be fine in most
cases.

BUG=599588

Change-Id: I9390f4ea73e5f40395ab8f87fc1d9c1c1f5c75ad
Reviewed-on: https://chromium-review.googlesource.com/889982Reviewed-by: David Dorwin <ddorwin@chromium.org>
Reviewed-by: John Rummell <jrummell@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532678}

https://boringssl.googlesource.com/boringssl/+log/94cd196a80252c98e329e979870f2a462cc4f402..a62dbf88d8a3c04446db833a1eb80a620cb1514d

Bug: crbug.com/boringssl/183

Change-Id: I37c7e11dab9c445fa9588fee4291dba597ad3223
Reviewed-on: https://chromium-review.googlesource.com/891698
Commit-Queue: Steven Valdez <svaldez@chromium.org>
Reviewed-by: David Benjamin <davidben@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532677}

Bug: 799101
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: If760fdaf17dfe79b54510c0eaa2e670dfb326a2c
Reviewed-on: https://chromium-review.googlesource.com/882665
Commit-Queue: Kurt Horimoto <kkhorimoto@chromium.org>
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532676}

This CL corrects the wrong UV ordering for the call to x010ToAR30() in
GpuMemoryBufferVideoFramePool, and also corrects the IOSurface format
used for creation, from 'R10k' to 'l10r' [1], the latter having the
correct channel endianness (i.e. XR30 == BGRX2101010 [2] .

tested via e.g.
./out/gn/Chromium.app/Contents/MacOS/Chromium 'http://crosvideo.appspot.com?codec=vp9.2&loop=true'
./out/gn/Chromium.app/Contents/MacOS/Chromium 'http://crosvideo.appspot.com?codec=vp9.2&loop=true' --disable-mac-overlays

both showing correct colors :-)

[1] https://github.com/phracker/MacOSX-SDKs/blob/HEAD/MacOSX10.13.sdk/System/Library/Frameworks/CoreVideo.framework/Versions/A/Headers/CVPixelBuffer.h#L83
[2] https://github.com/torvalds/linux/blob/master/include/uapi/drm/drm_fourcc.h#L98

Bug: 797347
Change-Id: I56c06fc06bef950ba2053750cdd7aee937e3c00b
Reviewed-on: https://chromium-review.googlesource.com/891521Reviewed-by: ccameron <ccameron@chromium.org>
Reviewed-by: Frank Barchard <fbarchard@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532675}

Bug: 784451
Change-Id: I62f97469bc9cf4c1aad0cc9ed5695b8a20012e18
Reviewed-on: https://chromium-review.googlesource.com/888262Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532674}

TBR=falken@chromium.org

Bug: 807014
Change-Id: I14f0ed5edfe3ace787c74b9f518c46c154477a45
Reviewed-on: https://chromium-review.googlesource.com/892038Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532673}

Do not place magic constant before the sampled allocations.
This have the following benefits:
  - Much simpler logic of allocator hooks.
  - Will later allow easily hook Oilpan allocator and PartitionAlloc
    as they do not allow shimming, but just provide observers interface.

The only negative effect is lower performance, but it will be addressed
in an upcoming patch.

BUG=803276

Change-Id: Ia3de003f0b7286a1be4e62a33be28b1770f73018
Reviewed-on: https://chromium-review.googlesource.com/890819
Commit-Queue: Alexei Filippov <alph@chromium.org>
Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532672}

This CL was uploaded by git cl split.

R=mfoltz@chromium.org

Bug: 177475
Change-Id: Ie99aa136b2a0d27de3e24ae1e9b737edd0c6e875
Reviewed-on: https://chromium-review.googlesource.com/891921Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532671}