GitLab

Continuation of http://crrev/c/802464 to test that Active, Inactive,
and Background application states are interpreted correctly.

Bug: 781361
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ie86cfca571e8e0a4159a591d0d357a3458384483
Reviewed-on: https://chromium-review.googlesource.com/802618Reviewed-by: Gregory Chatzinoff <gchatz@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Peter Lee <pkl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521453}

TBR=dcheng

Bug: 789374
Change-Id: I68d41425e0cf376d1346546268fac60bd9a6dc3c
Reviewed-on: https://chromium-review.googlesource.com/807184Reviewed-by: Chris Palmer <palmer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521452}

Passing the wrong coordinates was causing it to scroll to the wrong
place and sometimes bounce around. Adds a new browser test to
ensure that calling scrollTo works as expected.

Bug: 788696, 646117
Change-Id: Ia8014c9ee96eef14a60a7f4ddebeec02e2f15db4
Reviewed-on: https://chromium-review.googlesource.com/802235
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Doug Turner <dougt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521451}

https://chromium.googlesource.com/angle/angle.git/+log/d5f44c986038..f5be5bafa6f6

$ git log d5f44c986..f5be5bafa --date=short --no-merges --format='%ad %ae %s'
2017-11-29 geofflang StateManagerGL: Use dirty bits for program binding.

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

Change-Id: Ie71348cfd4615b8351935f75ae9ae23e32d6bed9
Reviewed-on: https://chromium-review.googlesource.com/806921Reviewed-by: angle-deps-roller . <angle-deps-roller@chromium.org>
Commit-Queue: angle-deps-roller . <angle-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521450}

Talk about the special powers of WebUI renderers, and not-yet-shipped features.

Bug: 789374
Change-Id: I94e76ee58aec2ac0e5c8ca65cbf810e83f6c17e3
Reviewed-on: https://chromium-review.googlesource.com/798393Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521449}

This reverts commit ae15c135.

Reason for revert:
https://bugs.chromium.org/p/chromium/issues/detail?id=791291
https://bugs.chromium.org/p/chromium/issues/detail?id=791316
https://bugs.chromium.org/p/chromium/issues/detail?id=791288
https://bugs.chromium.org/p/chromium/issues/detail?id=791348
https://bugs.chromium.org/p/chromium/issues/detail?id=791347
https://bugs.chromium.org/p/chromium/issues/detail?id=791450
https://bugs.chromium.org/p/chromium/issues/detail?id=791300
https://bugs.chromium.org/p/chromium/issues/detail?id=791522

Original change's description:
> Make DocumentThreadableLoader a ResourceOwner
> 
> It had a custom implementation to support use of RawResourceClientStateChecker.
> Separate these.
> 
> Bug: 790778, 640291
> Change-Id: Id8a29a5ad916429969b47a0258675437de284b55
> Reviewed-on: https://chromium-review.googlesource.com/802094
> Commit-Queue: Nate Chapin <japhet@chromium.org>
> Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#521177}

TBR=hiroshige@chromium.org,japhet@chromium.org

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

Bug: 790778, 640291
Change-Id: I15938754b57b8142f8d7e9c94f8aff9bdd8329ed
Reviewed-on: https://chromium-review.googlesource.com/806794Reviewed-by: Nate Chapin <japhet@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Commit-Queue: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521448}

Prevents double tapping of a survey info bar to prevent
  any double click issues e.g. this crash

Bug: 791076
Change-Id: Ifcb8961beaf4bcd1353eb1a121f51e2249de3809
Reviewed-on: https://chromium-review.googlesource.com/804751Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Daniel Park <danielpark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521447}

This test used to fail in Blink, because the default window width is
800px when running tests. The test assumed a slightly wider window in
order to not produce a horizontal scrollbar in the test (while the ref
would not produce any scrollbar that easily).

Bug: 788337
Change-Id: I30e5c628ad584862bdbdf71140d228a552e12e74
Reviewed-on: https://chromium-review.googlesource.com/806196Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521446}

Before this patch, parsing non-unicode fingerprints resulted in a failed
assertion when logging the string to terminal.

This patch adds a check for non-ASCII input string. A non-ASCII string
is not logged to terminal.

After this patch, parsing non-ASCII fingerprints does not result in a
failed assertion, because the non-ASCII fingerprint is not logged to
terminal.

Test: FingerprintParserTest.MustBeASCII
Bug: 791584
Change-Id: I0317203c1dae5a048008df074fee77982838fa1c
Reviewed-on: https://chromium-review.googlesource.com/806317
Commit-Queue: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521445}

When we restart jobs with a different proxy configuration, we might have
a delayed ResumeMainJob task in our task queue. When this ResumeMainJob() runs,
we will crash because the old main job is deleted and we have a new main
job.

