GitLab

BUG=none

Review URL: https://codereview.chromium.org/983413005

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

This brings in:
1e580a1 Fix multiply defined macro in zlib.h on linux_chromium_clobber_rel_ng builder.
0700106 Make conversions for CPDF_Link explicit.
8842c62 Fix the pdfium component build.
5a370d7 Make conversion between CPDF_Dest and its object explicit.
7348fae Add gmock to .gitignore
624b6c1 Make conversion between CPDF_Action and its dictionary explicit.
61c25ee Merge to Master: Fix a compiling error in fxjs_v8.h
e1671bd Kill off JS_ErrorString type.
b720d0a Return error information from pdfium to JS.
944ccad Do hit tests against Annots in reverse order.
5b47dbf Remove unused IDS_ constants from pdfium.
3d25502 Pull in gmock for standalone pdfium builds.
ec61a85 Upgrade openjpeg to revision 2997.
42a8ada Implement a delegate for EmbedderTests.
57e4c86 Kill off unused IFXJS interface methods.
4351ac2 Fix compilation error on Windows.
f9cfdda Add small flate decode unit test.
e796478 Kill multi-byte comments in fxet_list.h
23f9288 Convert pdfium expected test results to PNG format.
0fd53cf Fix GN build after PNG support backport
ef1e85c Backport PNG output format to origin/master branch.

BUG=445408,457493,PDFIUM:123

Review URL: https://codereview.chromium.org/989213003

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

$ git log 1b7bfeca3..b61a1802f --date=short --format='%ad %ae %s' | sed 's/@chromium\.org//'
2015-03-09 maruel Ignore command from included .isolate when the current one define a command.
2015-02-26 maruel Reduce ereporter2 spam on file locking on Windows.

R=vadimsh@chromium.org
BUG=440882

Review URL: https://codereview.chromium.org/986423002

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

This adds better tracking of free discardable memory in child
processes and the ability to release the memory without having
the browser process purge the memory.

BUG=460995
TEST=content_unittests --gtest_filter=DiscardableSharedMemoryHeapTest.*

Review URL: https://codereview.chromium.org/981403002

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

When the render widget is hidden, the hang monitor timeout is explicitly
stopped. However, tasks executed *after* the widget hide signal may
trigger a restart of the hang monitor timeout, e.g., queued input
events. Prevent such restarts when the widget is hidden while
also ensuring the hang monitor restarts when the widget is shown again.

BUG=458594

Review URL: https://codereview.chromium.org/938123003

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

