GitLab

When Clearing Presented state, it is possible that no Visible item
exists.
Check before dereferencing.

Fix cl/2606873002

BUG=674986

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

The favicon load was at race with the page load event, which caused flakiness
in tests that verified, at page load completion time, the number of evaluated
loads recorded into histograms.

BUG=677370

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

Changing directly the URL may have side effects beside hiding the
scheme.
Move the formatting to GetFormattedURL.
Also refactor ToolbarModelIOS to inherit ToolbarModel.

BUG=674986

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

Move existing description of HarfBuzzShaper internals into shaping
section, add outline of other essential text stack functionality.

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

Test times out randomly on device.

BUG=493427

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

Instead of blanket activation, bring the tests closer to reality by setting the
subresource filtering in `activation list` mode, and bringing up a
TestSafeBrowsingService, which allows the tests to artificially mark URLs as
blacklisted, and thus allows the subresource filtering logic to receive an
activation signal when these URLs are navigated to in the main frame.

The CL also changes the test to trigger same-page navigations from the renderer
side, to prevent WebContentsObservers seeing duplicate (nested) navigations
when browser-side navigation is enabled.

BUG=637415

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

This UMA and code is misleading because the shared worker ref count in
RenderProcessHostImpl actually is not the number of running shared
workers. We've added comments since that clarify it.

If we want this UMA back we can add a new one later. I'll add current
stats to the bug.

BUG=403258

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

At Opera we test our SDK on an old MIPS device with a
peculiar toolchain. To be able to compile ImageDocument.o,
we need to call std::round(), not just round().

BUG=none

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

- Switching of BVC is not always immediate. Add a Wait condition.
- FastSwipe gesture sometimes fail to switch mode. Use SlowSwipe.

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

Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/24348df..3ae6916

