GitLab

https://chromium-review.googlesource.com/c/chromium/src/+/902769

Adds missing <algorithm> include for std::min.

BUG=none

Change-Id: I1e4dd1ef72f9c28f5aa6836b8ac1a7fb1f532f47
Reviewed-on: https://chromium-review.googlesource.com/905204Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534817}

TBR=anthonyvd

Bug: 809609
Change-Id: I5ed542e233feae2bc6c5faa34805837c7f0ee555
Reviewed-on: https://chromium-review.googlesource.com/905096Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Commit-Queue: Andrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534816}

https://chromium.googlesource.com/angle/angle.git/+log/44dcb911c00e..bfeed4ddfcd6

$ git log 44dcb911c..bfeed4ddf --date=short --no-merges --format='%ad %ae %s'
2018-02-06 oetuaho Add a compiler perf test shader to stress traversers
2018-02-02 jie.a.chen Vulkan: Fix gcc compile error

Created with:
  roll-dep src/third_party/angle


The AutoRoll server is located here: https://angle-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.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
TBR=cwallez@chromium.org

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

Enable the print button in the new print preview UI.

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

This CL includes a group of minor fixes and improvements that will ease
the testing of Prefetching with limitless mode enabled:
* Fixes comments in PrefetchBackgroundTaskScheduler.
* Improves handling of invalid URLs and text output from
  OfflineInternalsUIMessageHandler.
* Improve debug logging in MetricsFinalizationTask.
* Use larger bundle size limit for GeneratePageBundleRequest in
  PrefetchNetworkRequestFactoryImpl when limitless mode is enabled.
* Complete TODO to make method private in SuggestedArticlesObserver.

Bug: 793109
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ic4c1931e6c8ab8f5bf2fdd8cfec8075f30a9b6cd
Reviewed-on: https://chromium-review.googlesource.com/869373
Commit-Queue: Carlos Knippschild <carlosk@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Reviewed-by: Dmitry Titov <dimich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534813}

Change:
Replace kOlderFolders with kFullscreenAppListFolders.

BUG=809324

Change-Id: Ib2f779fcd5e71a8e7a826c53b21ddaaf74126a22
Reviewed-on: https://chromium-review.googlesource.com/903412Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534812}

Bug: chromium:757625
Change-Id: I0f68e43c7f9700d3d02d640d36c2d9a4523b4495
Reviewed-on: https://chromium-review.googlesource.com/894167Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>
Commit-Queue: Allen Vicencio <allenvic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534811}

content/browser/loader/loader_io_thread_notifier.cc has the following
include chain:
content/browser/frame_host/render_frame_host_impl.h
-> content/public/browser/content_browser_client.h
-> device/usb/public/mojom/chooser_service.mojom.h (generated header)

And so //content/browser needs a //device/usb/public/mojom dep.  This
was previously only added in //content/public/browser and could therefore
cause build flakiness.

Bug: 806965
Change-Id: I6d5254cd427fc6a6b1cfaefb26a2eb4056f62638
Reviewed-on: https://chromium-review.googlesource.com/903165Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Cr-Commit-Position: refs/heads/master@{#534810}

It does timeout sometimes on Win.

Bug: 667560
Change-Id: I2047340ce939414ce19b1ddc61d4423d35e12e2d
Reviewed-on: https://chromium-review.googlesource.com/904771Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534809}

 * "Modernizes" ntp - change bg color, tweak values for fake search box
   to real omnibox transition, change fake search box bg, update tablet
   style
 * Increases modern toolbar background height to 40dp
 * Fixes an omnibox suggestions alignment issue in RTL to better align
   with omibox actions container
 * Updates toolbar shadow for NTP scroll
 * Moves toolbar primary color initialization to ToolbarModelImpl
   constructor

BUG=803087,803088

Change-Id: Ic656b5de656296c3fe592e5a0a3e106472fa0419
Reviewed-on: https://chromium-review.googlesource.com/902966Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534808}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/a24f6910..a462f872

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org

Change-Id: I45ba3fe78139547c4627dd56cb136c60c3368c6b
Reviewed-on: https://chromium-review.googlesource.com/905102Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534807}

The feature has lived on for too long in a broken state, and just makes
the code more complicated than necessary. Remove the rendering code
that deals with generating draft pages. There still exists browser side
code to be removed later.

This CL makes a PrintPreviewFailureType enum value obsolete. While
marking it as deprecated, also fix another issue with
PrintPreviewFailureType where the wrong enum got marked as deprecated.

BUG=746768

Change-Id: I5a61770d034ff33add6f4070984e46fa0a02b221
Reviewed-on: https://chromium-review.googlesource.com/899855
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Wei Li <weili@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534806}

Update DriveMetricsProvider::HasSeekPenalty() to not compare
base::SysInfo::GetStrippedReleaseBoard() against "parrot" to
determine if the device has a rotating disk.

