GitLab

Skip making a pending tree, we'll want to wait for it anyways before
we draw in single-thread mode. This should bring the single-thread
impl-side painting in line with the legacy compositing mode when
using a single thread (for the browser compositor).

R=enne@chromium.org, vmpstr
BUG=441012

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

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

This adds perf tests for CSS and Web Animations over the following
CSS value types:
 - Color
 - Filter
 - Length
 - Length 3D
 - Shadow
 - Transform

BUG=309981

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

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

As the app is not added to the dock the current name is misleading.

BUG=459082

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

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

BUG=

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

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

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

ShelfLayoutManager has three methods from which the ShelfAlignment can be
applied. For supressed user sessions the shelf is forced to the bottom, however
this was only addressing locked screens, and adding new users.

When the first login occurs a different state transition takes place, where the
SessionState becomes active while the user session is still supressed.
ShelfLayoutManager was not applying alignment changes made during this time.

Update ShelfLayoutManager to unify the logic for overriding the user set
ShelfAlignment. Update this logic to account for having to apply alignment
during the initial login.

Manual testing was also done for: logging in; locking screen; unlocking screen;
adding a new user; switching users.

TEST=ShelfLayoutManagerTest.SideAlignmentInteractionWithLockScreen, ShelfLayoutManagerTest.SideAlignmentInteractionWithAddUserScreen, ShelfLayoutManagerTest.SideAlignmentInteractionWithLoginScreen
BUG=456199

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

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

This adds 'clang-newlib' as new possible value for
$TOOLCHAIN and enables building of most examples
and libraries with the new toolchain.

Initially adding without ARM support.  ARM support
will follow shortly.

Also, update buildbot_common.py to output shorter
filenames (making output easier to read).

BUG=454962

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

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

re-entrant WebContentsObserver events running during ExtensionHost destruction.
This also reverts the fix I made in crbug.com/456920 since this should cover it.

BUG=459239, 456920
R=mek@chromium.org

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

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

This emulates in C++ and in the GN tool what
 https://chromium.googlesource.com/chromium/src/+/master/build/landmines.py#82 does.

gn-dev mailing list discussion for more background is:
https://groups.google.com/a/chromium.org/d/topic/gn-dev/KTyLEh2Um98/discussion

The link to the doc proposing this command is:
https://docs.google.com/document/d/1DH-t-jYwQxZAXhqoBvnWGHWeF1z4Lltxny9D7lgC4HU/edit?pli=1

Tested with the following command lines:

$ gn gen out_gn_testing_clean_command/Debug --args='is_component_build=true'

$ out_gn/Debug/gn clean out_gn_testing_clean_command/Debug

Then:

$ ls out_gn_testing_clean_command/Debug
args.gn  build.ninja  build.ninja.d

To verify the files were written to disk correctly.

And:

$ cat out_gn_testing_clean_command/Debug/args.gn
$ cat out_gn_testing_clean_command/Debug/build.ninja
$ cat out_gn_testing_clean_command/Debug/build.ninja.d

To see if everything was written to the files correctly.

BUG=None
TEST=see above
R=brettw@chromium.org

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

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

Previously AudioPump would buffer infinite amount of audio data even
when the network is congested. With this changed the amount of data
it can buffer is limited to 5kB and it starts dropping packets after
that.

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

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

With this change failure to close UDP socket will crash the process.
This will help to debug bug 452121, but is also necessary to avoid
leaking file descriptors.

BUG=452121

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

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

BUG=457691

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

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

To improve performance, again we try to fetch all the info once through
an internal command glGetUniformsES3CHROMIUM.  Basically it gets all the
information that can be provided by glGetActiveUniformsiv except for those
that can already be queried through glGetActiveUniform, which will be cached
on the client side through internal command glGetProgramInfoCHROMIUM.

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

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

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

BUG=454500

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

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

Since we can now have active script restriction without the scripts-require-
action switch, we need to adjust the reporting criteria for scripts injecting.

BUG=452756

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

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

Per this patch, Web Notifications on Android will inherit the system
default settings for the notification sound, vibration and light
indicator. These cannot yet be changed by the developer, but can be
fully customized by the user in the Android Settings screen.

BUG=459268

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

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

To support Slimming Paint, page overlays are now expected to declare the size
of the content they draw to a canvas. This moves the two subclasses, both
solid-color overlays, to use the new interface.

Following this CL, the deprecated method will be removed on the Blink side.

BUG=450761

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

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

