GitLab

Mean frame time is in other benchmarks, but it's a good
sanity check for understanding performance changes. So is
tasks-per-frame on each thread, so that is also added in this
patch.

BUG=None

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

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

BUG=425164

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

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

Mojo: mojob.sh: Add support for gn, and remove support for gyp and shared library (component) builds.

R=jamesr@chromium.org

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

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

This CL allows web pages to intercept the Ctrl+Shift+0 accelerator when no UI scaling is applied (which is the common case)

BUG=404473
TEST=None

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

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

Data reduction proxy statistics are accumulated in local
state preferences. To support desktop and other platforms
with multiple profiles, these are being ported to profile
preferences.

BUG=411373

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

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

There were several places in this file where an error code value
was checked to see if it was less than QUIC_NO_ERROR or
QUIC_STREAM_NO_ERROR, which are both 0. On some compilers this
causes a warning, because such checks are trivially true.

BUG=424334
TEST=net_unittests
R=rch@chromium.org

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

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

Remove references to cancel-button CSS class, which is not defined.

BUG=none
R=dbeam@chromium.org

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

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

bridged_native_widget_unittest.mm currently fails to compile on 10.7+
SDKs because there is no such thing as
NSWindowDidFailToEnterFullScreenNotification (whoops).

The failures don't go through NSNotificationCenter, but to the
NSWindowDelegate protocol directly. So, add a dummy protocol for 10.6
and, in the simulate fullscreen failure test, send
windowDidFailTo{Enter,Exit}FullScreen to the window delegate directly.

Call via `id` so the test compiles on 10.6 SDKs, but return early so the
tests pass when actually running on 10.6.

BUG=378134
TEST=By tracing the simulation test, and manually to generate an actual
failure.

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

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

R=lcwu@chromium.org,damienv@chromium.org
BUG=336640

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

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

This change makes Chrome on Chrome OS ask permission_broker for access to serial devices.

BUG=173548
TEST=Chrome can access serial devices on Linux, chromeos=1, full Chrome OS.

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

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

Throw IllegalArgumentException from Cronet HttpUrlRequestFactoryConfig.addQuicHint if host param is URL.

This is needed to prevent invalid hosts from being added to HttpServerProperties and having Quic hints not working correctly.

HttpUrlRequestFactoryConfig is pure Java and doesn't depend on native library, so it only provides a rudimentary check, but URLRequestContextAdapter::InitRequestContextOnNetworkThread() performs IsCanonicalizedHostCompliant check and uses LOG(ERROR) for invalid hosts.

BUG=420464
TEST=build/android/test_runner.py instrumentation  --test-apk=CronetTestInstrumentation -f *Quic*

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

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

Pick up the changes to use scoped_ptr<T>::Pass() instead of PassAs().

BUG=423621

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

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

Mechanical change that adds instrumentation required to locate the source of jankiness (i.e. a long-running fragment of code executed as a part of the task that causes jank) in the code. See the bug for details on what kind of jank we are after.
A number of similar CLs were landed, and none of them caused issues. The code of the instrumentation is highly optimized and is not expected to affect performance. The code simply creates a diagnostic task which is identical to ones created by PostTask or IPC message handlers.
Landing as TBR since this is a mechanical, safe and temporary change.

This CL instruments most popular OnRefreshTokenAvailable overrides.

BUG=422460
TBR=jam@chromium.org, zelidrag@chromium.org, erikwright@chromium.org

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

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

R=lcwu@chromium.org,gusfernandez@chromium.org
BUG=None

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

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

CQ somehow broke the binary file. This CL relands it.

BUG=318404
TBR=yfriedman@chromium.org

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

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

This is mostly to fix MSVC warnings about possible value truncation.

BUG=81439
TEST=none

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

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

The string is changing to "This computer will no longer receive Google
Chrome updates because its hardware is no longer supported." Previously,
the message began with "This computer will soon stop receiving".

The new string has existed in Chrome for a long time in preparation for
this change, and it has already been translated.

BUG=425259

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

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

This change does not affect the behavior when using a system-configured
proxy or a PAC file.

BUG=123716

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

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

This pulls in the following Native Client changes:

r13932: (jvoung) Keep the perf data of the trusted version of performance_tests separate.
r13933: (jvoung) Enable arm callingconv test for varargs and va_list.
r13934: (jvoung) Allow 'opt' bots to use GN with is_debug=false ('dbg' bots get debug).
r13935: (nfullagar) Initial benchmark suite for toolchain side.
r13936: (nfullagar) Add third_party/computer_language_benchmarks_game to DEPS
r13937: (petarj) [MIPS] Add missing -mtls-use-call when building irt libraries
r13942: (yyanagisawa) Avoid using mismatch compilers from buildbot.
r13947: (shyamsundarr) allow avx/xop/fma instructions but disable all them via def file attributes.

BUG=none
TEST=browser_tests and nacl_integration
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_nacl_sdk,linux_chromium_dbg

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

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

mojo_examlpes.gypi was removed in the following CL:
https://codereview.chromium.org/647263005

BUG=423889
TEST=NONE
R=jamesr@chromium.org

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

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