parrot stopped receiving updates in October 2017 (see
https://support.google.com/chrome/a/answer/6220366), and
code that wants to know if a rotating disk is present should
either read the "rotational" sysfs node or use the "has_hdd"
USE flag. Hardcoding device names in Chrome is fragile: if
someone brings up a new device with a rotating disk, they're
not going to know about miscellaneous Chrome code that needs
to be updated (and they're likely not even a Chrome
developer).

Bug: none
Change-Id: I2a8d3dce3a51546ba96e3f4cb7c5c2d4d9d8b2a9
Reviewed-on: https://chromium-review.googlesource.com/905464
Commit-Queue: Dan Erat <derat@chromium.org>
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534805}

As pointed out by jbroman@ during a review, null offsets will violate
the requirements of std::sort. Luckily CompareOffsets is only ever used
where offsets will be non-null, so enforce that in the code. Also since
it is used exactly once, just inline it.

Bug: 791086
Change-Id: I9df7f1201fdc3024a727c62bfd7e917791de12b6
Reviewed-on: https://chromium-review.googlesource.com/905100
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534804}

According to the CSSOM spec [1], all strings should be serialized with
double-quotes. The axis name in font-variation-settings was previously
serialized with single-quotes; change this to double-quotes to match
the spec and non-WebKit browsers.

[1]: https://drafts.csswg.org/cssom/#common-serializing-idioms

Bug: 806412
Change-Id: Ib0da551c2799a9dc4e273ac150972a7bd5c7a34f
Reviewed-on: https://chromium-review.googlesource.com/900843Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Chris Nardi <cnardi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534803}

* Adds FirstContentfulPaint to timeline markers
* Adjusts colors of paint markers to different shades of green

BUG=749814

Change-Id: I4ccbcae4c6906d9fbacc8a81eb55b6059f55f472
Reviewed-on: https://chromium-review.googlesource.com/619500Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Reviewed-by: Alexei Filippov <alph@chromium.org>
Commit-Queue: Patrick Hulce <phulce@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534802}

This increases test coverage in select-to-speak by creating a class to
be responsible for JS extension end-to-end tests for search+mouse.

Bug: 800524
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I81cbe9eee1965e5bb5c58dea8deee8eecffeda45
Reviewed-on: https://chromium-review.googlesource.com/898251
Commit-Queue: Katie Dektar <katie@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534801}

QueryTracker was tightly coupled to GLES2Implementation. This CL adds
a new QueryTrackerClient interface that GLES2Implementation implements.
This will allow reuse by a RasterImplementation.

Bug: 789238
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: I752663df76a29d026391190bdfb62e70e732d2e9
Reviewed-on: https://chromium-review.googlesource.com/902201
Commit-Queue: Jonathan Backer <backer@chromium.org>
Reviewed-by: Victor Miura <vmiura@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534800}

Changes:
Since new style folder is paged horizontally, Up and down key should
move focus only within a single page. Thus, hitting up key when focus is
on the first row in current page moves the focus onto the search box and
hitting down key when focus is on the last row in current page moves the
focus onto the folder title.

Specs:
See "Focus change" section in go/applist-focus

BUG=809316

Change-Id: Ic21e76ad176e117a4789fdba5b6428796ed99064
Reviewed-on: https://chromium-review.googlesource.com/903374
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534799}

Bug: skia:6718
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I6e5106a278c278b8e665d3dcaab0fff248507aeb
Reviewed-on: https://chromium-review.googlesource.com/901744
Commit-Queue: Brian Salomon <bsalomon@chromium.org>
Reviewed-by: Justin Novosad <junov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534798}

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

It is used by content/browser/tracing/arc_tracing_agent_impl.{cc,h}, which,
logically, should not depend on chrome compositor.

Also, we want to move arc_tracing_agent_impl to //chrome/browser
(crrev/c/868575) but we do not want to add a dependency from //chrome/browser to
//cc/base ( https://cs.chromium.org/chromium/src/chrome/browser/DEPS?l=89).

TBR=oysteine@chromium.org

Bug: 808839
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I30adee8ee4f3c7e8c4f478f15e6841937b4be96c
Reviewed-on: https://chromium-review.googlesource.com/899858Reviewed-by: danakj <danakj@chromium.org>
Reviewed-by: Ehsan Chiniforooshan <chiniforooshan@chromium.org>
Commit-Queue: Ehsan Chiniforooshan <chiniforooshan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534796}

When the beforeunload handler is ACKed by the OOPIF process with "do not
proceed", the corresponding speculative frame is cleaned up. When the
speculative frame is in the parent frame's process it will set the
EmbeddedContentView of the owner element to nullptr. This is incorrect
as the navigation has been canceled.

