GitLab

This is helpful for people not familiar with metrics internals to
quickly verify the result of a change to *.xml, especially when there's
complicated prefixing involved.

BUG=776834

Change-Id: Ie7066699d34a56564ec80f533a6d416dc03cc29d
Reviewed-on: https://chromium-review.googlesource.com/850545
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527132}

Bug: 785792
Change-Id: I82d7984d1be96e82425e5c1a43ee7686906212e5
Reviewed-on: https://chromium-review.googlesource.com/849694Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527131}

This is a continuation for:
https://chromium-review.googlesource.com/c/chromium/src/+/848085

There was a missing include which only became apparent when
attempting to build downstream. I have done more extensive local
testing and believe this should resolve the matter.

Bug: 795007
Change-Id: I368f14d00e784082f887f889f884b992daf06b86
Reviewed-on: https://chromium-review.googlesource.com/851153
Commit-Queue: Tommy Martino <tmartino@chromium.org>
Commit-Queue: Mathieu Perreault <mathp@chromium.org>
Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527130}

This CL adds a service function BufferFormatToString() to translate
from gfx::BufferFormat to a string, and uses it in a few places ISO
static_cast<>ing to int.

Bug: 798485
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: I01093983b62fb9c8c495b521c890c4b45e4708a6
Reviewed-on: https://chromium-review.googlesource.com/847772Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527129}

Add delays between the consecutive warmup probes. The delay is
exponentially increasing.

Bug: 760294
Change-Id: I2d0ea28d07f532d81d84a2463b72a9243d90af3d
Reviewed-on: https://chromium-review.googlesource.com/850838Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527128}

If the last event in the timeline is SetTarget, and the current value
is close enough to the final value, and if enough time has elapsed
since the beginning of the event, we can remove the SetTarget event
from the timeline.

Doing so allows all nodes to use the fast path because the AudioParam
is a constant over the rendering quantum.

Bug: 794712
Test: 
Change-Id: I7274f9ba10b4a31728c6329c0fca3394cb557e7f
Reviewed-on: https://chromium-review.googlesource.com/826124
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527127}

present, CSSStyleValue.parse/parseAll thrown type or syntax error.
but CSSStyleValue.parse/parseAll was decided to throw only
type error.

https://github.com/w3c/css-houdini-drafts/issues/305

Bug: 798937
Change-Id: I00d6855d1b3667441d009d8d3551daed970cab1a
Reviewed-on: https://chromium-review.googlesource.com/849972Reviewed-by: Darren Shen <shend@chromium.org>
Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com>
Cr-Commit-Position: refs/heads/master@{#527126}

This change helps avoid renderer hang crashes on Windows by doing the
potentially costly destruction of a ChunkDemuxer object on a background
task.  The duration of that destruction, not including any initial
scheduling delay, is recorded in a new histogram:
"Media.MSE.DemuxerDestructionTime".

BUG=774268,774288,796704

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: I77dcb3ac1f7d5c5777f5302a7fc87199bf693d74
Reviewed-on: https://chromium-review.googlesource.com/838701
Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
Reviewed-by: Robert Liao <robliao@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527125}

Also enabling an ESLint presubmit check to enforce it from now on.

Bug: 792774
Change-Id: If700ff1c88e31001019fc693e201c8c4ff8b9675
Reviewed-on: https://chromium-review.googlesource.com/841536
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527124}

These metrics haven't been queried for some time and are stable.

BUG=775335,759926,722654,691858,691857

Change-Id: I35324714cba22e62b3ebcfc183527cf9807a201c
Reviewed-on: https://chromium-review.googlesource.com/848427Reviewed-by: Gayane Petrosyan <gayane@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527123}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/e5cf9fb5..91af3882

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: I111c0105c8440fb048f17bee6cc15a0890dbc014
Reviewed-on: https://chromium-review.googlesource.com/851172Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527122}

Upon screen resize, only the WebContents on the front got the timely
update while those in the back were updated later when they got to
the front. This has a caveat when working with Android tabswitcher
where the one in the background can be shown without it going fore-
ground as reported in the bug. This CL fixes it by enumerating all
the WebContents and update their size when the resize event occurs.

This had been previously handled by each |ContentView.onSizeChanged|
per WebContents, but r522517 changed the way Chrome handles the resize
event. So this should also be taken care of by CompositorViewHolder
for each WebContents as well.

Bug: 798277
Change-Id: I4b0d25850ded812668e5bdbba52c4898492bd9cd
Reviewed-on: https://chromium-review.googlesource.com/848758Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527121}

