GitLab

https://chromium.googlesource.com/chromiumos/chromite.git/+log/f3f1110ffae1..33146a20418a

$ git log f3f1110ff..33146a204 --date=short --no-merges --format='%ad %ae %s'
2018-03-26 jkop Wall-clock time and dashboard are no longer new.

Created with:
  roll-dep src/third_party/chromite
BUG=chromium:None


The AutoRoll server is located here: https://chromite-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=chrome-os-gardeners@chromium.org

Change-Id: Ic21fc7ba8d0b7ce73a81d9006de2177212838be7
Reviewed-on: https://chromium-review.googlesource.com/986936Reviewed-by: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#547090}

TBR: hbos
Bug: 827450, 680172
Change-Id: I2d351e517dae3b011bd477c9482e756344df4fe2
Reviewed-on: https://chromium-review.googlesource.com/987394Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547089}

Changes Init() not to pass in the content_window_, and changes
implementations not to cache it. There is no reason for each DWTH to
cache the content_window_ when it can get it from the
DesktopNativeWidgetAura.

BUG=none
TEST=none

Change-Id: I7e89b81606079e5e291fd528d275c8a47ada1c8e
Reviewed-on: https://chromium-review.googlesource.com/986930
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Michael Wasserman <msw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547088}

https://skia.googlesource.com/skia.git/+log/04bea3bf76e2..f5b418839f2e

$ git log 04bea3bf7..f5b418839 --date=short --no-merges --format='%ad %ae %s'
2018-03-29 angle-skia-autoroll Roll skia/third_party/externals/angle2/ 96310cdad..5cf4d06cd (5 commits)
2018-03-29 robertphillips Add SK_API to GrContextThreadSafeProxy
2018-03-29 skcms-skia-autoroll Roll skia/third_party/externals/skcms/ ffeadc2cb..d8ece14aa (1 commit)
2018-03-28 halcanary SkPDF/Clusterator: expose to the light of a unit test

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;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=mtklein@chromium.org

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

https://webrtc.googlesource.com/src.git/+log/a21090b7706e..9d6f73bfb299

$ git log a21090b77..9d6f73bfb --date=short --no-merges --format='%ad %ae %s'

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


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

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

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


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

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

BUG=none
TEST=none

Change-Id: If0c65e7139facaa72f19a92e41939dd82646839e
Reviewed-on: https://chromium-review.googlesource.com/855776Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547085}

Otherwise there may be a clip inversion when drawing mask layers
for objects which include CSS clip.

Bug:822906

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I037304f4b6d777bf14aa1204631dac7cf24da139
Reviewed-on: https://chromium-review.googlesource.com/986718Reviewed-by: Tien-Ren Chen <trchen@chromium.org>
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547084}

This CL changes the implementation of the said method to print out the
primaries coordinates instead of the Chromatic adaptation matrix, if
CUSTOM primaries are used.  The former is more informative than the
latter.

Example output with primaries triangle (edid and recalculated) can be
found in https://i.imgur.com/p5q9uyY.png (https://imgur.com/a/gdj62).

Bug: 821393
Change-Id: I690d2d6df1ece0ca75e4dcbbdb42bbd092eb234c
Reviewed-on: https://chromium-review.googlesource.com/985316
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Reviewed-by: ccameron <ccameron@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547083}

Adds EmitAshInitialized method to session_manager_client - the
method requests from session manager to emit "ash-initialized"
upstart signal. The signal can be used to run upstart jobs that
should be run when ash is initialized - for example, to run hammerd
task, which will ensure that detachable base pairing signals are
emitted after ash starts observing dbus signals emitted by hammerd.
DetachableBaseHandler relies on hammerd signals to determine when,
whether a detachable base notification should be shown to the user, so
these signals should be re-emitted whenever ash is (re)started.

Makes ash::Shell::Init call the new session manager method to actually
emit the upstart signal once it's done initializing.

BUG=818057

Change-Id: If901e125f8f3f8563c22a0633644a0c6ff143f7d
Reviewed-on: https://chromium-review.googlesource.com/981330
Commit-Queue: Toni Barzic <tbarzic@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Reviewed-by: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547082}