Revert of [Andorid] ClipDrawable progress bar experiment. (patchset #2 id:20001 of https://codereview.chromium.org/960733002/)

Reason for revert:
Although ClipDrawableProgressBar showed some perf improvement on blink_perf.svg, it didn't show any meaningful difference on page_cycler.top_10_mobile test.

Original issue's description:
> [Andorid] ClipDrawable progress bar experiment.
>
> Recently it was discovered that ClipDrawable is much faster than our
> default ToolbarProgressBar on content shell.
>
> So we plan to experiment with ClipDrawable based progress bar
> implementation.
>
> BUG=455891
>
> Committed: https://crrev.com/b13144e2f2b4ac80f81cee8ce4d4c7f4831b625e
> Cr-Commit-Position: refs/heads/master@{#318292}

TBR=dtrainor@chromium.org,jaekyun@chromium.org
NOPRESUBMIT=false
NOTREECHECKS=false
NOTRY=false
BUG=455891

Review URL: https://codereview.chromium.org/994753002

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

BUG=417532

Review URL: https://codereview.chromium.org/983963002

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

Previously, the websocket was disconnected in __del__, whose timing was
non-deterministic. Now the websocket is disconnected as soon as its owner
(typically devtools_client_backend) no longer wants to keep track of it.

BUG=461153

Review URL: https://codereview.chromium.org/980043003

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

Revert of Add policy_templates target for GN. (patchset #4 id:60001 of https://codereview.chromium.org/957283003/)

Reason for revert:
Possibly broke some GN builds (reverted with https://codereview.chromium.org/865573002), e.g.:

http://build.chromium.org/p/chromium.linux/builders/Linux%20GN/builds/24483/steps/compile/logs/stdio
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20GN/builds/11092/steps/compile/logs/stdio

ninja: Entering directory `/mnt/data/b/build/slave/Linux_GN/build/src/out/Release'
ninja:error: expected depfile 'gen/chrome/policy_templates.d' to mention 'gen/chrome/app/policy/common/html/am/chrome_policy_list.html', got 'gen/chrome/app/policy/linux/examples/chrome.json'

Original issue's description:
> Add policy_templates target for GN.
>
> BUG=462362
>
> Committed: https://crrev.com/982eb90bf74535cbe5e21c4eacb759c42ff424d4
> Cr-Commit-Position: refs/heads/master@{#319759}

TBR=pneubeck@chromium.org,bauerb@chromium.org,knn@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=462362

Review URL: https://codereview.chromium.org/992863002

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

Revert of Add i18n support for Android App Restrictions. (patchset #7 id:120001 of https://codereview.chromium.org/865573002/)

Reason for revert:
Broke some GN builds, e.g.:

http://build.chromium.org/p/chromium.linux/builders/Linux%20GN/builds/24483/steps/compile/logs/stdio
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20GN/builds/11092/steps/compile/logs/stdio

ninja: Entering directory `/mnt/data/b/build/slave/Linux_GN/build/src/out/Release'
ninja:error: expected depfile 'gen/chrome/policy_templates.d' to mention 'gen/chrome/app/policy/common/html/am/chrome_policy_list.html', got 'gen/chrome/app/policy/linux/examples/chrome.json'

Original issue's description:
> Add i18n support for Android App Restrictions.
>
> GYP target policy_templates is now enabled for android as well.
> Remove code that generated Policy resources w/o translation for Android.
>
> roll grit 186:187
>
> 187: Add a Policy template writer that generates Android resources which can be exposed through Android's App Restriction Schema.
>
> Depends on:
> https://codereview.chromium.org/865563002
>
> BUG=450548
>
> Committed: https://crrev.com/cd9f7d3afef0d0d4054318e9999aee879eaf26e1
> Cr-Commit-Position: refs/heads/master@{#319758}

TBR=bauerb@chromium.org,bartfab@chromium.org,knn@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=450548

Review URL: https://codereview.chromium.org/992853002

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

This is a target I converted to GN and it is known to run fine.

Tested on Linux with the following command lines:

$ gn gen out-gn
$ ninja -C out-gn device_unittests
$ out-gn/device_unittests

BUG=None
TEST=see above
R=dpranke@chromium.org
TBR=brettw@chromium.org

Review URL: https://codereview.chromium.org/990653002

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

Those filters are generic to non-Android ARM platforms, and should be
available to all products.

BUG=

Review URL: https://codereview.chromium.org/986673003

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

Removing [Reset tracking data] was approved by jar@. It was never fully implemented (only for the browser process) and is superseded by taking and comparing snapshots in chrome://profiler tab.

BUG=456354

Review URL: https://codereview.chromium.org/980333004

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

Start upstreaming Layout/LayoutManager files.

BUG=428821

Review URL: https://codereview.chromium.org/987883002

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

ChildProcessConnectionImpl.mAlwaysInForeground is used
for the GPU process and indicates that it does not participate
in hiding like renderers. The 'important' binding gives it the
same priority as the main application, so upon termination we
should just use the app's foreground status to determine whether
the GPU process was in the foreground or not.

BUG=462422

Review URL: https://codereview.chromium.org/987193003

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

Advertise "h2" token with ALPN and accept "h2" token in NPN.  This is the final,
almost-published version of HTTP/2.

Add "h2" to relevant tests.

Also add placeholder values (in comment) for drafts 16 and 17.  They might be
used for testing, or for collecting statistics about servers in the wild (not to
be released on Stable channel though).

BUG=401166

Review URL: https://codereview.chromium.org/987123002

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

Having 'using' declarations in header files polutes the global
namespace and that is not what we want.

BUG=438860
R=piman@chromium.org

Review URL: https://codereview.chromium.org/972713002

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

Tested with the following command lines on Linux:

$ gn gen out-gn
$ ninja -C out-gn native
$ ninja -C out-gn gpu/gles2_conform_support/native
$ gn check out-gn //gpu/gles2_conform_support/native/*

BUG=432959
TEST=see above
R=dpranke@chromium.org,sievers@chromium.org

Review URL: https://codereview.chromium.org/992463002

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

Use linked hash map for broken alternative service queue in
HttpServerPropertiesImpl.  This container has efficient lookup by key, allowing
us to use it as the definitive set of broken alternative services in an upcoming
CL.

Also rename broken_alternate_protocol_map_ to
recently_broken_alternative_services_ for consistency and because it makes more
sense.

BUG=392576

Review URL: https://codereview.chromium.org/989253005

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

BUG=392576

Review URL: https://codereview.chromium.org/986413003

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

Some old devices are getting lollipop update without updated drivers.

BUG=

Review URL: https://codereview.chromium.org/982153002

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

Because for each overscroll session, OverscrollStarted() and
OverscrollCompleted() (in TouchEditableImplAura) are not necessarily
called the same number of times, TouchEditableImplAura could be stuck in
a state of thinking that there is always an scroll in progress which
prevents handles from appearing after a long-press.

BUG=430176

Review URL: https://codereview.chromium.org/963103003

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

Revert of [Sync] Refactor AttachmentStore classes. Introduce concept of referrer. (patchset #2 id:20001 of https://codereview.chromium.org/986743004/)

Reason for revert:
I think this CL is responsible for tree closing compile failure https://build.chromium.org/p/chromium.mac/builders/iOS_Device/builds/26672/steps/compile/logs/stdio.

Original issue's description:
> [Sync] Refactor AttachmentStore classes. Introduce concept of referrer.
>
> In this change:
> - Move towards following class hierarchy: http://www.plantuml.com:80/plantuml/png/ZPBTJiCm38NlynIvv4TzWsaI525nCI5jkw-on1EZQHmvwQG9U7TC4-ZeKiITvNm-EiSExbv1Hxc6VOszYs24mDHQeG6xFNcGRqBAknYLVkd0nKr40l7nmqrU1deDeRUHYrfPkrEw3LmJx848YCkWqODfkECZBIOAZuHin9abWxUo9b0Hdjt38RGJyEhwZ4YZI9kJq_mmRp0pAPKnd7pGgG8t6usTHyVe7_FPtY3mbOth9ghGDjGxTnwlaEq-ySjv-KmCwZflxvVyE5bSIa4Mw7ZGyDHvAyHurPkgkhZgz9PuoNp75tDhAUZcJ68cwkATHyfXnFWn4_PFDsKuNLuKLrFodGS-0G00
> - Remove AttachmenService::GetStore. Now attachment store is owned by
>   model type, there is no need to get it from AttachmentService
> - Introduce AttachmentReferrer. There is no functionality behind it yet
>   and interface is not complete for it. These will come in the next change.
>
> BUG=457735
> R=maniscalco@chromium.org
> TEST=Not exposed in chrome. Only unit tests available (sync_unit_tests)
>
> Committed: https://crrev.com/6adcada8a799057c31c0f17550c9e2747a8df847
> Cr-Commit-Position: refs/heads/master@{#319733}

TBR=maniscalco@chromium.org,cjhopman@chromium.org,pavely@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=457735

Review URL: https://codereview.chromium.org/996473005

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

BUG=462362

Review URL: https://codereview.chromium.org/957283003

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

GYP target policy_templates is now enabled for android as well.
Remove code that generated Policy resources w/o translation for Android.

roll grit 186:187

187: Add a Policy template writer that generates Android resources which can be exposed through Android's App Restriction Schema.

Depends on:
https://codereview.chromium.org/865563002

BUG=450548

Review URL: https://codereview.chromium.org/865573002

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

All clients have switched to the new method.

BUG=461148

Review URL: https://codereview.chromium.org/996493002

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

The test actually didn't test anything. Calling sock1.reset() would cause the
socket to disconnect before tlslite completed the handshake. tlslite doesn't
insert the session into the session cache.

This is a preparatory CL to get the tests in shape before reworking the session
cache implementation.

BUG=454044

Review URL: https://codereview.chromium.org/988223004

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

Each profile creates new instance of SignedInDevicesManager so, for example,
opening incognito window and then closing it, destroyes an instance of
SignedInDevicesManager created for this profile. The code has forgot to remove
itself from the list of observers in EventRouter.
That caused crash when that event was triggered later in a normal profile as
EventRouter would reference stale pointer.

In tests observer is not added and profile_ is null so null-checking it.

R=rdevlin.cronin@chromium.org
BUG=

Review URL: https://codereview.chromium.org/980663002

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

I'm proposing to add meacer to:
chrome/browser/interstitials/OWNERS
chrome/browser/ssl/OWNERS

For his work on captive portals and general code cleanup.

BUG=
R=rsleevi@chromium.org

Review URL: https://codereview.chromium.org/982343002

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

This CL also adds a generic persistence mechanism for //net, separate
from Chrome user preferences, that loads asynchronously on startup (i.e.
doesn't block startup).

This is a reupload of rdsmith@'s existing CL at
https://codereview.chromium.org/881413003/; see that CL for earlier discussion.

BUG=374915
R=mmenke@chromium.org
R=rdsmith@chromium.org
R=bauerb@chromium.org

Review URL: https://codereview.chromium.org/901303002

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

Field trial "CreditCardScan" enables scanning credit cards. "Credit card
scanning" flag can force enable or disable the feature.

BUG=462381

Review URL: https://codereview.chromium.org/991983003

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

Prefer DCHECK_EQ(a, b) over DCHECK(a == b) when possible as this
provides more data on a failure. Similar for other operators such as
>=, <, !=, etc.

This also applies to CHECK, EXPECT, and ASSERT macros.

R=Nico
BUG=464816

Review URL: https://codereview.chromium.org/985723003

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

When a ConnectJob failed and there was another socket request pending
for the same group, a new ConnectJob would always be created, even if
the pending request already had its own ConnectJob.

BUG=463960

Review URL: https://codereview.chromium.org/981643002

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

Previously, the historic network stats values on the net-internals
page were only updated every hour. Now, instead of using local
prefs, HistoricNetworkStatsInfoToValue reads from statistics prefs,
and the values are up to date.

BUG=403867

Review URL: https://codereview.chromium.org/989663002

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

Make the chrome://extensions page use chrome.developerPrivate API for
'inspect' calls.
Also convert the api function to a UIThreadExtensionFunction.

BUG=461039

Review URL: https://codereview.chromium.org/989843002

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

New component extensions should default to the front of the visible icons. The
user can still reposition them, but for the first startup, they will now appear
at the beginning.

BUG=463938

Review URL: https://codereview.chromium.org/981603002

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

between what Chrome thinks and what the driver is.

This is caused by we intercept invalid program and generate an error on
LinkProgram rather than passing it to the driver, so the driver still have
a valid program if the previous link succeeds.

BUG=463599
TEST=test case in the bug
R=sievers@chromium.org

Review URL: https://codereview.chromium.org/978193003

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

BUG=454983

Review URL: https://codereview.chromium.org/993703002

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/719867b..796d284

TBR=jbroman@chromium.org,dstockwell@chromium.org

Review URL: https://codereview.chromium.org/993683002

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

Add a map from stream ID to priority to track which stream IDs are already in
the write blocked list.  And git cl format.

This CL lands server change 86595615 by birenroy.

BUG=

Review URL: https://codereview.chromium.org/989523005

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