GitLab

The Previews infobar should read "Simplified page to save data and load
faster. Options available near bottom of the screen." when shown, rather
than reading the infobar strings.

BUG=732944

Review-Url: https://codereview.chromium.org/2939143002
Cr-Commit-Position: refs/heads/master@{#481046}

Improvements:
* devtools drives the test, not the inspected page
* tests are pure JS
* doc-typed test harness
* improved debugging experience (surround test function with TestRunner.debugTest)

This copies a couple of existing tests - once this infra has been proven,
we'll delete the originals.

Perf:
- Old (66 sec) [0]
- New (45 sec) [1]

[0] src/blink/tools/run_layout_tests.py --repeat-each=100 inspector/console/console-clear.html
[1] src/blink/tools/run_layout_tests.py --repeat-each=100 http/tests/inspector/devtools-js/console-clear.js

BUG=667560

Review-Url: https://codereview.chromium.org/2837083003
Cr-Commit-Position: refs/heads/master@{#481045}

This patch fully removes the dependency from NetworkRequest to
NetworkManager.

R=dgozman,pfeldman
BUG=374557

Review-Url: https://codereview.chromium.org/2921963002
Cr-Commit-Position: refs/heads/master@{#481044}

BUG=723734
R=mmenke@chromium.org

Review-Url: https://codereview.chromium.org/2909533002
Cr-Commit-Position: refs/heads/master@{#481043}

* Remove unsupported 1.5x.
* Removed obsolete  use_125_dsf_for_ui_scaling flag. (it should be always true)
* Added large FHD screen scenario.

BUG=734204
BUG=b/62475614
TEST=Updated unit tests. Manual test on cave with manually teaking the scale.

Review-Url: https://codereview.chromium.org/2945913003
Cr-Commit-Position: refs/heads/master@{#481042}

Converted a LOG_IF to a DLOG_IF to eliminate spam.

BUG=735249

Change-Id: Ic2642d5dc860ad6a43c2a54d1548018611024c84
Reviewed-on: https://chromium-review.googlesource.com/541899Reviewed-by: Joshua Bell <jsbell@chromium.org>
Commit-Queue: Joshua Bell <jsbell@chromium.org>
Commit-Queue: Chris Mumford <cmumford@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481041}

these are currently intended to be built using the host toolchain and
used on the host rather than the target; modeled after imagediff.

BUG=695482,732565

Change-Id: I3d95d9526ecef3026bbbaf4a13525cf6add02062
Reviewed-on: https://chromium-review.googlesource.com/540703Reviewed-by: Urvang Joshi <urvang@chromium.org>
Commit-Queue: James Zern <jzern@google.com>
Cr-Commit-Position: refs/heads/master@{#481040}

In LayoutGeometryMap, when there's a non-uniform step (e.g. when inside
multicol), we fall back on doing it the slow way, i.e. we involve
LayoutObject::MapLocalToAncestor(). The mode flags initially passed from
ClientQuads() will just be echoed here, so we need to make sure that
kUseTransforms is specified.

BUG=683925

Change-Id: I2a6fe83fb1332a19a581203d6c44c6924da3f027
Reviewed-on: https://chromium-review.googlesource.com/541339Reviewed-by: Chris harrelson <chrishtr@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@opera.com>
Cr-Commit-Position: refs/heads/master@{#481039}

Also add self to AUTHORS.

Bug: v8:6504
Change-Id: Ia5bceee91c7589bc53198e69c5623f1a26918c02
Reviewed-on: https://chromium-review.googlesource.com/541718
Commit-Queue: Adam Klein <adamk@chromium.org>
Reviewed-by: Adam Klein <adamk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481038}

This cl contains the following changes
* API impl of the three shorthand properties (CSSShorthandPropertyAPIOverflow,
 CSSShorthandPropertyAPIFont, CSSShorthandPropertyAPIFontVariant).

* Moved ConsumeFontVariantCSS21 from CSSPropertyParser.cpp to CSSPropertyFontUtils.
- this function is being used by CSSShorthandPropertyAPIFont and also another function
from CSSPropertyParser.cpp.

* Changed CSSPropertyParser.cpp to use the new API parseShorthand for the three properties
above. Additional changes are:
- moved two helper functions (ConsumeSystemFont and ConsumeFont) to be private functions of CSSShorthandPropertyAPIFont.
- removed ConsumeFontVariantShorthand that's no longer needed.

* Added the properties API to BUILD file and CSSProperties.json5.

* Changed AddProperty signature so that "implicit" is now an enum.
- We previously passed on bool literals to this function. This change makes the function follow the style guide.

Diff: https://gist.github.com/810c7e46324ff8db8efb6f34ddfdad42/revisions
Review-Url: https://codereview.chromium.org/2938983002
Cr-Commit-Position: refs/heads/master@{#481037}

This CL replaces ScopedVector with std::vector<std::unique_ptr>
in components/cronet.
ScopedVector is deprecated, see bug.

BUG=554289

Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester
Change-Id: I93ed1cac20093344cc9d46c648f02cb579629794
Reviewed-on: https://chromium-review.googlesource.com/536240
Commit-Queue: Han Leon <leon.han@intel.com>
Reviewed-by: Helen Li <xunjieli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481036}

Demo: https://bugs.chromium.org/p/chromium/issues/detail?id=705816#c20

BUG=705816

Review-Url: https://codereview.chromium.org/2950673002
Cr-Commit-Position: refs/heads/master@{#481035}

TBR=wez

Bug: 
Change-Id: I3016dea4130a0020158cb5d0ff610b6c4fa19268
Reviewed-on: https://chromium-review.googlesource.com/541897Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Commit-Queue: mark a. foltz <mfoltz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481034}

We currently build Skia with SK_IGNORE_GPU_DITHER, effectively disabling GPU dithering.

Remove the build flag to align Ganesh with the sw rasterizer dithering behavior.

Note: Blink only sets the SkPaint dither flag for gradients at this time.

Aside from the flag change, this CL includes:

  * a handful of trivial layout tests rebaselines
  * some Canvas layout/unit test adjustments (relaxed exact pixel value checks to
    accommodate dithering)
  * a couple of trivial GPU pixel test rebaselines

BUG=731693

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: I2fc6f8d23272504838c55427c48c4ea7d641deae
Reviewed-on: https://chromium-review.googlesource.com/535714Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Mike Reed <reed@chromium.org>
Commit-Queue: Florin Malita <fmalita@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481033}

- Add a new OOBE screen for waiting for container ready.
- The screen will be shown after user click Continue in the value prop
  screen.
- The screen will show a progress animation while container is
  initializing.
- The OOBE flow will end after container is ready or a one
  minute timeout is reached.
- A temporary 5 second delay is added to avoid potential racing issue.

BUG=b/62549680
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2947483002
Cr-Commit-Position: refs/heads/master@{#481032}

It's required recently that any extension loaded from a directory
as an unpacked extension in chrome://extensions must be whitelisted
with a unique whitelisted ID. Otherwise it will be blocked from being
installed.

Bug: 735242
Change-Id: I7c5aa5460cee5f17e76ef7fc4473ed9cf48e0bea
Reviewed-on: https://chromium-review.googlesource.com/541955Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Weiyong Yao <braveyao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481031}

Using the ScreenAvailability enum added in
https://chromium-review.googlesource.com/c/527376/ in content/browser
and chrome/browser/

BUG=723032
TEST=existing/updated tests

Change-Id: I6667f4d6237c242dfe9446cbf9e82dd30f0858ec
Reviewed-on: https://chromium-review.googlesource.com/530284
Commit-Queue: Anton Vayvod <avayvod@chromium.org>
Reviewed-by: mark a. foltz <mfoltz@chromium.org>
Reviewed-by: Derek Cheng <imcheng@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481030}

Revert of Move SafeBrowsing files from //chrome/browser to a separate target. (patchset #5 id:80001 of https://codereview.chromium.org/2940403002/ )

Reason for revert:
This caused compile failures (https://uberchromegw.corp.google.com/i/chromium.win/builders/Win%20x64%20Builder/builds/34160)

Original issue's description:
> Move SafeBrowsing files from //chrome/browser to a separate target.
>
> BUG=
>
> Review-Url: https://codereview.chromium.org/2940403002
> Cr-Commit-Position: refs/heads/master@{#481019}
> Committed: https://chromium.googlesource.com/chromium/src/+/e4da0e76b5b75d453fec094f62ad06659e66708e

TBR=rockot@chromium.org,vakh@chromium.org,yzshen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Review-Url: https://codereview.chromium.org/2951883002
Cr-Commit-Position: refs/heads/master@{#481029}

Bug: 654988
Change-Id: I7f91eafc52aed40c8243e657b85c821605f9c76b
Reviewed-on: https://chromium-review.googlesource.com/526914Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Commit-Queue: Sam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481028}

This can happen when rotation animator sets the orientation after
animation is completed. We shouldn't save this anyway, as we changed
the semantics as follows:
a) A user/API driven rotation change in tablet mode should be treated
  as user locked rotation.
b) A user will not be able to change the orientation in tablet mode
  from settings.

BUG=734107
TEST=Manual + unit test

Review-Url: https://codereview.chromium.org/2951773002
Cr-Commit-Position: refs/heads/master@{#481027}

This CL addresses a TODO from
chrome/renderer/chrome_render_view_observer.cc to associate
ChromeViewMsg_GetWebApplicationInfo and
ChromeViewHostMsg_DidGetWebApplicationInfo IPCs with frame, rather than
view.

This CL helps ensure that
ChromeRenderFrameObserver::OnGetWebApplicationInfo (formerly a method of
ChromeRender*View*Observer) always operates on a local frame (and
therefore this CL helps with moving GetDocument method from WebFrame to
WebLocalFrame in a follow-up CL - https://crrev.com/2928033002).

BUG=416660

Review-Url: https://codereview.chromium.org/2942583004
Cr-Commit-Position: refs/heads/master@{#481026}

Different types are: inline, fullscreen and picture-in-picture.

Bug: 727322
Change-Id: I72bfa1a1f8f46079880f60cfd5b5d886875d06d6
Reviewed-on: https://chromium-review.googlesource.com/528082Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481025}

BUG=None

Change-Id: I4f0c8fd5a24a5bfc1f3a81c96026a33c7700639e
Reviewed-on: https://chromium-review.googlesource.com/538980
Commit-Queue: Yuzhu Shen <yzshen@chromium.org>
Reviewed-by: Randy Smith <rdsmith@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481024}

The patch which added the butter-bar for unimplemented modes
broke the task-profiler case for the heap-profiling flag.

Bug: 705461
Change-Id: Ib293ce8aa8d4c2f763c7b9ba414923381f2d9870
Reviewed-on: https://chromium-review.googlesource.com/540983Reviewed-by: Sigurður Ásgeirsson <siggi@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481023}

This CL sends the Expect-CT expiration into the reporter, to include in the
report as key "effective-expiration-date".

http://httpwg.org/http-extensions/expect-ct.html#rfc.section.3.1

BUG=679012

Review-Url: https://codereview.chromium.org/2944953002
Cr-Commit-Position: refs/heads/master@{#481022}

[Offline pages]: Check ResponseHeader for null pointer before getting response code in Background Offliner

Bug: 734371
Change-Id: Ie5f3803931c67d27223564b542657c9824a59e36
Reviewed-on: https://chromium-review.googlesource.com/538740
Commit-Queue: Cathy Li <chili@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481021}

NOTRY=true
TBR=ulan@chromium.org

Bug: 735173
Change-Id: Iac9cceb3f5a05aa249d1565337602f2a65b4ee79
Reviewed-on: https://chromium-review.googlesource.com/541607
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Reviewed-by: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481020}

BUG=

Review-Url: https://codereview.chromium.org/2940403002
Cr-Commit-Position: refs/heads/master@{#481019}

Add a feature to enable a timestamp on the Previews infobar. This
timestamp should be shown when the previews_freshness is greater
than the finch controlled "min_staleness_in_minutes" and less than
"max_staleness_in_minutes".

BUG=718643

Review-Url: https://codereview.chromium.org/2940083003
Cr-Commit-Position: refs/heads/master@{#481018}

TBR=charliea@chromium.org

Bug: 713222
Change-Id: I012ede587d5445937b1a92f0c985853a91e6d55c
Reviewed-on: https://chromium-review.googlesource.com/540827Reviewed-by: rnephew <rnephew@chromium.org>
Commit-Queue: rnephew <rnephew@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481017}

At this point there seems to be little danger that people will add
lines to SmokeTests willy-nilly, so I don't think we need special
owners for it.

R=ojan@chromium.org, jbudorick@chromium.org, peter@chromium.org

Change-Id: I75a71b1ee800348742177caf3ea94b1aca675d8b
Reviewed-on: https://chromium-review.googlesource.com/541603Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Ojan Vafai <ojan@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481016}

PersistenceDelegate is a new interface for persisting the contents of
the HostCache. This commit includes the interface itself, the logic in
HostCache for interacting with it, and a mock implementation of the
interface for testing. It does not include support for immediate data
removal since that won't be needed for the currently planned use case.

BUG=605149

Review-Url: https://codereview.chromium.org/2943143002
Cr-Commit-Position: refs/heads/master@{#481015}

Added the logic to create archive directories if the result of SavePage
is ARCHIVE_CREATION_FAILED. This is going to prevent potential increment
of failure rate once we move temporary pages to cache directory, and
download pages to external downloads directory which are vulnerable to
removal by user actions.
Also added an UMA counting the result of archive creation and related
tests.

Bug: 725266
Change-Id: Id726365ea53786077d71b02831f6dfca0d16c4f3
Reviewed-on: https://chromium-review.googlesource.com/540995Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Filip Gorski <fgorski@chromium.org>
Commit-Queue: Yafei Duan <romax@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481014}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/821867d51814..1556ddcf20a1

$ git log 821867d51..1556ddcf2 --date=short --no-merges --format='%ad %ae %s'
2017-06-20 simonhatch Dashboard - Add link to bisect docs in output.

Created with:
  roll-dep src/third_party/catapult


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

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

Reintroduces document click handler to screen_account_picker which brings
lock screen app window (if one exists) to foreground - this should happen
if the user clicks outside user pod/login header bar.

Removes background dim when an app window is in background - new UI
has the background dimmed by default.

Review-Url: https://codereview.chromium.org/2945153002
Cr-Commit-Position: refs/heads/master@{#481012}

BlockingPool is deprecated and being replaced with TaskScheduler.

BUG=667892

Change-Id: I9a4f133ccb3b173de468e8e26be179182b57eb64
Reviewed-on: https://chromium-review.googlesource.com/540475Reviewed-by: Henrik Grunell <grunell@chromium.org>
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481011}

This file contains a ChromeRecovery.crx3, which contains a
ChromeRecovery.exe.

Bug: 687231
Change-Id: Ia9789b1c5c4cd8884d56c25e0765045dda7a749a
Reviewed-on: https://chromium-review.googlesource.com/541101Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481010}

The allocator shim is temporary until the base on can be refactored. The rest of the machinery for out-of-process heap profiling is hooked up in this patch.

Review-Url: https://codereview.chromium.org/2949703002
Cr-Commit-Position: refs/heads/master@{#481009}

This CL creates a feature-policy directory under wpt/, as well, provides some
framework for testing features with feature policy.

Using the framework, this CL added feature policy tests for payment, and
modified existing feature policy usb tests using the framework introduced.

Bug: 732003
Change-Id: I7d690acfe70b63d4935f6a4c6c43e94da5503827
Reviewed-on: https://chromium-review.googlesource.com/533953Reviewed-by: Ian Clelland <iclelland@chromium.org>
Commit-Queue: Luna Lu <loonybear@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481008}

NOPRESUBMIT=true

Bug: 
Change-Id: I5c5b274bb7f520ef55081f3779922c5d0955e12a
Reviewed-on: https://chromium-review.googlesource.com/540289
Commit-Queue: Pavel Feldman <pfeldman@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#481007}