The ownership chain for the instance_ variable looks like:
LayoutCustom -> CSSLayoutDefinition::Instance -> v8::Object (instance_).

As LayoutCustom isn't a ScriptWrappable (lifetime is manually controlled)
using the TraceWrapperV8Reference was the wrong thing to do there.

Changed to ScopedPersistent.

Bug: 726125
Change-Id: I452a44e6914d042ca12f3bce3f8bce4a068bba5d
Reviewed-on: https://chromium-review.googlesource.com/986986Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547081}

When in WebVR presentation, if we transition to metaworld to accept a
permission, when we return we should not show the permissions.

Bug: 827304,827306
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I9280a04950dcc24f163be9f58fbf27c14c9ab6bd
Reviewed-on: https://chromium-review.googlesource.com/986290
Commit-Queue: Ian Vollick <vollick@chromium.org>
Reviewed-by: Christopher Grant <cjgrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547080}

This patch refactors inefficient usage of GetCharacterRange
as discussed in CL:979732.

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I1d1b135eaf8d7a12f0fd918316428d5bf30cbcc0
Reviewed-on: https://chromium-review.googlesource.com/981848Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547079}

It's better to put the class definition in a header file and the implementation
in a C++ source file. Then, the source file is made part of the project, meaning
it is compiled separately.

BUG=791371

Cq-Include-Trybots: luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win10_chromium_x64_rel_ng
Change-Id: Ic58d0c714b405ced8974aca22c3b20f7b479682f
Reviewed-on: https://chromium-review.googlesource.com/982751Reviewed-by: Miguel Casas <mcasas@chromium.org>
Commit-Queue: Junwei Fu <junwei.fu@intel.com>
Cr-Commit-Position: refs/heads/master@{#547078}

These pieces will be used to store/represent page thumbnails.

Bug: 794828
Change-Id: I242ec8376a36b2b41590bf89fb166a924aa13ca7
Reviewed-on: https://chromium-review.googlesource.com/984952
Commit-Queue: Dan H <harringtond@chromium.org>
Reviewed-by: Carlos Knippschild <carlosk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547077}

This adds three new metrics and fixes two existing ones to ensure
they are recorded accurately when we suspend for preload=metadata,
the new metrics are:

Media.PreloadMetadataSuspendWasIdeal.Audio
Media.PreloadMetadataSuspendWasIdeal.AudioVideo
Media.PreloadMetadataSuspendWasIdeal.Video

Which are boolean and set to true if a preload=metadata suspend is
not *immediately* aborted. Immediate abort can only result from
WMPI not knowing the true playback state until HAVE_FUTURE_DATA;
this is a lie we'd like to fix, and these metrics will help us
judge the importance of fixing it.

The fixed metrics are:
Media.HasEverPlayed
Media.TimeToFirstFrame.SRC
Media.TimeToPlayReady.SRC

HasEverPlayed is fixed by not counting the BUFFERING_HAVE_ENOUGH
event when it's issued for a suspended startup (determined by
a new media log property, which is gross, but how the existing
system must work for now).

TimeToFirstFrame and TimeToPlayReady are metrics which are relative
to the time of load start, but this does not work when we stop
after metadata. To ensure these metrics are still accurate once
playback does resume, we cache the |time_to_metadata_| value and
use it account for the section of time prior to metadata suspend.

BUG=694855, 827396
TEST=manually verified chrome://histograms has the right values.

Change-Id: I5cdeea08a798ed0875c2ce603d6ee2b9a31d1e55
Reviewed-on: https://chromium-review.googlesource.com/980589
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547076}

Without this dependency component builds will fail to link.

TBR: sergeyu
Bug: 822474
Change-Id: Ieb8d7309117e6e55c5fa3c8866920e1495dbbc3b
Reviewed-on: https://chromium-review.googlesource.com/987395Reviewed-by: Wez <wez@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547075}