Previously, we would only support IdentityHashTranslator and the cases
where keys or values were simple Members. This patch adds general
support for translators as the barrier is moved out of the translator.
This enables using custom translators as well as having eager-traced
containers as keys or values in HeapHashMap.

Bug: chromium:757440
Change-Id: I8a1b2e840900099b587a5d6bc8106d4287f4227f
Reviewed-on: https://chromium-review.googlesource.com/850613
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527120}

https://webrtc.googlesource.com/src.git/+log/bb4bbfda1cf5..75ceef2ab41b

$ git log bb4bbfda1..75ceef2ab --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtc


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: I21f7b0d25e327a285cd1d0f73decb4cee59b511d
Reviewed-on: https://chromium-review.googlesource.com/851118Reviewed-by: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527119}

https://chromium.googlesource.com/catapult.git/+log/a1b4d5ea3410..634b18ca3700

$ git log a1b4d5ea3..634b18ca3 --date=short --no-merges --format='%ad %ae %s'
2018-01-03 eakuefner [TBMv2] Move Python GenericSet into diagnostics.generic_set.GenericSet
2018-01-04 eakuefner [TBMv2] Smoke test diagnostic equality in Python

Created with:
  roll-dep src/third_party/catapult


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

Change-Id: Ie56eac92ecf5aa81915a8d9f8b993e6157921cec
Reviewed-on: https://chromium-review.googlesource.com/850548
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527118}

Regressed in d875cacb

Bug: 780593
Change-Id: Ifee7a93ada73f5b757a35ece75b0bef61ea767c2
Reviewed-on: https://chromium-review.googlesource.com/820531Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Ted Choc (back but slow, ping me) <tedchoc@chromium.org>
Commit-Queue: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527117}

This flag skips to the user login screen.

BUG=753870
TEST=manual

Change-Id: I3682b825d8177b8676718f421335f4af1d940937
Reviewed-on: https://chromium-review.googlesource.com/850582
Commit-Queue: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Alexander Alekseev <alemate@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527116}

Change-Id: I7f15880761ab173ecebac694d042b0048707acae
Reviewed-on: https://chromium-review.googlesource.com/850836Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Darin Fisher <darin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527115}

Bug: 787139
Change-Id: I141466b5e8d7b68a91f3a844e07053b6c3720153
Reviewed-on: https://chromium-review.googlesource.com/851193Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Sidney San Martín <sdy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527114}

Safari plays a sound when a download fails. This does it for both
success and failure.

Bug: 787139
Change-Id: If3df9aaed218f8b66fcf124ae8f2529eba87c6f5
Reviewed-on: https://chromium-review.googlesource.com/850743Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Sidney San Martín <sdy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527113}

This method is called when a nested loop is done running work.
It will help simplify the TaskQueueManager code.

Use case:
The blink scheduler needs to adjust the time domain
when there is a transition between a nested/non-nested scope.
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc?l=2302&rcl=614a67e2c14cf8bb9a06f4fe8da7625cbf1ea7d7
Currently, it checks base::RunLoop::IsNestedOnCurrentThread()
every time a task completes to detect transitions between
a nested/non-nested scope. Code would be simpler with
an explicit notification.

Bug: 783309
Change-Id: I9748c287ad8418831598d84f45a518e4138c5e1b
Reviewed-on: https://chromium-review.googlesource.com/766388Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Commit-Queue: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527112}

For some reason duplicate onDismiss calls are sneaking through.  This
removes the crasher while a follow up patch will land to report more
logging information only for dev and canary.

BUG=791879

Change-Id: I62088ae7c966b6c75e667c7eff99fac7e6826bb5
Reviewed-on: https://chromium-review.googlesource.com/850575Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: David Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527111}

R=jbroman@chromium.org

Bug: 754124
Change-Id: Ia7dd043982d404afa316be81fb030c9caed9e7c0
Reviewed-on: https://chromium-review.googlesource.com/849476Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527110}

Bug:785306,743089

Change-Id: I4e13b4d4c2bcd81ca54c7fa7c5133fe046a53043
Reviewed-on: https://chromium-review.googlesource.com/850842
Commit-Queue: Steven Holte <holte@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527109}

Change-Id: If4cb73bc28300b5ed01ce10f7068ff930c8ba6cb
Reviewed-on: https://chromium-review.googlesource.com/847482
Commit-Queue: Greg Kraynov <kraynov@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527108}

BUG=