This CL cancels the ResumeMainJob task and adds regression test.

Bug: 790776, 789560
Change-Id: Icd2bc9ec36c98c973f80e14541d9f68d0a046835
Reviewed-on: https://chromium-review.googlesource.com/805354
Commit-Queue: Helen Li <xunjieli@chromium.org>
Reviewed-by: Zhongyi Shi <zhongyi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521444}

Bug: 667560
Change-Id: I99f69cf7105902a4f8c0bac38442ff058182338a
Reviewed-on: https://chromium-review.googlesource.com/804675
Commit-Queue: Erik Luo <luoe@chromium.org>
Reviewed-by: Will Chen <chenwilliam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521443}

This doesn't properly plumb through the click closure, which might need
some rethinking since we share the UI with framebusting.

Bug: 661629
Change-Id: I8a38140eee2f314e98183fc0874af94dda15a626
Reviewed-on: https://chromium-review.googlesource.com/763909
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Patrick Monette <pmonette@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521442}

Change-Id: Ib22c9ebe468065a572cf3d5a2ba61cbd3ed85698
Reviewed-on: https://chromium-review.googlesource.com/804752Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521441}

Design doc:
https://docs.google.com/document/d/1i7_1VkjJ2x86BXWocK4ZuEVq_DzdYRslt2ihJ2UdlZk

Screenshot: https://drive.google.com/a/chromium.org/file/d/1qGcgcNF3vCjb0cjkkrO_Sox9OEOSQtXo

Bug: 788872
Change-Id: Ib09a31a3914aa9b244e9e6117e1cc501a102e82c
Reviewed-on: https://chromium-review.googlesource.com/791910
Commit-Queue: Thanh Le <thanhdle@chromium.org>
Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521440}

https://skia.googlesource.com/skia.git/+log/bee6cacc5435..7ffbcf909d36

$ git log bee6cacc5..7ffbcf909 --date=short --no-merges --format='%ad %ae %s'
2017-12-04 robertphillips Add unit test for SkDeferredDisplayLists (take 2)

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

Change-Id: Ie17e899f94aa96c4dc8124fcbcd2fcd7b90b9d36
Reviewed-on: https://chromium-review.googlesource.com/806914Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521439}

- Remove redundant Find/Previous buttons
- Replace "Replace" checkbox with cool icon
- 'Replace all' button is disabled when 'Replace' is
  also disabled

Screnshot: https://imgur.com/a/gp0SZ

Bug: 631163
Change-Id: I98f2ca5b8e504d13d85057a80fa5bb9cf60c6118
Reviewed-on: https://chromium-review.googlesource.com/769991
Commit-Queue: Erik Luo <luoe@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521438}

- With typemap to media/cdm/cdm_proxy.h types.
- No methods are added yet.

Bug: 785563, 641132
Change-Id: I9012cc81bb52be694d004673881aedad8b8abb09
Reviewed-on: https://chromium-review.googlesource.com/772863
Commit-Queue: Rintaro Kuroiwa <rkuroiwa@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521437}

Camera: Take photo.
Video: Capture video.

Bug: 787673
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ie9f3a72991b6384533b6d5d031ab4e11ecf2e1b2
Reviewed-on: https://chromium-review.googlesource.com/804560Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521436}

- Full rewrite of the MediaFoundation implementation video part to use
IMFCaptureEngine
- Implementation of takePhoto, setPhotoOptions and getPhotoCapabilities
- takePhoto triggers a still image capture with the highest available
resolution without stopping the video stream thanks to IMFCaptureEngine

TEST=adapted video_capture_device_unittest.cc and
webrtc_image_capture_browsertest.cc; launch Chrome with
--force-mediafoundation on Win8+ and capture video using
e.g. https://webrtc.github.io/samples/src/content/getusermedia/gum/

R=mcasas@chromium.org

Bug: 730068
Change-Id: Ib8e7f475d8120a63dd08c7b215c1eaf2c6f3d800
Reviewed-on: https://chromium-review.googlesource.com/734042
Commit-Queue: Christian Fremerey <chfremer@chromium.org>
Reviewed-by: Miguel Casas <mcasas@chromium.org>
Reviewed-by: Christian Fremerey <chfremer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521435}

Bug: 789712
TBR: gab
Change-Id: I16e059c18420a56e875c5564015531177bc213d3
Reviewed-on: https://chromium-review.googlesource.com/806815Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521434}

virtual/gpu/fast/canvas/canvas-createImageBitmap-colorClamping.html is
consistently passing on Linux and Windows and failing on Mac. Trying
to re-enable this disabled test.

https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=virtual/gpu/fast/canvas/canvas-createImageBitmap-colorClamping.html