This reverts commit d3f34a3f.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> [DevTools] prepare tests for new queryObjects
> 
> TBR=pfeldman@chromium.org
> 
> Bug: chromium:825349
> Change-Id: I70390a29c684afa69555b14aa20b96345d5ce1e3
> Reviewed-on: https://chromium-review.googlesource.com/978844
> Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
> Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#545600}

TBR=pfeldman@chromium.org,kozyatinskiy@chromium.org

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

Bug: chromium:825349
Change-Id: I21810896da315aea8179938f32e90a2512012252
Reviewed-on: https://chromium-review.googlesource.com/987332Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547074}

When the P2P Sharing flag is turned on, all existing offline pages will
become legacy pages.  Before they can be shared, they must be published
to a public directory.

This is the first of several changes to publish the file.  This adds the
calls down to publish the file, but doesn't actually do the publishing
yet.

Subsequent changes will publish the file, and update the OfflinePageModel
database with the new file path, and include unit tests for the code
which actually takes the actions.

This change covers one of two entry points.  Here we cover entry via
the chrome overflow menu.  The second entry point is from the downloads
home activity, which will be in another changelist, but should be able
to re-use parts of this change (perhaps the
OfflinePageBridge.publishThenShare method).

Bug: 758690
Change-Id: I31dc8db25b274d98f4bac88c98a96776c1d02c0b
Reviewed-on: https://chromium-review.googlesource.com/982734
Commit-Queue: Peter Williamson <petewil@chromium.org>
Reviewed-by: Yafei Duan <romax@chromium.org>
Reviewed-by: Jian Li <jianli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547073}

Change API contract for IPAddress::AssignFromIPLiteral() such that on failure the original value is reset.

This addresses a TODO, and avoids a copy in the success case.

Change-Id: I8ff0f1154a7e4a8085c6a9f50d72bb99dfb51b7c
Reviewed-on: https://chromium-review.googlesource.com/985907Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547072}

Bug: 821118
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I6233f1767349def3ff810d49af61ebd800ac9b26
Reviewed-on: https://chromium-review.googlesource.com/981788Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>
Commit-Queue: Alexander Alekseev <alemate@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547071}

Introducing new OfflinePages.StorageInfo metrics which are collected
after every time an offline page is successfully saved.

The new metrics will report storage info for both internal and external
volumes:
- internal refers to the volume that contains the Chrome app.
- external refers to the volume that contains public download directory.

Also marking old UMAs as deprecated, as they are no longer collected in
taskified offline page model.

Bug: 772204
Change-Id: Id3e7ed89d2106d0c255bc708be8a0b131f8f41f2
Reviewed-on: https://chromium-review.googlesource.com/976688Reviewed-by: Dmitry Titov <dimich@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Commit-Queue: Yafei Duan <romax@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547070}

This is a precusor cleanup extracted from
https://chromium-review.googlesource.com/c/chromium/src/+/957760/7
in the broader effort to restrict usage of MessageLoop::current().

This CL was uploaded by git cl split.

R=jamiewalch@chromium.org

Bug: 825327
Change-Id: Iaf8fde959a9c7848200e2c08d9c43236b1b331aa
Reviewed-on: https://chromium-review.googlesource.com/986536Reviewed-by: Joe Downing <joedow@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547069}

It has the default implementation of Node::MaxCharacterOffset() = NOTREACEHD(),
and all call sites call it for CharacterData.

After using ToCharacterData(node)->length() to instead of Node::MaxCharacterOffset(),
The api has no use, so remove it.

Bug: 825077
Signed-off-by: Yuhong Sha <yuhong.sha@samsung.com>
Change-Id: Ic624fd70504cdf68a4b5b1e9808ba8cf41d85dd3
Reviewed-on: https://chromium-review.googlesource.com/985678Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547068}

The profile chooser should not be shown when off the record. The button
still shows but cannot be selected.

Bug: chromium:822070
Change-Id: I40d9cf451444d8f16053282c638ee46abcca2af3
Reviewed-on: https://chromium-review.googlesource.com/984398Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Commit-Queue: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547067}

This subpage and link to it are hidden when the
--enable_experimental_accessibility_features flag is on.

