GitLab

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/f3ed19e..d231f16

TBR=apavlov@chromium.org,dpranke@chromium.org

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

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

There are situations where we really don't mind if firing a
[Pending]Intent doesn't return to Chrome.  In these cases,
alerting the user about Chrome dying is unnecessary.

BUG=453170

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

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

Currently, this is gated behind the enable-namespace-sandbox switch.
Furthermore, the namespace sandbox is only used if seccomp-bpf is
supported.

BUG=312380

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

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

See inline comment for gruesome details. Basically, just ignore the
Ultra Bold weight.

R=shrikant@chromium.org,eae@chromium.org,cpu@chromium.org
BUG=408093

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

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

Refactored pixel format resize operations in  media/video/capture into a function called VideoCaptureFormat::GetSizeForVideoPixelFormat().

BUG=

TEST=VideoCaptureDeviceTest.* in media_unittests pass.

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

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

Revert of Enable libc++ on Android (patchset #16 id:460001 of https://codereview.chromium.org/835633003/)

Reason for revert:
Slows down the slowest bot on CQ by 20 minutes.

BUG=456396

Original issue's description:
> Enable libc++ on Android
>
> Switch Chrome on Android to use libc++ instead of stlport. WebView AOSP
> builds will continue to use stlport for the time being.
>
> Note that this change is exploratory, and likely to be reverted before
> branch point (ensuring WebView and Chrome remain in lockstep with
> respect to standard library dependencies).
>
> BUG=427718
> TBR=cpu@chromium.org
>
> Committed: https://crrev.com/8681920fa22ded465054301ce96657a4ddaf2a04
> Cr-Commit-Position: refs/heads/master@{#315085}

TBR=fdegans@chromium.org,pasko@chromium.org,jamesr@chromium.org,thakis@chromium.org,boliu@chromium.org,davidben@chromium.org,cpu@chromium.org,piman@chromium.org,jdduke@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=427718

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

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

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.

BUG=456327,456331,456373
TEST=none
TBR=mmenke, erikwright

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

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

BUG=456042

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

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

And use it to flush rendering before page flip.

BUG=412508,442605
TEST=build and run Chrome on jerry/freon. It should render one frame.
R=alexst@chromium.org
R=dnicoara@chromium.org
R=kbr@chromium.org
R=piman@chromium.org
Signed-off-by: Dominik Behr <dbehr@chromium.org>

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

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

This addresses a TODO and reverts
https://codereview.chromium.org/777303004.

With https://codereview.chromium.org/889083002
the CrashDumpManager is now initialized earlier so
things should be fine.

BUG=439322

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

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

BUG=419551

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

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

Mojo's MessagePipe constructor uses assert() to check that
MojoCreateMessagePipe returns OK. This change makes the test explicitly
check the return value of MojoCreateMessagePipe, giving more correct
test behavior for Release builds.

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

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

This is a simple command, except we need to update client side caching of
uniform block info accordingly.

BUG=429053
TEST=gpu_unittests
R=piman@chromium.org

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

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

Note ui/views/BUILD.gn actually has it already.

views_unittests compile regressed in r314860

It's only a problem for toolkit-views on Mac because it doesn't depend
on aura_test_support there (which also has the dependency).
views_test_support should have it for the call to
ui::{Initialize,Shutdown}InputMethodForTesting() in ViewsTestBase.

BUG=378134

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

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

They may have problems when using hardware video decoding and D3D11 on the discrete gpu.

BUG=451420

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

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

By default, Android optimizes what it shows on top of
SurfaceViews (saves power). Effectively, layouts determine
what gets drawn and post-layout animations outside of this
area may be 'clipped'.

This adds an convenient way to avoid this clipping. When an
animation wishes to animate over a SurfaceView, it just
needs to register itself in its ContentViewCore and the
rest will be taken care of automatically.

BUG=441805

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

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

additional_res_dirs was removed and additional_res_packages does not use it
anymore.

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

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

This reverts commit 787196b9 and disables the handle closer on Windows 10.

BUG=452613

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

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

[Taken over from blundell's https://codereview.chromium.org/901843003/,
PS3.]

Adapt to de-clienting of Surface.

Have nacl_listener pass a SimplePlatformSupport instance to
mojo::embedder::Init to satisfy the DCHECK introduced in
https://codereview.chromium.org/898623002/diff/40001/mojo/edk/embedder/embedder.cc.

[Additional fix backported from
https://codereview.chromium.org/898423002/.]

Fix mojo::embedder::CreateChannel(): the evaluation order between the
channel_info->channel_id and channel_info.release() is undefined.

[Additional "fix" backported from
https://codereview.chromium.org/903933006/.]

Add an "unnecessary" .Pass() to try to placate MSVS.

[Additional fix:]

surfaces_service.mojom.h -> surfaces.mojom.h

R=jamesr@chromium.org
NOPRESUBMIT=true
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win8_chromium_gn_dbg,win8_chromium_gn_rel

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

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

This option diagnoses dependency problems at build time.

TEST=linux_chromium_chromeos_ozone_rel_ng & linux_ecs_ozone tryjobs
BUG=445627

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

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

NetworkChangeNotifier was moved to HostProcessMain in crrev.com/314165.
On windows it must be initialized after MessageLoop for the calling
thread is created, so the host was failing to start.

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

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

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

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

We have users of html_viewer that do not use the mojo provided services, including the surface service. This change: https://codereview.chromium.org/869343004/ broke them

I think it's right to check this here. Another option would be to do it in layer_tree_host.cc

BUG=

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/722374e..f3ed19e

TBR=apavlov@chromium.org,dpranke@chromium.org

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

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

If in software rendering mode, a copy request
of the layer is not in the root or first render pass,
there will be double locking resource crash.
Now we first release old lock and then try to
obtain lock again.

R=danakj@chromium.org

TEST=SoftwareRendererTest.DoubleLockTextureCrash

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

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

http://build.chromium.org/p/chromium.fyi/builders/Closure%20Compilation%20Linux/builds/14594/steps/compile/logs/stdio

TBR=kalman@chromium.org
BUG=none
TEST=green closure bots

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

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

The scripts-require-action feature should affect all extensions that don't have
the "can run on all urls" preference. When an extension is installed, it should
have the "all urls" flag by default if the switch is off.

This is necessary in order to allow syncing the all urls flag.

BUG=454832

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

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

Only download from cloud_storage when a bucket parameter is used in the user_story. Use system_stub instead of going over the network in user_story_runner_unittest.

BUG=447462

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

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

This is enabled via an attribute CODECAPI_AVLowLatencyMode set on the decoder transform.
From testing I could see a reduction in the number of input packets before the first output frame was delivered
by the decoder. On vimeo, with this attribute, the first output frame came after 9-10 input packets. Without it
after 25-30 input packets. On the HTML5 quirksmode site, with this attribute the first output frame came after 9-11 input
packets. Without it it came after 16-17 input packets.

We can enable this on Canary to see how this fares in the wild.

BUG=456359

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

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

Support for the wildcard is going away in GN, so this helps prepare for the transition away from GYP.

This cl adds dummy targets so that that logic for when the proper targets should be build can be kept in the platform-specific files.

Some of the conditions in the host gyp files needed to be restructured so that the dummy targets would be defined only when necessary.  The updated conditions caused a lot of lines to be re-indented, but the target definitions are the same (except for the new dummy targets).

BUG=

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

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

friends) to measure the time from the URL starting to load to it loading.
Extensions.BackgroundPageLoadTime2 replaces Extensions.BackgroundPageLoadTime
etc.

Previously it measured from ExtensionHost construction to it loading, which is
wrong because URL loading can be deferred in a queue, or callers just might not
try to load it until later.

Also do a bit of UMA cleanup: delete some of the Extension page load UMA that
isn't used anymore, and add back the Extensions.PopupLoadTime2 load UMA which
was forgotten about at some point.

R=yoz@chromium.org, isherman@chromium.org
BUG=453073

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

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

We were previously using --test-data to push files from
//content/test/data/android/device_files to
$EXTERNAL_STORAGE/chrome/test/data/content. Switching to isolate lets
us handle all of our instrumentation and gtest data dependencies in the
same way. It will also facilitate using isolate to optimize transfers
between builders and testers.

BUG=400499
TBR=maruel@chromium.org

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

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

Let extensions with active tab use XHR in background pages to any hosts that
they have active tab on.
Also move tab_finder to //extensions.

BUG=452688

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

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

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

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

Revert of NaCl: Update revision in DEPS, 2363d1a -> a3e1e94 (patchset #1 id:1 of https://codereview.chromium.org/905793003/)

Reason for revert:
Looks like it broke the build:

http://build.chromium.org/p/chromium.webkit/builders/Linux%20Tests/builds/42375/steps/compile/logs/stdio

FAILED: cd ../../remoting; python ../native_client/build/build_nexe.py --root .. --product-dir ../out/Release/xyz --config-name Release -t ../out/Release/gen/sdk/ --arch pnacl --build newlib_pexe --name ../out/Release/remoting_key_tester_newlib.pexe --objdir ../out/Release/obj/remoting/remoting_key_tester_pexe.gen/pnacl_newlib/remoting_key_tester_pexe "--include-dirs=../out/Release/gen/tc_pnacl_newlib/include .. .. \"../out/Release/gen\"" "--compile_flags=-O2 -g -Wall -fdiagnostics-show-option -Werror  -gline-tables-only -Wno-unused-function -Wno-char-subscripts -Wno-c++11-extensions -Wno-unnamed-type-template-args -Wno-extra-semi -Wno-unused-private-field -Wno-char-subscripts -Wno-unused-function " --gomadir /mnt/data/b/build/goma "--defines=\"__STDC_LIMIT_MACROS=1\" \"__STDC_FORMAT_MACROS=1\" \"_GNU_SOURCE=1\" \"_DEFAULT_SOURCE=1\" \"_BSD_SOURCE=1\" \"_POSIX_C_SOURCE=199506\" \"_XOPEN_SOURCE=600\" \"DYNAMIC_ANNOTATIONS_ENABLED=1\" \"DYNAMIC_ANNOTATIONS_PREFIX=NACL_\" \"NACL_BUILD_ARCH=x86\" \"BINARY_CORE=1\" \"BINARY_DESKTOP=2\" \"BINARY_HOST_ME2ME=3\" \"BINARY_NATIVE_MESSAGING_HOST=4\" \"BINARY_REMOTE_ASSISTANCE_HOST=5\" V8_DEPRECATION_WARNINGS \"_FILE_OFFSET_BITS=64\" CHROMIUM_BUILD \"CR_CLANG_REVISION=223108\" \"TOOLKIT_VIEWS=1\" UI_COMPOSITOR_IMAGE_TRANSPORT \"USE_AURA=1\" \"USE_ASH=1\" \"USE_PANGO=1\" \"USE_CAIRO=1\" \"USE_DEFAULT_RENDER_THEME=1\" \"USE_LIBJPEG_TURBO=1\" \"USE_X11=1\" \"USE_CLIPBOARD_AURAX11=1\" ENABLE_ONE_CLICK_SIGNIN ENABLE_PRE_SYNC_BACKUP \"ENABLE_REMOTING=1\" \"ENABLE_WEBRTC=1\" ENABLE_PEPPER_CDMS ENABLE_CONFIGURATION_POLICY ENABLE_NOTIFICATIONS USE_UDEV DONT_EMBED_BUILD_METADATA \"ENABLE_TASK_MANAGER=1\" \"ENABLE_EXTENSIONS=1\" \"ENABLE_PLUGINS=1\" \"ENABLE_SESSION_SERVICE=1\" \"ENABLE_THEMES=1\" \"ENABLE_AUTOFILL_DIALOG=1\" \"ENABLE_BACKGROUND=1\" \"ENABLE_GOOGLE_NOW=1\" \"CLD_VERSION=2\" \"ENABLE_PRINTING=1\" \"ENABLE_BASIC_PRINTING=1\" \"ENABLE_PRINT_PREVIEW=1\" \"ENABLE_SPELLCHECK=1\" \"ENABLE_CAPTIVE_PORTAL_DETECTION=1\" \"ENABLE_APP_LIST=1\" \"ENABLE_SETTINGS_APP=1\" \"ENABLE_SUPERVISED_USERS=1\" \"ENABLE_MDNS=1\" \"ENABLE_SERVICE_DISCOVERY=1\" V8_USE_EXTERNAL_STARTUP_DATA \"USE_LIBPCI=1\" \"USE_GLIB=1\" \"USE_NSS=1\" __STDC_CONSTANT_MACROS __STDC_FORMAT_MACROS" "--link_flags=-B../out/Release/gen/tc_pnacl_newlib/lib -O3 -lppapi_stub -lppapi_cpp" "--source-list=../out/gypfiles/remoting/pnacl_newlib.remoting_key_tester_pexe.source_list.gypcmd"
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/bin/le32-nacl-ld.gold:error: cannot open ../out/Release/gen/tc_pnacl_newlib/lib/libnacl.a: No such file or directory
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to '_realloc_r'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to 'read'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to 'close'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to 'lseek'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to '_calloc_r'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to 'isatty'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to 'fstat'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to '_free_r'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc.a:error: undefined reference to '_malloc_r'
../out/Release/gen/tc_pnacl_newlib/lib/libpthread.a:error: undefined reference to '__pthread_initialize_minimal'
../out/Release/gen/tc_pnacl_newlib/lib/libpthread.a:error: undefined reference to '__nacl_tls_initialize_memory'
../out/Release/gen/tc_pnacl_newlib/lib/libpthread.a:error: undefined reference to 'calloc'
../out/Release/gen/tc_pnacl_newlib/lib/libpthread.a:error: undefined reference to '__nacl_tls_combined_size'
../out/Release/gen/tc_pnacl_newlib/lib/libpthread.a:error: undefined reference to '__libnacl_mandatory_irt_query'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc++.a:error: undefined reference to 'malloc'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/le32-nacl/lib/libc++.a:error: undefined reference to 'free'
../out/Release/gen/tc_pnacl_newlib/lib/libppapi_stub.a:error: undefined reference to '_exit'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/lib/clang/3.5.0/lib/le32-nacl/unwind_stubs.bc:error: undefined reference to 'write'
/mnt/data/b/build/slave/Linux_Tests/build/src/out/Release/gen/sdk/linux_x86/pnacl_newlib/lib/clang/3.5.0/lib/le32-nacl/unwind_stubs.bc:error: undefined reference to 'abort'
FAILED with 1: ../out/Release/gen/sdk/linux_x86/pnacl_newlib/bin/pnacl-clang++ -o ../out/Release/remoting_key_tester_newlib.pexe.debug -Wl,--as-needed ../out/Release/obj/remoting/remoting_key_tester_pexe.gen/pnacl_newlib/remoting_key_tester_pexe/remoting_key_tester_61dcf3bc.o -B../out/Release/gen/tc_pnacl_newlib/lib -O3 -lppapi_stub -lppapi_cpp

Original issue's description:
> NaCl: Update revision in DEPS, 2363d1a -> a3e1e94
>
> This pulls in the following Native Client changes:
>
> 6c11549: (mseaborn@chromium.org) Add IRT interface for PNaCl's sandboxed linker to use to talk to Chromium
> 9fff96b: (teravest@chromium.org) NaCl: Remove use of NACL_BUILD_* in common.gypi.
> 76d6d4b: (mseaborn@chromium.org) PNaCl: Update Binutils revision in pnacl/COMPONENT_REVISIONS: use new IRT interface
> a3e1e94: (jfb@chromium.org) Delete Windows time code
>
> BUG=none
> TEST=browser_tests and nacl_integration
> CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_compile,linux_nacl_sdk_build
>
> Committed: https://crrev.com/accbfa1e86e32df669fad209185cba4acd6847be
> Cr-Commit-Position: refs/heads/master@{#315133}

TBR=jfb@chromium.org,mseaborn@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=none

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

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

TBR=bauerb@chromium.org
BUG=456375

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

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

Roll third_party/cros_system_api to 286cec023e90f54f391fe32bae512f9ec7b4e807

BUG=407773
TBR=benchan@chromium.org

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

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

This broke quite a bit ago when the implicit ordering changed by refactoring.
It is unfortunate that we don't have test coverage for mac, and
trying to type accent characters (IME) from browsertest didn't work.

BUG=450641
Test=On mac, load a chrome app with <webview> that has an input box.
Try to type accent chars, e.g. Option+e followed by e. Proper chars
should be seen in the input box.

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

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

VideoFrame conversion is to RGBA, so we need to specify
BGRA when uploading the texture.

BUG=450205

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

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

Now that we have fastbuild and Goma working, it should be safe to
link the full binary again in a GN build on the bots.

TBR=brettw@chromium.org, jam@chromium.org
BUG=354261

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

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