Change-Id: I9297f99b73e8a91b208f3faa51644bd5574c4e0c
Reviewed-on: https://chromium-review.googlesource.com/846248Reviewed-by: Theresa <twellington@chromium.org>
Commit-Queue: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527107}

SigninScreenHandler::HandleAuthenticateUser should ignore 'User not found'
errors.

Bug: 794487
Change-Id: I83c9a9bc2787514384e5bbee9253eaf383df94db
Reviewed-on: https://chromium-review.googlesource.com/850998Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Alexander Alekseev <alemate@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527106}

The ui::PAGE_TRANSITION_FORWARD_BACK bit has always been set for
different-document back/forward navigation. It's not clear why
same-document back/forward navigation should be treated differently.
Making this change matches the Blink behavior (so should be low risk)
and removes a special case.

Bug: 798832
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I39e7a28caa9069cbb12ebdc965918792b36b77b2
Reviewed-on: https://chromium-review.googlesource.com/847940
Commit-Queue: Danyao Wang <danyao@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527105}

This patch adds initial support for images in OOP raster, using the transfer cache.
This has a few sub-components:
- Update the ServiceImageTransferCacheEntry to handle > max texture size images.
- Update the GPUImageDecodeCache to have a TRANSFER_CACHE upload mode.
- Update GPUImageDecodeCache's unit tests to hit the TRANSFER_CACHE path.
- Update RasterCHROMIUM to take an ImageProvider, and re-factor RasterCHROMIUM to reduce number
  of arguments.

Bug: 777628
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: I037391fa150529883f1fc534064e29802cf2d08f
Reviewed-on: https://chromium-review.googlesource.com/822605
Commit-Queue: Eric Karl <ericrk@chromium.org>
Reviewed-by: vmpstr <vmpstr@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527104}

(Reland with WebviewProxyAuthLoginTest browsertext fix)
This CL fixes sign-in behind an authenticated proxy. Two actual changes
were necessary in this context:
- SigninPartitionManager: copy auth data from the system request
  context into new StoragePartitions
  This makes SigninPartitionManager::StartSigninSession async, which
  made small changes necessary in GaiaScreenHandler and
  EnrollmentScreenHandler
- SigninScreenHandler: Don't reload gaia immediately when auth details
  have been supplied. This was unnecessary (the URL Request will
  continue) and actually harmful, as now that we're using a new
  StoragePartition for each sign-in attempt, this discards auth data
  before we can copy them.

To support the browsertest, testserver.py gets an argument to redirect
CONNECT requests to localhost when proxying (in sign-in browsertests
without proxy, we use RuleBasedHostResolverProc::AddRule to achieve
this effect).

      unit_tests --gtest_filter=SigninPartitionManagerTest*
  Manual test:
   Setup:
     Setup an proxy server with Basic authentication
     Configure the device to use the proxy server.
     Make sure the user adding screen is shown.
   Test 1:
     Expect proxy auth dialog. Enter correct proxy auth data.
     Expect that sign-in screen is shown and sign-in works.
   Test 2:
     Expect proxy auth dialog. Enter incorrect proxy auth data.
     Expect that proxy-auth dialog is shown again.
   Test 3:
     Expect proxy auth dialog. Press Cancel.
     Expect that a network error dialog is shown.
     Press "try to sign in again".
     Expect proxy auth dialog. Enter correct proxy auth data.
     Expect that sign-in screen is shown and sign-in works.

Bug: 793524
Test: browser_tests --gtest_filter=WebviewProxyAuthLoginTest.* &&
Change-Id: I56dafa240ad3bb5902517688a5cb17e309f2982d
Reviewed-on: https://chromium-review.googlesource.com/850472Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Pavol Marko <pmarko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527103}

- The omnibox icon size was changed
- The strings used in the dialog were changed

TBR=estade@chromium.org

Bug: 797112
Change-Id: I05d70320f41940899663d871a6dc1948ee6fd0d2
Reviewed-on: https://chromium-review.googlesource.com/841245
Commit-Queue: Patrick Monette <pmonette@chromium.org>
Reviewed-by: Patrick Monette <pmonette@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527102}

CR fixes after crrev.com/c/846422

Bug: 796644
Change-Id: I3395664131d44ecf2108582b28d1cac52ae26b91
Reviewed-on: https://chromium-review.googlesource.com/849708Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527101}

file_data_pipe_producer.g should have been file_data_pipe_producer.h

TBR=rockot@chromium.org

