GitLab

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}

This can be disabled with the allow-sandbox-debugging flag.

BUG=455964

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

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

This is really silly, but gyp doesn't seem to track if a static_library
contains C++ code and needs a C++ link step. So our copy of libxml
breaks us by containing a little bit of (chromium-added) C++ code. This
is really a correctness issue in the build system.

BUG=445627
TEST=linux_ecs_ozone tryjob

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

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

The public hardly knew ye.  Replaced by AppBannerInfoBars.

BUG=453170

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

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

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

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

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

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

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

This reverts commit c2e11810.

TBR=viettrungluu@chromium.org
NOTRY=true
NOTREECHECKS=true
NOPRESUBMIT=true
BUG=

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

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

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

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

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

Re-landing the CL reverted here:
https://codereview.chromium.org/866573004

Opus audio codec is supported by the android platform starting from
Lollipop.  This CL enables canPlayType() support for Opus on Clank and
MSE playback of Opus in Clank. This brings Opus feature parity with
Desktop Chromium.

BUG=318436
TBR=jschuh,rsleevi

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

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

--single-process mode may happen crash in file of webrtc_logging.cc where code
is CHECK(!g_webrtc_logging_delegate). The orginal code is to expect there to
be only one g_webrtc_logging_delegate instance in a single render process.
g_webrtc_logging_delegate is created when a RenderThreadImpl is created.
But in --single-process model all RenderThreadImpl are created in browser
process and so g_webrtc_logging_delegate may be created many times in a single
process. Then CHECK(!g_webrtc_logging_delegate) will fail.

In this patch will change g_webrtc_logging_delegate from process-level to
RenderThread-Level. Since every render process will only keep one RenderThread.

BUG=455573

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

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

Add unittesting for ToolbarActionView to, at minimum, test the basic UI. Also
remove the dependency of ToolbarActionView on Browser, and create a
TestToolbarActionViewController.

BUG=453841
TBR=sky@chromium.org (gn file change)

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

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

Changes in this CL include:
1. Moved the daemon controller implemetation from |ElevatedDaemonController| to |DaemonControllerDelegateWin|
2. Removed all code related to COM activation in |DaemonControllerDelegateWin|.
3. Removed code related to SetOwnerWindow in |DaemonControllerDelegateWin|.
4. Removed code related to the pin confirmation dialog in |DaemonControllerDelegateWin|
5. Cleaned up the COM-styled code to make it adhere to the chromium coding styles.

BUG=453172

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

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

[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.

TBR=blundell@chromium.org
NOPRESUBMIT=true

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

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

$ git log 94aea8c60..bdad1183d --date=short --format='%ad %ae %s' | sed 's/@chromium\.org//'
2015-02-06 maruel Further limit parallelism on 32 bits user process.

R=vadimsh@chromium.org
BUG=419862

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

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