Currently the subpage has no functionality, but what exists does match
the UI mock, except that I've also added a description to the link to
the subpage that lets users know they can manage TTS engines from that
page.

Bug: 823359
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ia3e049cb0c9b64e861a7981d29a6642061029275
Reviewed-on: https://chromium-review.googlesource.com/969322Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Katie Dektar <katie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547066}

Reading blobs that are loaded from IDB seem to be flaky. Large value
wrapping is a big customer of IDB blobs, so disable this for now while
the problem is investigated.

Bug: 756447
Change-Id: I64e46803d7433cbf6c701457c8e96028a015d74f
Reviewed-on: https://chromium-review.googlesource.com/987207
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547065}

BUG=827365

Change-Id: I494e7d01245cd91864cb49ab9a64990f1be93752
Reviewed-on: https://chromium-review.googlesource.com/987240Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Weidong Guo <weidongg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547064}

This CL adds two classes for the suggestion and keyword views within an
OmniboxResultView. The intent is to move many of the members of
OmniboxResultView to one of the new classes. This CL doesn't move the
code yet. This Cl is boilerplate for the new classes. This is done to
better focus following CLs that will move code.

Bug: None
Change-Id: I0e9c766afef9c8cb13ff3b2a9b01a85c2041c51e
Reviewed-on: https://chromium-review.googlesource.com/980798
Commit-Queue: Dave Schuyler <dschuyler@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547063}

Add animation for the Android toolbar view to slide in when JavaScript
tab modal dialog is showing and the renderer is blocked on input events.

Bug: 817461
Change-Id: I3ed69ffabe736100ac209be59f34eeedd4189cee
Reviewed-on: https://chromium-review.googlesource.com/942378
Commit-Queue: Becky Zhou <huayinz@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547062}

Change-Id: Ie95ff321b2f5cdb0448e83bbba3a77b7954aab34
Reviewed-on: https://chromium-review.googlesource.com/987102
Commit-Queue: Carlos IL <carlosil@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547061}

This ports the following wiki pages into markdown:
https://www.chromium.org/developers/testing/gpu-testing
https://www.chromium.org/developers/testing/gpu-testing/gpu-bot-details
https://www.chromium.org/developers/how-tos/gpu-wrangling
https://www.chromium.org/developers/how-tos/debugging-gpu-related-code

and updates *some* of the old outdated content.

Bug: 813153
Change-Id: Ic5f1b58659bbdb691343785cb18c50f4d55c177f
Reviewed-on: https://chromium-review.googlesource.com/987233Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547060}

- Creates History Coordinator and TableViewController files.
- The Coordinator presents a TableViewContainer with the newly created HistoryTableVC.
- HistoryTableVC is no-op, only inherits from ChromeTableVC.

Screenshot:
https://drive.google.com/open?id=1FuHK2MaTRNCJ9HhkBZO3zGPjEiRY7Ih1

Bug: 805192
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: Ia090ed755e5a67097cf21b88fad0d198b405e8bf
Reviewed-on: https://chromium-review.googlesource.com/985235Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Commit-Queue: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547059}

This was just needless indirection.

BUG=none
TEST=none
TBR=rohitrao

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I9b052d50e21b29e5a18181c99f0b4be974e10046
Reviewed-on: https://chromium-review.googlesource.com/987101Reviewed-by: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547058}

Bug: 773295
Change-Id: I5a1783b49d8e20119843af96919bc341d6612f6e
Reviewed-on: https://chromium-review.googlesource.com/986675
Commit-Queue: Mark Pilgrim <pilgrim@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547057}

I introduced a bad feature flag infobar for SignedHTTPExchange.
[1] https://chromium.googlesource.com/chromium/src/+/a753229
This CL introduced the bad flag infobar to Android Chrome not only for
SignedHTTPExchange but also for all bad flags which are listed as kBadFlags in
bad_flags_prompt.cc.
Before this CL, Android Chrome didn't show any bad flag infobar.

