GitLab

When the "--disable-background-networking" command-line switch is
specified, extensions updates shouldn't be downloaded.

BUG=489244

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

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

BUG=None
R=jam@chromium.org

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

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

This is because they have an unowned pointer to cc::SurfaceManager.

Also ensure we don't have a double delete of display_ in DisplayImpl's destructor. This occurred because when display_ was deleted, it would end up calling back into  OutputSurfaceLost() which would attempt to delete display__ again.

This is split off https://codereview.chromium.org/1139673003

BUG=484234

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

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

Saw a build error in http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_gn_chromeos_rel/builds/38730/steps/compile/logs/stdio

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

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

Otherwise when Browser::OnViewManagerDisconnected gets called, Browser::ui_ will get deleted twice.

This is split off https://codereview.chromium.org/1139673003

BUG=484234

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

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

Which of UIWebView and WKWebView to use can be controller by command line
flags, finch and availability (iOS version).

BUG=429756

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

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

This is split off https://codereview.chromium.org/1139673003

BUG=484234

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

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

This should have already been closed before we get to the destructor. Otherwise we get double deletes since the Destroy() call can end up calling the delegate which will delete this object.

Also in the X11 implementation, don't call delegate's OnClosed() before null'ing xwindow_, otherwise Destroy() can be called multiple times.

This is split off https://codereview.chromium.org/1139673003

BUG=484234

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

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

This was a good experiment, but there's nobody currently driving this iteration, and the code is intrusive, accumulating technical debt around it.

BUG=370696

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

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

BUG=452726

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

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

PushMessagingAppIdentifier::FindByAppId was DCHECKing when called from
PushMessagingServiceImpl::CanHandle to see if an app_id belongs to
Push, since it assumes that it would only be called with valid Push
app_ids.

This patch makes it test whether the app_id is a Push Messaging app_id
before performing any validation DCHECKs.

BUG=489304

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

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

BUG=none
TEST=none

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

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

This addresses a TODO and removes an ugly constant from NetLog.  On
the down side, TimeTicks::UnixEpoch() is cached, which may result in
less accuracy (And all bets are off if Chrome is suspended to disk).

BUG=488660

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/f6e36c6..d804fe3

TBR=kbr@chromium.org,yhirano@chromium.org,hiroshige@chromium.org

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

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

Using 'cache = ...' always assigns ... to the local scope, while it
should be stored in the parent scope instead. Because of this oversight,
the cache only contained one list of revisions at any time, namely the
list of revisions for the current settings (e.g. "-a linux64" or "-l -a
linux64").
This patch fixed the issue by iterating over the cached dictionary and
copying the keys and values to the variable (in the parent scope).

R=rsesek@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/5df6fee..9b77796

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=bungeman@google.com

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

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

Revert of Revert of Use BoringSSL in the implementation of ClearKey for Chromecast. (patchset #1 id:1 of https://codereview.chromium.org/1141243003/)

Reason for revert:
OK to replay now

Original issue's description:
> Revert of Use BoringSSL in the implementation of ClearKey for Chromecast. (patchset #1 id:1 of https://codereview.chromium.org/1130893008/)
>
> Reason for revert:
> Temporary revert until Monday (blocked on some other internal changes).
>
> Original issue's description:
> > Use BoringSSL in the implementation of ClearKey for Chromecast.
> > (Previously, this was used only with the "Chrome" branding).
> >
> > BUG= 488572
> >
> > R=gunsch@chromium.org
> >
> > Committed: https://crrev.com/defd2af08f3753687cb2d86b34bbec9da31ccbab
> > Cr-Commit-Position: refs/heads/master@{#330156}
>
> TBR=ddorwin@chromium.org,dougsteed@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG= 488572
>
> Committed: https://crrev.com/564d81833f28f73b9413c6d01725530ff91de64c
> Cr-Commit-Position: refs/heads/master@{#330230}

TBR=ddorwin@chromium.org,dougsteed@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 488572

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

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

This patch re-enables loading from APK on Samsung devices where the mmap
exec check is skipped (due to
https://code.google.com/p/chromium/issues/detail?id=448084).

BUG=489267

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

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

It's not used in net, and net/ knowing about frames is a layering
violation.  External consumers that depended on it now use the
corresponding ResourceType instead.

BUG=426442

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

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

BUG=489239

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

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

This CL lands server change 93655511 by bnc.

BUG=488484

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

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

BUG=345769

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

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

WebRTC 9189:9205
Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/872cf25..3c9a06a

Libjingle 9186:9198
Changes: https://chromium.googlesource.com/external/webrtc/trunk/talk.git/+log/a8c7824..87e6b84

TBR=sergeyu@chromium.org
BUG=

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

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

If overriding cert errors is disabed by policy, users shouldn't be able
to use the "danger" command to bypass it.

BUG=440949
R=meacer@chromium.org

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

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

BUG=None

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

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

This CL lands server change 92358475 by bnc.

BUG=488484

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/b19d3aa..f6e36c6

TBR=kbr@chromium.org

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

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

BUG=429756

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

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

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

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

BUG=480652
TBR=asanka@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/5b5ddd7..5df6fee

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=bungeman@google.com

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

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

BUG=488302

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

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

ios/chrome/browser/chrome_url_util.{h,mm} provides some URL utilities for
use on iOS.

Add missing link dependency on UIKit framework.

BUG=429756

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

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

Update swipe recognizer to differentiate between edge/non-edge
swipes.

BUG=None

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

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

BUG=429756

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

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

Logs test cases which will be run, which helps to catch cases when you
misspell config parameter name and run all test cases instead of
specified once.

BUG=369521
R=vabr@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/4657068..5b5ddd7

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=bungeman@google.com

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

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

Revert of Reland: Lazily initialize MessageLoop for faster thread startup (patchset #5 id:160001 of https://codereview.chromium.org/1129953004/)

Reason for revert:
Massive data race reports, see https://crbug.com/489263

Original issue's description:
> Reland: Lazily initialize MessageLoop for faster thread startup
>
> Original review: https://codereview.chromium.org/1011683002/
>
> Reverted because it's suspected for following flakiness issues:
> http://crbug.com/485157 - Windows race
> http://crbug.com/485091 - Android ThreadWatcher
> http://crbug.com/485178 - interactive_ui_tests Menu* tests
>
> PS1 is the original patch set that gets reverted.
>
> BUG=465458, 485157, 485091, 485178
> TBR=jam
>
> Committed: https://crrev.com/8b6133a69f16702a32a3c3104630c4d9ac393b7a
> Cr-Commit-Position: refs/heads/master@{#330329}

TBR=thakis@chromium.org,toyoshim@chromium.org,jam@chromium.org,kinuko@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=465458, 485157, 485091, 485178

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

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

Improved logging in chrome code in case of error in policy fetching.

BUG=489230

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/36757d61..b19d3aa

TBR=kbr@chromium.org

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

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