This CL will avoid setting the EmbeddedContentView to nullptr when the
ContentFrame() is not equal to Document::GetFrame() (i.e., speculative)
and also the EmbeddedContentView is a RemoteFrameView.

Bug: 578349, 673170, 807772
Change-Id: Ic6f5a16ae6f02bfbe11238851936ee323472a407
Reviewed-on: https://chromium-review.googlesource.com/898028
Commit-Queue: Ehsan Karamad <ekaramad@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Ehsan Karamad <ekaramad@chromium.org>
Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534795}

Previously SimCompositor collected drawing commands through an
implementation of WebDisplayItemList interface (SimDisplayItemList).
However, SPv175 and SPv2 don't use WebDisplayItemList's Append*
methods, but use cc::DisplayItemList directly.

Now collect drawing commands using PaintRecordBuilder.

Bug: 805049
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I29034b55a01e8e73a1b0f3933ec48b52d3e0b7a1
Reviewed-on: https://chromium-review.googlesource.com/900559
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: vmpstr <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534794}

A recent CL got rid of the shared constants and instead copied
them to relevant files. To work in jumbo builds they must then
have unique names.

(The alternative to find a shared location for the constants
seems not applicable in this case)

Bug: 783018,723882
Change-Id: I8e64dcf56bfe74d30279549291e89bdec80bc3d2
Reviewed-on: https://chromium-review.googlesource.com/905304
Commit-Queue: Daniel Bratell <bratell@opera.com>
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Reviewed-by: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534793}

The proto-spec defines requested locks as an ordered set, ordered
by request order, and granted locks as an ordered set, ordered by
grant order, which can differ.

The order is visible using the query() API. Add tests to ensure that
an implementation doesn't re-use the request order for the granted
order, or use unordered sets, etc.

Bug: 807016
Change-Id: I8b82db794b5d4431e7873dbb78c8f9f81064db40
Reviewed-on: https://chromium-review.googlesource.com/892143
Commit-Queue: Joshua Bell <jsbell@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534792}

I am preparing to remove this flag in Skia.  Beyond these two debug use
cases, the flag and the performance-only feature it enables appear to
only be used antagonistically by fuzzers.

Bug: skia:7566, chromium:808830
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I99c87f6600200e825aa7a1590d4031030d55af42
Reviewed-on: https://chromium-review.googlesource.com/905107Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534791}

NetworkQualityEstimatorTest.RecordAccuracy: https://build.chromium.org/p/chromium.fyi/builders/Fuchsia/builds/13666

AddressSpaceRandomizationTest.Random: https://build.chromium.org/p/chromium.fyi/builders/Fuchsia%20ARM64/builds/5325

Bug: 793176, 809660
Change-Id: Ie4c01c4aa7889cc71ed7cec91b8c2f5d358f865c
Reviewed-on: https://chromium-review.googlesource.com/905203
Commit-Queue: Scott Graham <scottmg@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534790}

Make the SVGResources and SVGResourcesCache interfaces take a
LayoutObject& instead of a LayoutObject* where appropriate, and then
update users (transitively) as reasonable.

Bug: 769774
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Idfa28bb01551332064ca1ab6d6260d29d3264145
Reviewed-on: https://chromium-review.googlesource.com/904529Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#534789}

This simplifies interface, OfflineItemsAggregator impl and is another step away from
loading all the items on startup and rather loading/caching them transparently as
needed.

Next steps:
- remove GetItemById as not used.
- consider unloading items from memory/caches using a new method, like ClearCaches().

Bug: 806617
Change-Id: Ife57fa13e72273a6bac9b4e843a131f7c9c8de15
Reviewed-on: https://chromium-review.googlesource.com/896553Reviewed-by: vitaliii <vitaliii@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: Carlos Knippschild <carlosk@chromium.org>
Reviewed-by: Shakti Sahu <shaktisahu@chromium.org>
Commit-Queue: Dmitry Titov <dimich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534788}

The 3.22 upgrade has settled, so it's fairly safe to remove our snapshot
of 3.21.

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

The interface isn't needed after an unrelated refactoring. Listen for
connection errors on OffscreenCanvasSurfaceClient interface instead,
since the message pipe endpoints are the same, and use that signal to
destroy OffscreenCanvasSurfaceImpl. Also cleanup data members to remove
an unneeded callback.

Bug: none
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ib21e0974b981dcd63eb76a9ac1f9f10591eea08f
Reviewed-on: https://chromium-review.googlesource.com/901922Reviewed-by: Justin Novosad <junov@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Olivia Lai <xlai@chromium.org>
Commit-Queue: kylechar <kylechar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534786}

This is a re-land of commit bc976225,
which had a type-casting bug due to failure to override child
overflow recursion. The new patch also includes a test adjustment
and DCHECK to avoid such security bugs.