This change caused parformance regressions because a bad flag
"--ignore-certificate-errors-spki-list" is used for perfomance testing.
(crbug.com/822258#c13)

So I added the check of "--enable-automation" flag in ShowBadFlagsPrompt().
[2] https://chromium.googlesource.com/chromium/src/+/33b13b2
And I added --enable-automation flag in GetFromBrowserOptions() in catapult.
[3] https://chromium.googlesource.com/catapult.git/+/df3154e
I thought that this check already existed in startup_browser_creator_impl.cc for
Desktop not to show the infobar during tests.
But on Desktop, a infobar for the automation is always shown when the flag is set.
https://chromium.googlesource.com/chromium/src/+/7a66b24

After the CL [3] pixel_tests are failing on Windows.
https://chromium-review.googlesource.com/c/chromium/src/+/969827
And Catapult AutoRoll is being blocked by the failure.
https://crbug.com/824779
I think this is due to the shadow of the infobar.

What I wanted to do was to show the infobar for SignedHTTPExchange.
And I think we don't need to show the bad flag infobar on Android for the flags
which can't be added on non-rooted devices.

So this CL changes the behavior of Android Chrome to show the the infobar only
for flags which are available in about:flags.


Bug: 825082,803774,822258
Change-Id: Ieee2bee3f31219645c642d9238599ff82483feeb
Reviewed-on: https://chromium-review.googlesource.com/977252
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Reviewed-by: Tommy Martino <tmartino@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547056}

Change-Id: I30afa26406d7313a220fada540f211026bf255f9
Reviewed-on: https://chromium-review.googlesource.com/985963Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Commit-Queue: Paul Irish <paulirish@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547055}

This cl adds the mechanism to make Keyboard Shortcut Viewer (KSV)
searchable in launcher.
1. Adds an InternalDataSource.
2. Adds an InternalAppResult.
3. Adds unit tests for searching internal app.

Bug: 790748
Test: manual and AppSearchProviderTest.FetchInternalApp
Change-Id: I8cae318b51d75516575c9b02cb7565b1dc748951
Reviewed-on: https://chromium-review.googlesource.com/982995
Commit-Queue: Tao Wu <wutao@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547054}

When ServiceWorkerServicification is enabled, this CL is to make
ServiceWorkerURLLoaderJob and ServiceWorkerSubresourceLoader work without
NetworkService. This CL tries to create an URLLoaderRequestHandler for a service
worker (ServiceWorkerControlleeRequestHandler) even if no NetworkService flag.

ServiceWorkerServicification still doesn't work since
ServiceWorkerURLLoaderJob::ProceedWithResponse() hasn't been implemented yet.

Bug: 752394
Change-Id: I808346f248bba254e20e708d372dde46ecc103fe
Reviewed-on: https://chromium-review.googlesource.com/888361Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547053}

Since notification_helper is a standalone executable, adding UMA support for
it needs extra work. We need to create persistent histograms in memory and
write the memory to a file upon exit of the process. These files are
accumulated inside a new directory called NotificationHelperMetrics under User
Data directory, and will eventually be reported by the browser process.

This CL moves persistent_histogram_storage.{h,cc} from chrome/installer/setup
to base/metrics as the class is now also used by notification_helper.exe, in
addition to setup.exe. This CL updates this class for general-purpose usage,
and adds unit tests for it.

This CL also logs the runtime of the notification_helper process to UMA.

Bug: 734095
Change-Id: Ia95f002378718b0e033f8da1aa7c260ec04030c3
Reviewed-on: https://chromium-review.googlesource.com/959543
Commit-Queue: Xi Cheng <chengx@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Brian White <bcwhite@chromium.org>
Reviewed-by: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547052}

This is a re-land of 865455, which actually works.

TEST=ran video_decoder ppapi example

Bug: 754986
Change-Id: Ibf0e20244173b8ade0c1fca134825b2cd2ace660
Reviewed-on: https://chromium-review.googlesource.com/902459
Commit-Queue: Fredrik Hubinette <hubbe@chromium.org>
Reviewed-by: ccameron <ccameron@chromium.org>
Reviewed-by: Bill Budge <bbudge@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547051}