Bug: 791554
Change-Id: I4e89ec29a19f1d9c3d7b122214dcd39dfd7eb648
Reviewed-on: https://chromium-review.googlesource.com/803903Reviewed-by: Justin Novosad <junov@chromium.org>
Commit-Queue: Mohammad Reza Zakerinasab <zakerinasab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521433}

Change-Id: Ie912416a96a1ad49df0beb01f3e974b0b17f53a1
Reviewed-on: https://chromium-review.googlesource.com/804680Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Saman Sami <samans@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521432}

Beginning with macOS 10.12, the APIs Chromium uses
to enumerate client certificates in the Keychain may
miss certificates from some smartcards; notably, ECDSA
certificates on Tokend-backed cards.

This is because Chromium uses the deprecated macOS
APIs for detecting client certificates. However, those
legacy APIs are the only way to access some identities
on other cards.

To resolve this, use both the deprecated and the
current API to enumerate client identities,
deduplicating along the way.

This is largely based on a patch from agaynor@mozilla.com
in https://codereview.chromium.org/2910893002/, updated
for the current API.

Bug: 769699
Change-Id: I706ad121d0e6827ac4830f36aeacbc7d1c959560
Reviewed-on: https://chromium-review.googlesource.com/804118Reviewed-by: Matt Mueller <mattm@chromium.org>
Commit-Queue: Ryan Sleevi <rsleevi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521431}

Profiled on android arm64, seems to speed up
net::QuicUtils::CopyToBuffer() by about 10%, translating to about 0.1%
overall.

Merge internal change: 177612619

R=rch@chromium.org

Bug: 
Change-Id: I7b85a4c91538a73af58c3fdbe32637e51693bd4a
Reviewed-on: https://chromium-review.googlesource.com/804744Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Buck Krasic <ckrasic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521430}

This CL adds a histogram to track how often loadDataWithBaseUrl is
called with a historyUrl parameter that is empty, the same as
baseUrl, or different from baseUrl.

Bug: 779661
Change-Id: Ifba359d6527454cedf4879990c8ce73f7e0225ef
Reviewed-on: https://chromium-review.googlesource.com/798354Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Richard Coles <torne@chromium.org>
Commit-Queue: James Wallace-Lee <jamwalla@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521429}

PullDeltaForMain() happens right before the commit and is supposed to
return the latest active tree delta. Therefore calling it on a pending
tree doesn't make sense and should never happen.

Bug: 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I80c4e9727d5600dfe701c948e0ae822a8718f731
Reviewed-on: https://chromium-review.googlesource.com/806475
Commit-Queue: Yi Gu <yigu@chromium.org>
Reviewed-by: Robert Flack <flackr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521428}

Remove more than a dozen test-only accessor methods and more than two
dozen FRIEND_TEST_ALL_PREFIXES() macros from SpdySession.  Move these
accessor methods to SpdyNetworkTransactionTest (three of them),
SpdySessionPool (one of them), and SpdySessionTest (many) base classes.
Also wrap direct access to private members from test subclasses into
accessor methods in test base classes.

I was originally writing a SpdySessionPeer class, see earlier patch
sets, but since almost every method is called multiple times within
SpdySessionTests, and always with |session_| argument, I ended up
relaying them through SpdySessionTest protected methods anyway, and it
was just simpler in the end to befriend SpdySessionTest and access
SpdySession members directly.

Change-Id: Iaef2084f4f5a5d81e85c34bdbbbcfd963282d86e
Reviewed-on: https://chromium-review.googlesource.com/803937
Commit-Queue: Bence Béky <bnc@chromium.org>
Reviewed-by: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521427}

We emulate the correct text selection colors for CodeMirror, but we were
awlays displaying the active colors even when the window was not focused

Bug: none
Change-Id: Idaf4983ef758a6e66479244b641b3e0245b13ded
Reviewed-on: https://chromium-review.googlesource.com/671533Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Commit-Queue: Joel Einbinder <einbinder@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521426}

Some print jobs take a very long time to render.  Add some buffer to
allow rendering to complete.  Long term, we hope to reduce the timeout
to 20s to detect if a printer is reachable.

If a printer is unreachable, this change will result in extending the
time a user waits from 20 to 120s.  This should only be encountered
by users setting up their printer for the first time.

Bug: 786182, 791230
Change-Id: I5088654b3f8a68b1178efeef98c5cb66dfe4647c
Reviewed-on: https://chromium-review.googlesource.com/804606Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Commit-Queue: Sean Kau <skau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521425}

https://pdfium.googlesource.com/pdfium.git/+log/a827770faf5c..0ddd5dc79696