Release notes for Checkstyle 5.9
http://checkstyle.sourceforge.net/releasenotes.html

The most important part is all the fixes for Indentation check that allows
us to finally enable it for presubmit.

Also adding OWNERS file to checkstyle directory.

BUG=318404
TBR=yfriedman@chromium.org

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

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

When navigation away from a page fails, the displayed error page uses
the zoom level of the current page since the underlying RenderViewImpl
is re-used without resetting its zoom level before displaying the error
page.

This CL forces the RenderViewImpl's zoom level to be reset to 0 whenever
the url being navigated to is unkown to the RenderViewImpl.

BUG=422464

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

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

BUG=422142
R=asvitkine

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

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

This deadlock involves SyncBackendRegistrar running on the sync thread and at least two
workers - one of them (worker A) running on UI thread and
another (worker B) running on some other background thread.
In this particular case it was a password worker running on Password model thread.

The deadlock occurs when the SyncBackendRegistrar shutdown
code running in the sync thread blocks on a waitable event
set by worker B, at the same time the worker B's thread is
blocked for whatever reason waiting to make a synchronous
call into the UI thread, and at the same time the worker A
is blocked on a lock owned by the sync thread while it attempts
to unregister itself with SyncBackendRegistrar.

I think the main issue here is that the sync thread might
block on a worker. This is possible in only one situation -
when that worker hasn't have a chance to run any tasks because
its own thread was blocked doing something else.

The fix removes this dependency. The only reason this
blocking occurs is because ModelSafeWorker has a mechanism
for subscribing / unsubscribing to the MessageLoop descruction
which has to be done in the right thread context.
In order to unsubscribe the worker needs to remember the message loop it runs on which is done
in SetWorkingLoopToCurrent callback, which is called from a task
posted on the worker's thread. There is also a waitable
event which is signalled at that time. The even is used to
block ModelSafeWorker::UnregisterForLoopDestruction from
delegating the call on an uninitialized message loop which
is possible only when SetWorkingLoopToCurrent has never been called.

There is no need to block in UnregisterForLoopDestruction.
If the registration for even loop destruction hasn't been
been done yet (because the thread is blocked) then there
really no reason to wait for that and then immediately
unsubscribe. We can skip both subscription to the even loop
notification altogether and let the registrar know that
it is OK to remove the worker. That's essentially what the
fix does.

I added a test that verifies that SyncBackendRegistrar::Shutdown
doesn't block on a blocked model type thread anymore.

BUG=423078

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

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

Added a mobile and desktop testing in case the browser being tested has a
user agent that defaults to the tablet user agent.

BUG=421229

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

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

Enable RenderTextHarfBuzz on Chrome OS.

BUG=321868,376077,423791

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

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

On some Android devices, the sysfs-based C-State info contains extra blank
lines which will cause an exception in C-State stat parser in Android platform
backend if the power metric and a Monsoon Power Monitor is in use. More
precisely, /sys/devices/system/cpu/cpu*/cpuidle/state*/name contains two
trailing newlines on at least Samsung Galaxy Note 3 and S5.

This patch filters out these blank lines from the sysfs output and fixes nearby
style.

BUG=421008
TEST=SysfsPowerMonitorMonitorTest.testGetCpuStateForAndroidDevices

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

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

This change causes sites using SSLv3 to show a "mixed content" level warning on
the lock icon and for the section in the connection status to go red.

There's no message added to the connection status because, if we plan on
removing SSLv3 by default in Chrome 40, then we want to merge this to Chrome 39
and thus there's no time to get a translation.

BUG=419870

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

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

Disable direct write if font count in registry is greater than or equal to threshold (currently 1750).

R=scottmg,ananta,cpu,asvitkine
BUG=421305

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

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

This will pave the way for a future CL that completely removes all //content
dependencies from the class.

TBR=ben@google.com
BUG=371476
TEST=No visible impact.

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

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

Also move a class implementation out of feature_channel.h.

BUG=424746

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

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

BUG=425248
TBR=nasko@chromium.org

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

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

Define a new macro S_MODEBITS to represent the
mode bits settable by chmod/fchmod.

Add test to verify that file type cannot be changed
with chmod.

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

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

This fixes keypresses on Logitech K400. There's no key repeat yet.

BUG=412996

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

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

Don't spam the logs on Windows, Mac, or Android when
GetFontRenderParams() is called with a non-empty query;
RenderTextHarfBuzz sends non-empty queries and is being
enabled for more platforms now.

Continue to log an error if a family name is requested on
these platforms, though.

BUG=424971

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/83506e0..c6bd766

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

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

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

Handle STAGE_INITIALIZATION_ERROR with a logging message

BUG=420111
TEST=manual

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/33e6466..43b8b36

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=mtklein@google.com

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

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

Extensions: Split the constants from the extensions_common target into its own extensions_common_constants target.

The extensions constants are used in too many places and very hard to
get rid of. Making it into its own target so it can be included in all
builds, even when extensions are not enabled.

BUG=424746

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

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

Some of the updated files got accidentally stashed and I needed to pop
and recheck that the content was up-to-date.

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

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