GitLab

BUG=488661
TEST=None

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

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

Update tz db to 2015d

 Summary of changes available at:
 https://chromium.googlesource.com/chromium/deps/icu/+log/ce41627..5788e27

BUG=473288
TBR=jsbell@chromium.org
TEST=In JS console, run the following to get 1:00 PM for 2015
instead of 2:00 PM. In 2014, it should return 2:00 PM.
  may_25_2015_1100 = new Date("05/25/2015 11:00Z")
  may_25_2015_1100.toLocaleString("en", {timeZone: "Africa/Cairo"})
  may_25_2014_1100 = new Date("05/25/2014 11:00Z")
  may_25_2014_1100.toLocaleString("en", {timeZone: "Africa/Cairo"})

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

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

The current implementation, in fact, doesn't support specifying both
"--v" and "--log-level" command line switches: only "--log-level" switch
will take an effect. This looks counterintuitive.

For example, browser_tests always start the browser with the
"--log-level" switch specified, so with current implementation there's
no way to enable vlogging in browser_tests using a command line.

The suggestion is, as both of the options refer to the same underlying
"min_log_level", to make the "--v" switch have a higher priority.

BUG=

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

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

Added the Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histogram to track time between tab switches.

TEST=Manually verified histogram data added on ChromeOS.

BUG=478930

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

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

BUG=486739

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

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

This CL classifies the following fallback triggers in BoringSSL ports:

- ClientHello intolerance.
- Microsoft's buggy AES-GCM implementation.
- CertificateVerify was rejected (some servers broke TLS 1.2 client auth).
- Resuming a session at the wrong version.
- F5's buggy NPN implementation.
- Other.

It also records how often each error code triggers the fallback.

Initial numbers for the fixed metrics are actually rather encouraging,
but put this in place for the same release if larger populations' numbers
are worse and we'd need to carve out large swathes of intolerance with
less problematic workarounds.

BUG=459690

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

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

The functions being called already checks for NULL input.

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

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

Use WebContentsObserver::RenderViewHostChanged() instead.

BUG=170921
Test=No behavioral change.

Sanity checked with JS speech, whether WebContentsTracker class gets created and destroyed properly.

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

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

X uses 96 as default, and alters the screen size
but it can be overwritten. Compute the dpi from X
to match what gtk sues as base.

Round the dsf to 1 decimals as finer grained value can cause rendering problem than benefit.

Gpu test expectation on nvidia needs to be updated, so mark these tests as failing now.

BUG=485183

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

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

BUG=486739

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

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

On some systems with buggy power managers, calls to the power
manager would result in long pauses that would block the FILE thread.
This yielded degraded performance for other browser tasks
scheduled on the same thread.

BUG=173354
R=miu@chromium.org,rdsmith@chromium.org

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

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

This updates libsrtp to 1.5.2 and uses the OpenSSL/BoringSSL crypto code.
See https://codereview.chromium.org/889083003/ and https://codereview.chromium.org/1098043003/

Summary of changes available at:
https://chromium.googlesource.com/chromium/deps/libsrtp/+log/6446144..9c53f85

This previously landed in https://codereview.chromium.org/1130413005/ but got
reverted in https://codereview.chromium.org/1139233002/ as it broke the Chrome
Linux x64 builds due to a added dependency to "libgcc_s.so.1(GCC_3.0)(64bit)".
The dependency was already allowed for 32bit, so it is allowed for 64bit now,
too.

BUG=328475

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

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

Properly transfer ownership between PermissionBubbleManagers as
tabs are switched.

BUG=340720
R=rohitrao@chromium.org

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

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

The only place where Device ID is stored on Chrome OS is the local state from now on.

Order of events:
1) User logs in. There are two options:
  a) User athenticated through GAIA. We read known user device ID from local state if it exists or generate a new one if it doesn't. Then we save the device ID in a user context.
  b) Otherwise, we don't store device ID in user context.
2) If it is GAIA_WITHOUT_SAML-mode, we send device ID to LSO in "oauth2/token" request (OAuth2TokenInitializer class). If it is not, we'll send it in "o/oauth2/programmatic_auth" request on step (5).
3) After NotifyUserLoggedIn() call, we store the device ID from the user context (if it is set) to local state. Then we start to create/load a profile for the user.
4) In constructor of ChromeSinginClient we migrate device id from kGoogleServicesSigninScopedDeviceId pref to the local state, if a value in the local state is empty.
5) We request tokens with "o/oauth2/programmatic_auth" if we haven't done that on step (2). Old code for sending device ID works here, because it relies on ChromeSigninClient which returns device ID set on step (4).

BUG=486044,486136

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

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

This patch updates record_wpr to use argparse for:
+ record_wpr <XYZ> will make <XYZ> a benchmark name to be recorded.
+ record_wpr --user-story <XYZ> will make <XYZ> a user
story (page set) name to be recorded
+ record_wpr --list-benchmarks & --list-user-stories will
list out benchmark names & user stories (page sets) names.
This is useful since the current record_wpr target name
matching is a bit funky due to  discover.DiscoverClasses's
keys naming scheme.

Subsequent patches will update it so that the target
matching name for benchmarks matched the convention in
benchmark_runner.py & matching name for user stories use
the user stories's class name.

Since more code refactoring are required, I haven't added
unittest for these new changes yet.

TEST="
git cl patch this.
./perf/record_wpr --help
./perf/record_wpr --list-user-stories
./perf/record_wpr --list-bechmarks

"

BUG=470865

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

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

BUG=476587

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

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

Removed extra 'dependency' on target 'media_router_test_support'.

Also removed the ':' on some local dependencies.

