GitLab

path() is being deprecated, GetPath() has better checking against wrong use.

For more context, see https://codereview.chromium.org/2275553005/.

BUG=640599

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

After https://codereview.chromium.org/2317203002 and
https://build.chromium.org/p/chromium.fyi/builders/Browser%20Side%20Navigation%20Linux/builds/21220

BUG=607205,575210
TBR=clamy@chromium.org

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

This CL adds a browser test for desktop geolocation where persistence is
toggled off, ensuring that the underlying content setting isn't changed.
Additional checks of the content setting are added to the existing grant
and block tests.

This CL also simplifies the feature list setting used in the
PermissionContextBase test.

BUG=632269

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

A third go at https://crrev.com/3c64df1fc98aa06eabfc18d1f5c2f2b0aec1a658

Although I still can't reproduce the clusterfuzz reports locally I'm confident
this will cure the specific crashes because I'm no longer transforming the
first line's text unless it has a distinct first-line style (:/).

BUG=129669,644733

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

If it's an exclusive end offset, we need to know this, so that we map to the
former column set, rather than the latter, if the offset is at a column set
boundary.

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

The spec (https://drafts.csswg.org/css-tables-3/#repeated-headers) tells us that
we shouldn't bother repeating headers if a row of content doesn't fit. It's not
clear what to do in the situation where it's just the first page that the row
doesn't fit. I think we ultimately want to do the same as Edge/IE and still
repeat the headers on subsequent pages.

That would require a second layout pass to achieve so for now just drop the
repeating headers if we can't fit a row on the first page.

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

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

IFrame Scrollbar needs reconstruction. In the updateWidgetGeometry set
frameview needs layout if frameview scrollbars needsreconstruction.

BUG=492785

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

BUG=604851

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

This CL replaces EventSender in HTMLLinkElement with postTask():
Instead of calling HTMLLinkElement::dispatchPendingLoadEvents() in
Document::implicitClose() to enforce <link>'s onload events to be executed
before the document's onload event, this CL blocks document's onload until
<link>'s onload event using IncrementLoadEventDelayCount.

This CL also modifies SimTest:
This CL potentially makes onload events of <link> and documents async and
delayed, and thus SimTest can be destructed before document's onload event,
which cause assertion failure.
This CL calls testing::runPendingTasks() in ~SimTest() to flush scheduled
onload events to make Document to be loaded before SimTest's destruction.

BUG=624697

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

Make cc::SoftwareImageDecodeController, cc::GpuImageDecodeController, cc::ResourcePool, and cc::StagingBufferPool MemoryCoordinatorClient.

- implement OnMemoryStateChange for background renderer's purge +
  suspend. So the method reduces memory only when state == SUSPENDED.
- intent-to-implement-and-ship of background renderer's purge + suspend is
  https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/DK189tnM8l4
- one of the documents attached in the above intent is
  https://docs.google.com/document/d/1EgLimgxWK5DGhptnNVbEGSvVn6Q609ZJaBkLjEPRJvI/edit?usp=sharing

BUG=635419
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

There are a large number of mechanical changes of usages of WrapUnique()
to MakeUnique(). The blame information for these is not useful as it
does not reflect the people responsible for the code.

Add the relevant commits to .git-blame-ignore-revs so that they will be
skipped by git hyper-blame.

R=mgiuca@chromium.org
BUG=637812

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

It takes 1-3 seconds on Release and ~10 seconds on Debug.

TBR=brucedawson@chromium.org
BUG=647192
NOTRY=true

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

This is a refactoring patch which stores the definite/indefinite height
detection in a new attribute m_hasDefiniteLogicalHeight in LayoutGrid.
That way we just only call LayoutBlock::hasDefiniteLogicalHeight() once,
from LayoutGrid::layoutBlock(). Then in LayoutGrid::gridTrackSize()
we reuse the cached value.

No new tests, no change of behavior.

BUG=624301

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

This test consistently takes 2 seconds on Release and 10 seconds on
Debug, thus it meets the bar of Slow tests.

TBR=brucedawson@chromium.org
BUG=647192
NOTRY=true

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

This cl splits the big CL with changes to net/ folder and
has changes only to net/nqe and some url_request tests
according to the changes to URLRequest API.

What the big cl does:
It modifies net/ clients that use URLRequest API as long as
URLRequest::Read returns int net errors and
URLRequest::Delegate and NetworkDelegate methods
(for example, OnResponseStarted or OnCompleted) have int
net_error in the arguments now.

The reason behind splitting the CL into small one is that
an android bot started to be unstable and unittests became
flaky. It was not possible to locate the problem and the
decision was to split the CL and upload small parts with a
6+ hours interval in order to make it possible to locate
the problematic code.

The big CL is located here -
https://codereview.chromium.org/2265873002/

BUG=423484

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

This CL introduces seven new fuzzers for WebAssembly which test the
decoding of single sections in the WebAssembly module decoder.

R=mmoroz@chromium.org

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

This CL renames '-webkit-clip-path' to 'clip-path', and makes the former
an alias of the latter.
For LayoutSVGRoot, clip-path is now applied only by the PaintLayer and
not by SVGPaintContext.

Intent-to-ship:

https://groups.google.com/a/chromium.org/d/topic/blink-dev/FBE05hzCmPo

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

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

This patch adds UMA to measure time to open hyphenation dictionary.

The previous CL[1] added UMA that includes IPC as "Hyphenation.Open",
but this metric measures the time of the file task without the IPC.

[1] 2113933003

BUG=605840

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

This exercises webview state save/restore, and also makes using the
shell as a testing/demonstration tool easier, because the current URL
is retained and reloaded when you switch webview providers.

BUG=
NOPRESUBMIT=true

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

TransparentButton has state-change animations enabled despite being invisible,
which causes a bunch of spurious repaints of the combobox on mouse enter/exit.

BUG=645266

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

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/5148bd4b2be6..3cc40b373729

$ git log 5148bd4b2..3cc40b373 --date=short --no-merges --format='%ad %ae %s'
2016-09-15 perezju Revert of "[telemetry] Wire TsProxy through telemetry's network stack." (patchset #1 id:1 of https://codereview.chromium.org/2335403003/ )

BUG=639632

TBR=catapult-sheriff@chromium.org

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

There's no longer any WebSocket files under content/common/.

R=yhirano@chromium.org,jochen@chromium.org
BUG=none

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

Previously, internals.isPreloaded() depends on |ResourceFetcher::m_preloads|,
which is cleared by ResourceFetcher::clearPreloads().
This caused internals.isPreloaded() to turn false after around document's
load event.

This CL makes ResourceFetcher to keep a set of preloaded URLs (separately
from |m_preloads|) if a blink::Internals object is created, and use the set
to calculate internals.isPreloaded().

BUG=643621

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

This patch converts "editing/inserting/insert-at-end-0[12].html" to use w3c test
harness to simplify test script, and consolidates into "insert_at_end.html"
since these scripts use similar sample, for improving code health.

BUG=n/a
TEST=n/a; no behavior changes

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

Some EGL drivers do not support binding one component textures to an
EGL image. One component textures are used for YUV video frames, so
to support those drivers in WebView, we need to do YUV to RGB
conversion in software, and then upload an RGB texture instead of a
set of 3 Y,U,V textures.

BUG=579060, 632461
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

The Android policy cache was originally created to avoid Strict Mode
violations, but doesn't do so in Webview. Actually, although reading
the app restrictions on the UI thread is a strict mode violation it
is unlikely to cause UI visible delays, As such the solution is to
simply to allow file reads in this component.

BUG=628627

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

BUG=498716

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

Makes indexed properties return undefined if the index is out of range.

BUG=229750

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

Revert of Implement QuicHttpStream::GetLoadTimingInfo (patchset #7 id:200001 of https://codereview.chromium.org/2324183002/ )

Reason for revert:
URLRequestQuicTest.TestTwoRequests is flaky, see https://crbug.com/647071.

Original issue's description:
> Implement QuicHttpStream::GetLoadTimingInfo
>
> This CL implements QuicHttpStream::GetLoadTimingInfo.
>
> |dns_start| and |dns_end| are obtained in
> QuicStreamFactory::Job::DoResolveHost and
> DoResolveHostComplete. These two values are passed to
> QuicChromiumClientSession's constructor.
>
> |connect_start| and |connect_end| are obtained in
> QuicChromiumClientSession::CrytoConnect() and when
> handshake is confirmed.
>
> |ssl_start| and |ssl_end| are the same as |connect_start|
> and |connect_end| because QUIC always does encryption.
>
> If a session is reused, the connect_timing fields should
> all be null. This CL also sets
> |socket_reused| field of LoadTimingInfo accordingly.
>
> BUG=637051
>
> Committed: https://crrev.com/5608fc1bd4f0a80b6d1996f3f4059f2efa0a0777
> Committed: https://crrev.com/83ac2f22700c563fef8a670ad4c5c57d34032e03
> Cr-Original-Commit-Position: refs/heads/master@{#417969}
> Cr-Commit-Position: refs/heads/master@{#418567}

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

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

BUG=

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

path() is being deprecated, GetPath() has better checking against wrong use.

For more context, see https://codereview.chromium.org/2275553005/.

BUG=640599

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

This patch converts "editing/inserting/insert-div-02[34].html" to use w3c test
harness to simplify test script, and consolidates into
"insert_paragraph_and_style.html" since these scripts were committed in same
patch, for improving code health.

BUG=n/a
TEST=n/a; no behavior changes

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

Refactor disk_mount_manager_unittest by replacing gmock with a dedicated mock class of DiskMountManagerObserver.

This change is based on
https://codereview.chromium.org/2333983004/
which was once merged but reverted by
https://codereview.chromium.org/2343593002/

BUG=641943
TEST=chromeos_unittest build with is_asan and is_lsan = true in gn args, and run with ASAN_OPTIONS="detect_leaks=1 symbolize=1" environment variable.

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

PR for DOM Standard is: https://github.com/whatwg/dom/pull/329

Note: Blink's IDL uses [Unscopeable], instead of [Unscopable].
I have filed a bug: https://bugs.chromium.org/p/chromium/issues/detail?id=647133.

Let me omit to add a layout test. It should be covered in WPT.

BUG=643869

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

BUG=636710

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

BUG=617056
TBR=rdevlin.cronin@chromium.org

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

This CL fixes an issue where the CBD button would stay grey after the
CBD button was activated via spacebar.

BUG=640894
NO_DEPENDENCY_CHECKS=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

I have mistyped the mixin name. Sorry.

BUG=645295
TBR=dpranke@chromium.org

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

It seems that for "group" labels GN does not include data of dependencies
declared as "deps", but it would be convenient if we could refer to
chromium_builder_perf group to query all the runtime dependencies of the
test binaries.  This change does not touch dependency logic, though: all
the prerequisites of chromium_builder_perf continue to build as before.

The runtime_deps of this target can then be used, e.g., to make archive
with all the files required to run all (telemetry-based and binary)
performance tests.

R=jochen,thakis

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

path() is being deprecated, GetPath() has better checking against wrong use.

For more context, see https://codereview.chromium.org/2275553005/.

BUG=640599

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