Bug: 807900,808876
Change-Id: I9210cf48b25ca255ceccd955fd4699e76620f90f
Reviewed-on: https://chromium-review.googlesource.com/902763Reviewed-by: Stephen Chenney <schenney@chromium.org>
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534785}

getRPTextureSize is a function that is used for getting render pass
texture size, which is not used since the size is stored implicitly
inside the actual texture size.

R=enne@chromium.org

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I5c68546802eeeb607acd9f07a0d6f0fc11b26453
Reviewed-on: https://chromium-review.googlesource.com/899771Reviewed-by: enne <enne@chromium.org>
Commit-Queue: weiliangc <weiliangc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534784}

Recently landed test is flaky on Android. Disabling until I have time
to investigate the cause.

TBR=alexmos@chromium.org

Bug: 809580
Change-Id: I6824b9e75c8114fc95716ccb8caa7af8867cab17
Reviewed-on: https://chromium-review.googlesource.com/905093Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Commit-Queue: Ken Buchanan <kenrb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534783}

Textures in oop raster are currently cleared only when they are new.
However, when textures are reused (both for partial raster and for
recycling) they need to clear what's underneath them.

This also fixes several bugs in RasterSource clearing.  In particular,
clipDeviceRegion (possibly only for gpu?) does not reset the clip,
only adds to it.  It also unions together all the rects in the
region.  This leads to increased clipping, and no clearing of texels
outside of the raster full rect.

This passes more parameters to RasterSource and PaintOpBuffer so that
the following clearing can be done:
(1) if not opaque and not partial raster, clear everything to transparent.
(2) if not opaque and partial raster, clear inside clip to transparent.
(3) if opaque and playback rect intersects with one texel inside the
content size for the layer, then draw 2xheight and widthx2 pixel rectangles
to clear the last pixels.

Also add a number of unit tests to verify that minimal clearing is done
that would have caught the RasterSource changes.

This also removes debug clearing code in RasterSource which is old and
is not as useful as it once was.

Bug: 805746,806014
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;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: I5f68ef63ee8c2831906890e3e2239ac1e3db430e
Reviewed-on: https://chromium-review.googlesource.com/888078
Commit-Queue: enne <enne@chromium.org>
Reviewed-by: Khushal <khushalsagar@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534782}

This reverts commit 55424d2f.

Reason for revert: Windows compile failure.

Original change's description:
> Remove code from obsolete field trial from 2015.
> 
> Originally added here: https://codereview.chromium.org/1117193004
> 
> Bug: 799180
> Change-Id: I8e46beafdc57c249fb9909e39c197519b2652c7b
> Reviewed-on: https://chromium-review.googlesource.com/902769
> Reviewed-by: Mark Mentovai <mark@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#534776}

TBR=asvitkine@chromium.org,mark@chromium.org

Change-Id: If512ffd90d5218190d3bddb4931491f76e161aed
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 799180
Reviewed-on: https://chromium-review.googlesource.com/905462Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534781}

* Remove EnableDebugRecording/DisableDebugRecording from AudioManager, and add a getter for AudioDebugRecordingManager instead, used by AudioDebugRecordingSessionImpl to toggle debug recording directly on AudioDebugRecordingManager.
* Pass a file creation callback instead of file name to AudioDebugRecordingManager::EnableDebugRecording.
* Move CreateFile from AudioDebugRecordingManager to AudioDebugRecordingSessionImpl.
* Add unittest for AudioDebugRecordingSessionImpl.cc:CreateFile.
* Add MockAudioDebugRecordingManager.
* Rename AudioDebugFileWriter::GetFileNameExtension to GetFileExtension, so that it's not confused with extensions of filename, e.g output.<ID>.


Bug: 788657
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: I9d0c4323f0e9630185d5e8af81c9e1b8f7b07a71
Reviewed-on: https://chromium-review.googlesource.com/881501
Commit-Queue: Marina Ciocea <marinaciocea@chromium.org>
Reviewed-by: Olga Sharonova <olka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534780}

TBR=jmadill@chromium.org

https://chromium.googlesource.com/external/khronosgroup/webgl.git/+log/98a3d3b..a182a9a

Cq-Include-Trybots: master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.angle:linux_angle_rel_ng;master.tryserver.chromium.angle:win_angle_rel_ng
Change-Id: Ia2dff519f9b9c8f28bb494a2ea825f0f3e7330f3
Reviewed-on: https://chromium-review.googlesource.com/904858Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Zhenyao Mo <zmo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534779}

This prepares for servicification of the unzipping library, where the
library will not have direct access to the filesystem.

Bug: 792066
Change-Id: I696dd8ef0936f22dc637e078bd8bba565e854ead
Reviewed-on: https://chromium-review.googlesource.com/860996
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: Satoru Takabayashi <satorux@chromium.org>
Reviewed-by: Jay Civelli <jcivelli@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#534778}