Revert of Remove dependency on content from remoting_host. (patchset #5 id:160001 of https://codereview.chromium.org/929493002/)

Reason for revert:
Compile failure on Linux ChromiumOS Ozone Builder:
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Ozone%20Builder/builds/11867

Original issue's description:
> Remove dependency on content from remoting_host.
>
> The host was depending on content just to access content::BrowserThread.
> It's cleaner to just pass the references to the threads explicitly.
>
> BUG=458581
>
> Committed: https://crrev.com/d5197934178ef0da1b630792cfbda8ddb6b34388
> Cr-Commit-Position: refs/heads/master@{#316648}

TBR=kelvinp@chromium.org,sergeyu@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=458581

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

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

BUG=None

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

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

BUG=none
TEST=Compile video_decode_accelerator_unittest with component build

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

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

This fixes and reenables liblouis chromevox tests.  Instead of relying on
the 'load' event of the nacl component, just queue up requests to
the component as soon as it is attached.  This simplifies the state management
in the wrapper.

As a bonus, the tests are fixed to use the appropriate callback
wrappers so that errors are immediately reported in failure cases
instead of making the tests painfully block and time out.

BUG=458910

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

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

I just discovered a source code spell checker tool called `codespell`.

BUG=

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

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

Fix for incorrect previous patch.

TBR=ajuma
BUG=458240

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

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

BUG=458939

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

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

user has opted-in to malware-extended-reporting.

BUG=447772,342617

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

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

Revert of [Android] Migrate DeviceUtils.GetMemoryUsageForPid to adb_wrapper. (patchset #5 id:80001 of https://codereview.chromium.org/924003003/)

Reason for revert:
Collision with https://codereview.chromium.org/929603002, see failurs on trybots like http://build.chromium.org/p/tryserver.chromium.linux/builders/android_rel_tests_recipe/builds/2499/steps/pylib_py_unittests/logs/stdio

Original issue's description:
> [Android] Migrate DeviceUtils.GetMemoryUsageForPid to adb_wrapper.
>
> BUG=267773
>
> Committed: https://crrev.com/bcec82d59869b43f0507d244e84798c6ce1cfcf9
> Cr-Commit-Position: refs/heads/master@{#316624}

TBR=perezju@chromium.org,jbudorick@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=267773

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

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

BUG=457393

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

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

The idea being that this make it easier to diagnose and fix issues like http://crbug.com/439598.

Sample run: http://simonhatch0.mon.corp.google.com:8013/builders/Android%20Nexus7v2%20Perf/builds/337/steps/host_info/logs/json.output

The output from this run on the Nexus 7v2 bots looks like:
{
  "failures": [
    "build_versions"
  ],
  "host_info": {
    "devices": {
      "build_types": "userdebug/dev-keys",
      "build_versions": "MISMATCH",
      "build_versions_list": [
        "4.4.2/KOT49H/937116",
        "5.0/LRX21P/1570855"
      ],
      "device_names": "google/razor/flo",
      "devices": [
        "09275b25",
        "09e0cda5",
        "09e19954",
        "09e19f78",
        "09e28a6c",
        "0a38ca3e",
        "0a9f170a",
        "0a9f1772"
      ]
    },
    "free_disk_space": 543.9997482299805,
    "num_cpus": 40,
    "os_release": "3.13.0-44-generic",
    "os_system": "Linux",
    "processor": "x86_64",
    "python_path": "/usr/bin/python",
    "python_version": "2.7.6"
  },
  "valid": false
}

This can be enabled by adding this to testing/buildbot/chromium.perf.json:
{
  "Android Nexus7v2 Perf": {
    "scripts": [
      {
        "name": "host_info",
        "script": "host_info.py"
      }
    ]
  }
}

Needs to land before enabling (that CL enables src-side tests for perf bots): https://codereview.chromium.org/873403002/

BUG=443708, 454934

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

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

GetLastEGLErrorString must be exported since is now used from content
since r310941 ("Reland: V4L2VDA: Generalize EGLImage import and query
driver for output formats.").

BUG=none
TEST=build CrOS chrome for link

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

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

This CL depends on the following one:
  https://codereview.chromium.org/914373003/

BUG=432527, 350378, 90795

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

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

The host was depending on content just to access content::BrowserThread.
It's cleaner to just pass the references to the threads explicitly.

BUG=458581

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

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

Partition sizes reside all together after the first partition in the streaim,
not at the beginning of each DCT partition.
Also, fix an erroneous DCHECK in V4L2SVDA.

BUG=chrome-os-partner:36432
TEST=media_unittest for VP8Parser,VP8 playback,vdaunittest

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

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

the old user gets hidden, which also hides all its
transient child windows. That will clear the transient
window's modal property and set it to a normal window -
which is incorrect.

Based on the comments above the code that clears the
window's modality state, we move this line out into
OnWindowDestroying().

BUG=447743

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

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

--self-destruct does not make sense for SxS since SxS cannot be
installed at system-level. --force-uninstall is useful for debugging and
testing, so there is no reason not to support it for SxS.

BUG=456602
R=gab@chromium.org

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

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

BUG=

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

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

This causes build failures in the aosp build because the aarch64 gold linker
is not present in the aosp tree.

BUG=440793

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

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

Rename SPDY tab to HTTP/2, and replace "SPDY" with "HTTP/2" on this tab wherever
it makes sense.  Do not rename internal strings (variable names and such).

BUG=456986

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

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

Outside tap should hide <select> dropdowns, but not dropdowns associated
with input fields. Tapping on an input field should not hide the
associated autofill or password generation popup.

BUG=430543

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

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

Given the code:
  scoped_ptr<std::string> x;
  LOG(ERROR) << x;
the logging output will always be 0 or 1. The reason is the
scoped_ptr<T> is getting turned into a bool before being logged. This
behavior doesn't seem very useful, so add a logging overload to make it
print out the pointer value.

BUG=none

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

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

Pulls the audio handling out of copresence and into its own component, with no functional changes. This is part of a larger plan to make audio token exchange available directly, via a private API.

rkc@ and xiyuan@ for copresence review.
caitkp@ for component review.
thestig@ for chrome/browser/DEPS review.
jam@, avi@ for content/ DEPS review.
dalecurtis@ for media/ DEPS review.
andrew@webrtc for WavWriter DEPS review.

BUG=455823

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

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