$ git log a827770fa..0ddd5dc79 --date=short --no-merges --format='%ad %ae %s'
2017-12-04 thestig Break out CFX_ImageTransformer color calculations.
2017-12-04 thestig Break out CFX_ImageTransformer mono calculations.
2017-12-04 thestig Add text unit to glyph unit conversion to CPDF_Type3Char.
2017-12-04 dsinclair Cleanup CFXJSE_Engine visibility
2017-12-04 thestig Break out CFX_ImageTransformer alpha calculations.

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

After the removal of byte-swapping in V8 value serialization,
SerializedScriptValue::ToWireBytes() is strictly worse than
GetWireData(), as it creates a copy of the serialized value bytes.
Removing it makes copying obvious in the callers, so it can be removed
where that results in a performance improvement.

Bug: 
Change-Id: I2c299ac4baa89ba3d33210ce64762730390c4952
Reviewed-on: https://chromium-review.googlesource.com/804902Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521423}

https://webrtc.googlesource.com/src.git/+log/86de7e898a63..83d27683a83d

$ git log 86de7e898..83d27683a --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
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: I95402af01d9171fb8d66451818543c87471f46e2
Reviewed-on: https://chromium-review.googlesource.com/806739Reviewed-by: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521422}

The Blink Rename has made most uses of this-> unnecessary. The
relevant variable names no longer clash with the getters.

Bug: 789653
Change-Id: I6ce6cb4107354a411da57e674df667658ea5d2fc
Reviewed-on: https://chromium-review.googlesource.com/798130Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: Robert Hogan <robhogan@gmail.com>
Cr-Commit-Position: refs/heads/master@{#521421}

Support deleting a window with tracked occlusion from LayerAnimationObserver::OnLayerAnimationEnded.

Previously, if a window with tracked occlusion was deleted from
LayerAnimationObserver::OnLayerAnimationEnded, an invalid memory
access occurred in WindowOcclusionTracker::CleanupAnimatedWindows
(because the window was deleted before being removed from
|animated_windows_|).

This CL fixes this problem by removing the Window* from
|animated_windows_| in WindowOcclusionTracker::OnWindowDestroyed.

Bug: 668690
Change-Id: I77749c79d72d7cebc3a77647552605c8abec3abb
Reviewed-on: https://chromium-review.googlesource.com/806356
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521420}

In an engine with subpixel support, these tests would render ever so
slightly "incorrectly", compared to the ref, which assumed that all
values less than 1px be rounded down to 0.

Bug: 788337
Change-Id: I5810a9620763fa0065ee480ed953a1426c95cf93
Reviewed-on: https://chromium-review.googlesource.com/806173Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521419}

Relevant V8 CLs have landed in this autoroll:
https://chromium-review.googlesource.com/c/chromium/src/+/805157

Bug: 700624, 727514
Change-Id: I73392015ee47bb79499cc7ee8b447673c00387da
Reviewed-on: https://chromium-review.googlesource.com/806258Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Commit-Queue: Erik Luo <luoe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521418}

The tracing request format currently in use is deprecated and does not
support enabling some features such as system tracing. Switch to the new
format. This requires us to tokenize the category filter ourselves;
this is done using the same algorithm as chrome.

This also fixes a regression where the category filter was set to '"*"'
(double quotes included) which does not work properly, disabling all
events. We should use the empty string, which simply means to apply the
defaults. The android backend appears to be prepared to handle this as
well (mapping it to "_DEFAULT_CHROME_CATEGORIES").

BUG=786091

Change-Id: Ic60a3db7f4decc5ee046ce50bfc58d3c22211558
Reviewed-on: https://chromium-review.googlesource.com/804857Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Commit-Queue: Michael Spang <spang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521417}

r516772 was the last of many commits that resolved a number of problems
that made this test suite flaky on Windows. This CL enables tests that
had been disabled over time due to flakes, yet now seem to run okay.

BUG=133341,751031,751543,764415,789913
This CL was uploaded by git cl split.

R=sky@chromium.org

Change-Id: I753c741a9194f489f072cd86d72407a1294150c3
Reviewed-on: https://chromium-review.googlesource.com/784937Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521416}

r516772 was the last of many commits that resolved a number of problems
that made this test suite flaky on Windows. This CL enables tests that
had been disabled over time due to flakes, yet now seem to run okay.

BUG=69475,129235,147579,419468,764415
This CL was uploaded by git cl split.

R=sky@chromium.org

Change-Id: I99053d0c38c27a0ad91b426214bff6365c3b4a5e
Reviewed-on: https://chromium-review.googlesource.com/784934Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521415}

r516772 was the last of many commits that resolved a number of problems
that made this test suite flaky on Windows. This CL enables tests that
had been disabled over time due to flakes, yet now seem to run okay.

BUG=499858,764415
This CL was uploaded by git cl split.

R=sky@chromium.org

Change-Id: Ic884babd5f4814fdf173a18fe91c6e71ce828f92
Reviewed-on: https://chromium-review.googlesource.com/785018Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521414}