GitLab

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/68ecb73..a537ab2

TBR=rtoy@chromium.org

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

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

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

BUG=466315

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

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

BUG=452403
TEST=content_unittests --gtest_filter=ServiceWorkerDispatcherTest.*

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

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

BUG=467004
TBR=jonross@chromium.org
NOTRY=true

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

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

The previous one wasn't available for Win32 builds.

BUG=
TBR=stgao@chromium.org

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

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

BUG=None

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

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

BUG=448263
TEST=none

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

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

Do not re-add alternate protocol from canonical host to origin in
HttpServerPropertiesImpl::SetBrokenAlternateProtocol(), because brokenness is
recorded independently of the alternate protocol mapping.  I have overlooked
this detail in
https://crrev.com/997953003/diff/80001/net/http/http_server_properties_impl.cc.

BUG=392576

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

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

BUG=

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

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

The return statement should never be reached due to the LOG(FATAL), but
some compilers seem to think that it is possible to reach the end of
this function.

TBR=jln@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/f7fd532..68ecb73

TBR=rtoy@chromium.org

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

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

step, find out how often opens and creation fail.

Also refactored casing of an enum.

BUG=461654

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

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

BUG=462467
TEST=Run VDA test.

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

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

iOS does not use //components/feedback to report feedback so remove
the unused dependencies.

BUG=None

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

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

Revert of Revert of Disable a test that fails under ASan and MSan (patchset #1 id:1 of https://codereview.chromium.org/1018443002/)

Reason for revert:
The underlying issue is still not fixed, re-enabling the test causes bots to turn red.

Please wait for the underlying issue to be resolved before re-enabling.

Original issue's description:
> Revert of Disable a test that fails under ASan and MSan (patchset #1 id:1 of https://codereview.chromium.org/1007293002/)
>
> Reason for revert:
> This test is a victim of a bigger issue of main renderer message loop being used after it is deleted. Details are in crbug.com/467369.
>
> I've also run the test with ASAN locally and didn't reproduce after a few runs.
>
> Original issue's description:
> > Disable a test that fails under ASan and MSan
> >
> > It can be flaky in other configurations as well
> >
> > BUG=467483
> > TBR=nasko
> >
> > Committed: https://crrev.com/992ff715f61e90562983139ac31392b72e716822
> > Cr-Commit-Position: refs/heads/master@{#320713}
>
> TBR=timurrrr@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=467483
>
> Committed: https://crrev.com/216981c97018a8b8dea097cbde58889bf4e68c1a
> Cr-Commit-Position: refs/heads/master@{#320755}

TBR=nasko@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=467483

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

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

On a fresh install of Ubuntu Trusty + Cinnamon, install-build-deps.sh failed, because:

* it was looking for libgl1-mesa-dev
* Cinnamon forced libgl1-mesa-dev-lts-utopic to be installed
* there cannot be two versions of libgl1-mesa-dev* installed

This CL adds libgl1-mesa-dev and mesa-common-dev to the group of packages, for which the correct "mesa_variant" suffix is guessed. It also adds "-lts-utopic" to the list of explored suffixes.

BUG=466787

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

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

BUG=467072

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

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

This CL also changes ownership of the TickClock object used by the AffiliationFetchThrottler, so that it is now owned by the AffiliationBackend.

Furthermore, this CL makes the AffiliationBackend no longer rely on using ThreadTaskRunnerHandle to schedule asynchronous operations, but instead take a SingleThreadTaskRunner to use in its constructor. This allows delayed tasks (which are now present) to be posted to the correct task runner in tests too (namely, in AffiliationServiceTests).

BUG=437865

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

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

This is a straight upstreaming of code that is currently used
on iOS.

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

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

This CL introduces ServiceWorkerMessageSender that is a thin wrapper
of ThreadSafeSender so that unittests can inject a mock sender, and
adds some unittests related to reference counting using the mock.

BUG=452403
TEST=content_unittests --gtest_filter=ServiceWorkerDispatcherTest.*

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/2fbbb1a..f7fd532

TBR=mkwst@chromium.org,rtoy@chromium.org

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

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

These tests will be enabled again once the AppRTC app is working again which we expect will happen rather soon.

TBR=phoglund
BUG=467471

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

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

To pass the code with the closure compiler, this patch reformats the code, adds the annotation, and fixes some of the minor bugs.

R=fukino@chromium.org
BUG=448263
TEST=browsertest passes

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

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

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

BUG=464051

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

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

BUG=chromium:442831

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

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

Also adds SetCapabilities and HasCapability functions for more fine-grained control over capabilities.
BUG=

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

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

TEST=Not used yet, but used in an upcoming CL.
BUG=440674

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

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

BUG=None
TEST=Verified on linux_chromeos.
TBR=nona@chromium.org

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

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

BUG=412254

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

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

This field is used when deciding whether to show the NTP promo.

BUG=436681

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

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

Entry.cachedUrl is not standard property and the closure compiler detected it as an error. To prevent the error, we need to access it with dictionary-way (using brackets).

BUG=448263
TEST=none

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

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

Earlier left and right navigation key's are getting used without check
for modifiers.
Check added for alt,ctrl,meta,shift key, when we have modifer then default
handling will happen that will move it to previous url with alt+left arrow,
and when we have just left or right arrow keys then cookies list specific handling will happen.

BUG=467476

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

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

In crbug.com/466196, the gpu_times benchmark unintentionally filters out skip
value, and made it hard to figure out what went wrong. When people do whitelist
filtering, most of them will also make the mistake of filtering our skip &
failure values unintentionally. I also think that these values shouldn't be
filtered out in any situation.

BUG=466196

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

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

BUG=413541
TEST=manually tested

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

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

quic_session_test by using a StrictMock, and adding the necessary
EXPECT_CALL()s.

Merge internal change: 83469149

R=rch@chromium.org

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

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

R=stevenjb@chromium.org, jlklein@chromium.org

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

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

According to the issue (crbug.com/437293), the root issue is solved. Then we need to re-enable the disabled tests. This patch does it.

BUG=437293
TEST=browsertest passes
R=hirono@chromium.org, yawano@chromium.org

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

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

BUG=465575
TEST=browser_test passes

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

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