$ git log 24348df..3ae6916 --date=short --no-merges --format=%ad %ae %s
2016-12-29 kjellander@webrtc.org Add iOS64 simulator bot running on iOS 9.0 simulator.
2016-12-28 zhihuang@webrtc.org Prepare to introduce the IceTransportInternal.
2016-12-28 kjellander@webrtc.org Revert of Disable flaky test VideoProcessorIntegrationTest.ProcessNoLossChangeFrameRateFrameDropVP9 (patchset #1 id:1 of https://codereview.webrtc.org/2568743007/ )
2016-12-28 sprang@webrtc.org Add frame rate throttling to vp8 screenshare_layers.
2016-12-28 mbonadei@webrtc.org Refactor bitrate_controller, remote_bitrate_estimator and congestion_contoller for gn check.

TBR=kjellander
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng
BUG=

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

This has the side-effect of using the new GPU integration test harness
for these tests.

BUG=352807, 676701
CQ_INCLUDE_TRYBOTS=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=zmo@chromium.org

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

[ToolsMenuViewToolsCell imageForImageId:] checked that id was not-zero.
Current implementation DCHECKS.
Add tests.
This is a follow up of https://codereview.chromium.org/2602903002/

TBR=marq

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

Cr-Commit-Position: refs/heads/master@{#440945}

BUG=677275
TEST=manual

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

BUG=

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

BUG=621697

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

BUG=579229
CQ_INCLUDE_TRYBOTS=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

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

- adds options to automatically duck/unduck, suspend/resume audio based on tts events.

BUG=621697
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

BUG=none

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

BUG=579229

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

- os.path is provided by input_api, and it's preferred to use it
- time does not seem to be used any more
- input_api.change.AffectedFiles() with a "starts with my dir" filter
  is equivalent to input_api.AffectedFiles(); also, harness file_filter=
- os.path.split()[1] is equivalent to os.path.basename(), which is more
  readable, IMO

R=calamity@chromium.org
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

Include quic_str_cat.h in all the places it's used.

Merge internal change: 142874387
https://codereview.chromium.org/2609483002/

QUIC - refactor in preparation for two streams per HTTP transaction.

The IETF WG is moving to model of using a pair of streams per

https://quicwg.github.io/base-drafts/draft-ietf-quic-http.html#rfc.section.4.2

This scheme above came about in the course of discussions on QUIC
specific changes to HPACK that will hopefully eliminate HPACK induced
HoL blocking.

This change is a refactor of QuicHeadersStream and QuicSpdySession,
moving per-session granularity items from the former to the latter.  It
does not yet make headers stream per request, but this helps prepare the way.

na/ (refactor only - no functional change).

Merge internal change: 142825457
https://codereview.chromium.org/2607873002/

BUG=

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

https://chromium.googlesource.com/chromium/src/+/009b4fbe8df40

BUG=674098
TBR=chrishtr@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#440934}

CertificateReportingService::ShutDown posts a task to the IO thread to
delete reporter_. If the service's destructor is already run, this will
cause a crash during shutdown. Instead, delete reporter_ using a static
method and without referencing CertificateReportingService's |this|.

BUG=677329

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

https://skia.googlesource.com/skia.git/+log/e02d3caab823..a4cb388c0fe8

$ git log e02d3caab..a4cb388c0 --date=short --no-merges --format='%ad %ae %s'
2016-12-29 brianosman Revert "Add ImageToColorSpace helper in SkImageFilter"

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.blink:linux_trusty_blink_rel
TBR=liyuqian@google.com

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

New code should be using HttpRequestHeaders or HttpResponseHeaders,
as appropriate.

BUG=none

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

BUG=579229

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

Before this patch and the one that was the proximate cause of issue 674098,
clips for composited bounds worked like this:
1. If there was a clip between the compositing container and a PaintLayer, the
clip would be that PaintLayer's contribution to composited bounds.

2.  If there was no such clip, we would fall back to physicalBoundingBox, and
also recurse into children.

After the patch causing issue 674098, we would always recurse, even in case 1.

This patch observes that we can always use a local bound of the intersection of
any clip with physicalBoundingBox. Recursion still always happens.

This avoids situations like in issue 674098 where skew matrices yielded bounds
much too large.

BUG=674098
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

They fail due to subpixel differences.

TBR=wkorman

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

BUG=

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

BUG=662859

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

BUG=645957
TEST=PaintLayerScrollableAreaTest.OnlyAutoClippedScrollingContentsLayerPromoted
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

https://skia.googlesource.com/skia.git/+log/7cf774573cc5..e02d3caab823

$ git log 7cf774573..e02d3caab --date=short --no-merges --format='%ad %ae %s'
2016-12-28 brianosman Add ImageToColorSpace helper in SkImageFilter

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.blink:linux_trusty_blink_rel
TBR=liyuqian@google.com

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

PlzNavigate: cancel any scheduled navigations when deciding to let the embedder handle a navigation.

This fixes the following layout tests in PlzNavigate mode:
imported/wpt/html/browsers/browsing-the-web/navigating-across-documents/005.html
imported/wpt/html/browsers/browsing-the-web/navigating-across-documents/006.html
imported/wpt/html/browsers/browsing-the-web/navigating-across-documents/007.html

BUG=673751

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

BUG=599523

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

BUG=579229

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

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

This patch introduces a list control, which maintains an array-like model. This makes
it possible to update model efficiently (not in O(n) all the time).

It also improves fast smooth scrolling of viewport on mac (less flashes)
by using gap elements instead of absolute positioning.

Only supports fixed/measured height elements, variable height is coming next (to be used in treeoutline).

BUG=none

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

On Android, we restrict who can open incognito tabs to only
Chrome.  The logic to enforce this was spread across
IntentHandler#shouldIgnoreIntent and
ChromeTabbedActivity#processUrlViewIntent.  It was unclear and
confusing to read, so this removes the invalid paths and makes
it more explicit about what is allowed.

BUG=

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