Original patch: https://codereview.chromium.org/1136623003/
Original patch description:

- Refactor GYP and GN files to share source lists.
- Extract a proper test_support target.
- Fix upstream files with new target structure.

BUG=461815, 464199, 464205
TEST=Linux trybots. It's not clear to be exactly which bot, since
original patch failed after getting through trybots/CQ.

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

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

requests.

For StartSession, PSImpl keeps a queue of StartSession requests to be
processed. (It can only process one at a time). If the queue has
reached maximum allowed size (10), subsequent incoming requests will
be immediately rejected.

For Joinsession, PSImpl keeps a map of currently JoinSession requests
that are currently processed. If the map reached maximum allowed size
(10), subsequent incoming requests will be immediately rejected.

Also, tried to simplify some of the cleanup flow for
NewSessionMojoCallbacks by introducing a wrapper class that ensures
callbacks are not dropped. Currently the cleanup logic is sprinkled
all over the file so hopefully this makes things cleaner.

We could generalize the callback wrapper pattern in the future if
it makes necessary.

BUG=412331

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

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

Change https://codereview.chromium.org/1014543006 was supposed to disable
AutomationApiTest.Mixins because it is flaky (it fails ~8% of the time on
Windows, and is also flaky on Linux).

However the disabling didn't work because the newly created MAYBE_Mixins
macro is never used. This fixes that by renaming the test.

The line that always fails is:

assertEq(2, input.textSelStart);

The output is:

Actual: 0
Expected: 2

R=ellyjones@chromium.org
TBR=aboxhall@chromium.org
BUG=467921

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

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

This moves native_viewport_service into core_services on desktop
mandoline builds, colocating the service with the surfaces service. We
can't do this on android yet because android does some tricks with the
ApplicationLoader to make the native_viewport_service run in the shell's
main thread.

BUG=484234

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/f3c3784..db1f311

TBR=timvolodine@chromium.org,kbr@chromium.org

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

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

Added to GYP recently https://codereview.chromium.org/1136953009 .
Adding the same flag to GN.

Also update android_tools' BUILD.gn to use v14_skip

BUG=487391

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

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

BUG=486739

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

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

This pulls in the following Native Client changes:

7aaf68d: (dschuff@chromium.org) Fix mode of run.py
a55fb84: (sbc@chromium.org) Update ARM sysroot in toolchain_revisions/arm_trusted.json
a0d24db: (bradnelson@chromium.org) Enabling asan pnacl toolchain builders.
9f62c3e: (sbc@chromium.org) Update QEMU version 1.0.1 -> 2.3.0
e04b61c: (kschimpf@google.com) Fix typo in iss https://codereview.chromium.org/1140793002
ce18634: (sbc@chromium.org) Roll ARM sysroot image to bring in new QEMU version
4e5d3d4: (mseaborn@chromium.org) PNaCl: Update LLVM revision in pnacl/COMPONENT_REVISIONS
30d4b12: (mcgrathr@chromium.org) toolchain_build: Update glibc revision
0beb554: (jvoung@chromium.org) PNaCl: Update Subzero revision in pnacl/COMPONENT_REVISIONS
473ef0b: (jvoung@chromium.org) Update revision for PNaCl
2da5677: (sbc@chromium.org) Re-enable some ARM qemu tests now that we are using 2.3.0
862ca26: (dschuff@chromium.org) pnacl-ld: Don't run expand-varargs with relocatable output

BUG=none
TEST=browser_tests and nacl_integration
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_chromium_dbg_32_ng,linux_arm_compile,linux_nacl_sdk_build

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

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

In the past due to a kernel error, modesetting with a null fb didn't
release the front buffer, this workaround was put in place. Kernel bug was
fixed and this is no longer needed.

BUG=

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

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

Make BrowserCdmCast::Initialize() a virtual function so that a subclass can override the initialization routine.

BUG=internal b/20222899

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

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

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

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

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

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

The code that tried to inject ChromeVox into WebViews in
chrome_web_view_guest_delegate.cc wasn't working. Instead, this allows
whitelisted content scripts to be injected into WebViews.

BUG=484904

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

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

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

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

BUG=488489
TEST=With Chrome SxS running, allow Google Update to download and install an update, then navigate to chrome://help. It should report that a restart is needed.
R=pkasting@chromium.org

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

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

Fix an issue where the selection handles remain hidden if "empty"
selection updates occur after a longpress but before the nonempty
selection update.

BUG=393025
NOTRY=true

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

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

Passing down the URL parameters given to the MojoShellActivity from the
intent and making it handle HTTP/HTTPS URLs.

BUG=None

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

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

This is in preparation for for creating abi splits

BUG=484797

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

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

This permits determining the impact on HTTP vs HTTPS Data Reduction Proxy
use by changing the secure proxy check behavior (per the changelist
https://codereview.chromium.org/1132083004/).

BUG=466753

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

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

First-run tutorial is shown for regular ephemeral users, but only if we 100%
sure that this user has never seen the tutorial yet.

BUG=333264
TEST=manually

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

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

BUG=

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

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

This reverts https://crrev.com/5171d85366fe1bd7c1de43e866adc3c3e50bc419

BUG=none
TBR=thakis

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

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

Currently the utility call for emphasizing urls uses useDarkText
as a signal to decide whether https scheme should be colored. Since
with theme colors this logic gets more compicated, this change adds
another explicit param to control the coloring of https.

BUG=482780

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

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

This CL implements the RegisterAdvertisement and UnregisterAdvertisement
functions for the Chrome API.

BUG=466375

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

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

There was a mismatch between core_services_initialization.cc, the
BUILD.gn file, and the package listing.

BUG=484234

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

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