Bug: None
Change-Id: Ia25c80f2c62198fe6114d1a170542f710242e016
Reviewed-on: https://chromium-review.googlesource.com/836453
Commit-Queue: Chris Mumford <cmumford@chromium.org>
Reviewed-by: Chris Mumford <cmumford@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527100}

The main improvement is that source files with unhandled extensions
other than .h, for example .json and .idl, are not counted towards the
merge limit, and thus can't lead to too many chunks being generated.

By restructuring, the code also iterates the source file list once
instead of twice, and calls get_path_info() once per source file instead
of twice.

Change-Id: I130c815c84fff08b6ad2d6f36434140efbeb2d29
Reviewed-on: https://chromium-review.googlesource.com/850397Reviewed-by: Daniel Bratell <bratell@opera.com>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#527099}

This reverts commit d7d410b6.

Reason for revert: Test is crashing on linux-chromeos-dbg builder.
See crbug.com/799229 for details.

Original change's description:
> Add test coverage for ChromeVox api
> 
> Bug: 
> Change-Id: I8b4c002427f14426753e6a4ddc083e50a33cb985
> Reviewed-on: https://chromium-review.googlesource.com/847896
> Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
> Reviewed-by: Aaron Leventhal <aleventhal@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#527035}

TBR=dtseng@chromium.org,aleventhal@chromium.org,katie@chromium.org

Change-Id: I73164ee8c477eff6878e59993f52168c552ed523
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/851075Reviewed-by: Greg Levin <glevin@chromium.org>
Commit-Queue: Greg Levin <glevin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527098}

R=robertma@chromium.org

Bug: 
Change-Id: If362b229cde4d29397528ee361d8c11638beae1c
Reviewed-on: https://chromium-review.googlesource.com/851154Reviewed-by: Robert Ma <robertma@chromium.org>
Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527097}

Clean up and simplify audionodeoptions.js.  We also now test all the
possible channel count modes when the mode is fixed so that we verify
we didn't miss a case.

Bug: 
Test: constructor/*.html
Change-Id: Ife8770e16097d243f2ce46dc98386109cdbe18e8
Reviewed-on: https://chromium-review.googlesource.com/832821Reviewed-by: Hongchan Choi <hongchan@chromium.org>
Commit-Queue: Raymond Toy <rtoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527096}

This change:
1. Adds NavigationSimulator::SetMethod to set the initial HTTP method.

2. Converts navigation intercept tests to use the nav simulator. Note
   well that this conversion *relaxes* the test constraints. We now
   simply check that the request fails to commit, rather than fails
   to start. This is to allow future optimizations to apply policy
   in an async way without blocking the majority of network requests.
   For POSTs, we still check that the nav is cancelled at start.

3. Converts flash download intercept test to use the nav simulator.

Bug: 793053

Change-Id: Ic30bf123fa94abe978ca4f5b02802899bc9a423e
Reviewed-on: https://chromium-review.googlesource.com/813417
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Reviewed-by: Tao Bai <michaelbai@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527095}

This is a reland of 0346a3a1
Original change's description:
> Adds predictable multimatch support in PreviewsOptimizationGuide
> 
> Supports multiple hint matches with precendence given to the match
> that occurs first in the configuration. Also, allows matches
> without whitelisted optimization to be able to turn off whitelisting
> for a host pattern (that otherwise would be whitelisted for another
> matching pattern that occurs later in config).
> 
> Bug: 783237
> Change-Id: I9239b74b170a1d5a19b13caccecc732f9715de90
> Reviewed-on: https://chromium-review.googlesource.com/847816
> Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
> Commit-Queue: Doug Arnett <dougarnett@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#526763}

Bug: 783237
Change-Id: I75cf52fbe3b471f81b3f18ae451d7daa0844190e
Reviewed-on: https://chromium-review.googlesource.com/849673
Commit-Queue: Doug Arnett <dougarnett@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527094}

Due to a minor oversight, the FrameSinkVideoCapturer will occasionally
drop refresh frame requests. This is because the VideoCaptureOracle can
reject the request for a frame capture, based on its own sampling
heuristics. This change adds a simple mechanism where a "retry timer" is
started if this should occur. The "retry" is canceled if other frame
captures are triggered in the meantime. Essentially, this guarantees
that the consumer will eventually receive a new video frame after making
a refresh request for one.

Bug: 785072
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: Ic3dd0d5ca1f8cb48193123e714ed8cf89076f953
Reviewed-on: https://chromium-review.googlesource.com/838320Reviewed-by: Xiangjun Zhang <xjz@chromium.org>
Commit-Queue: Yuri Wiitala <miu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527093}