GitLab

BUG=322664

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271459 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271457 0039d316-1c4b-4281-b951-d872f2087c98

BUG=371805
TBR=mcasas@chromium.org
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271456 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271455 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271454 0039d316-1c4b-4281-b951-d872f2087c98

We recently switch to using glibc hooks for interception on glibc.
Where in the past calls like ki_accept and accept could be assumed to be the
same, now they rely on wrapping.

Changing inconsistent use of ki_ vs not in socket_test.

BUG=None
TEST=nacl_io_socket_test
R=binji@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271453 0039d316-1c4b-4281-b951-d872f2087c98

Reason for revert:
Gamepad list lookup causes a startup crash on Ice Cream Sandwich devices.  Please reland after adding the appropriate version check and testing locally (note: you don't need to get OWNERS approval a second time when relanding if you reupload to this codereview entry instead of creating a new one).

TBR=feng@chromium.org
NOTRY=true

Original issue's description:
> Gamepad API support for chrome on android
> 
> This change adds code to get gamepad data from java objects and
> provide these updates to the GamepadProvider which then writes to
> GamepadHardwareBuffer which in turn is read by SharedMemoryReader and
> later returned by JS to the web page.
> 
> Added singleton class GamepadsReader which is responsible for
> communication with java class and accessing gamepad data. It adds
> methods for communication with singleton java GamepadList class to
> get gamepads data.
> 
> This also adds new framework classes/methods required for Gamepad API
> support. Frameworks changes are responsible for :
>   - Identifying gamepad devices and their capabilities.
>   - Managing connected gamepad devices
>   - Map the connected gamepad devices to standard Gamepad format.
>   - Keeping gamepads axes/buttons data up-to-date and returning it to
>     native whenever requested. In android we cannot get gamepad data
>     directly from sources, so framework is modified to capture
>     gamepad key and motion events and extract gamepad data from these
>     events.
> 
> * Class GamepadPlatformDataFetcherAndroid : Android specific
>   implementation of gamepad data fetcher.
> 
> * Class ContentViewCore : Manages gamepad list and notifies of new
>   key/motion event for gamepads.
> 
> * Class GamepadList : A new class to manage connected gamepad devices
> 
> * Class GamepadDevice : A new class to manage information related to
>   each gamepad device.
> 
> * Class GamepadMappings : This class is responsible for mapping of
>   known gamepads to the standard gamepad.
> 
> This change enables gamepad API by default.
> 
> Adds support for parsing float array return type in JNI generator.
> 
> NVIDIA Shield and XBox360 gamepads are mapped to the standard gamepad
> 
> BUG=330094
> 
> TEST=http://www.html5rocks.com/en/tutorials/doodles/gamepad/gamepad-tester/tester.html
> 
> R=tsepez@chromium.org
> R=darin@chromium.org
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270620

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271452 0039d316-1c4b-4281-b951-d872f2087c98

The attempted fix only locked one side of the race, which isn't effective.

BUG=370244
TBR=ernstm@chromium.org
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271451 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271450 0039d316-1c4b-4281-b951-d872f2087c98

R=brettw@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271449 0039d316-1c4b-4281-b951-d872f2087c98

Revert of Add default options for ThreadSanitizer on Linux. Enable deadlock detection by default. (https://codereview.chromium.org/272763005/)

Reason for revert:
The linux_clan_tsan trybot caught a few deadlock reports that break memory bots after this change. 

e.g. http://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20Tests%20%28TSan%20v2%29%281%29/builds/9705/steps/media_unittests/logs/stdio


Original issue's description:
> Add default options for ThreadSanitizer on Linux. Enable deadlock detection by default.
> 
> BUG=372754,302040
> R=earthdok@chromium.org
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271412

TBR=earthdok@chromium.org,glider@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=372754,302040

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271447 0039d316-1c4b-4281-b951-d872f2087c98

Revert of Unconditionally include <string> and <iostream> in json_reader.cpp (https://codereview.chromium.org/291803003/)

Reason for revert:
This adds a static initializer.

Original issue's description:
> Unconditionally include <string> and <iostream> in json_reader.cpp
> Non-PNaCl builds that use libc++ also require these headers.
> 
> BUG=None
> R=mallinath@chromium.org
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271429

TBR=mallinath@chromium.org,ronghuawu@chromium.org,glider@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=None

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271446 0039d316-1c4b-4281-b951-d872f2087c98

- Added APIs for trackpad scroll begin/change/end events.
- Modified the API for momentum scroll begin to take additional parameters to
specify the deltaX and deltaY. Added the function MouseMomentumBegin2 to
accomplish this. This will be replaced by MouseMomentumBegin once the blink
change is landed.

BUG=330887

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271445 0039d316-1c4b-4281-b951-d872f2087c98

The V8 team signed off on removing this as well.

BUG=361460
TEST=none

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271444 0039d316-1c4b-4281-b951-d872f2087c98

BUG=94925
TBR=erg@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271442 0039d316-1c4b-4281-b951-d872f2087c98

BUG=374215
TEST=compiles
R=asvitkine@chromium.org, xiyuan@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271441 0039d316-1c4b-4281-b951-d872f2087c98

BUG=310373
TBR=brettw@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271440 0039d316-1c4b-4281-b951-d872f2087c98

BUG=374871
TBR=rtenneti@chromium.org
NOTRY=true

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271439 0039d316-1c4b-4281-b951-d872f2087c98

This CL addresses a potential problem due to a race: 
|suspend_observer_| [1] could have been deallocated in 
~AVFoundationMonitorImpl while still being in use 
in Device Thread [2]. This race didn't show up in normal
circumstances since it needed the Device Thread to
be active for a very short time; xians@ stumbled upon
it during some unrelated code activities that made a
browser_test fail.

This CL then cleans up the management of 
|suspend_observer_| from the  AVFoundationMonitorImpl
to a new class SuspendObserverDelegate, the former 
living exclusively in UI thread, the latter in Device 
Thread. SuspendObserverDelegate gets the Device
Thread parts of AVFoundationMonitorImpl.

The new class sits between AVFoundationMonitorImpl
and the Objective-C++ CrAVFoundationDeviceObserver.

[1] https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/device_monitor_mac.mm&sq=package:chromium&type=cs&l=259
[2] https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/device_monitor_mac.mm&sq=package:chromium&type=cs&l=336

BUG=288562, 372418

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271438 0039d316-1c4b-4281-b951-d872f2087c98

aura_unittests should run on linux and windows (including mac)
ash_unittests should run on chromeos and windows.

BUG=none
TEST=none
R=phajdan.jr@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271430 0039d316-1c4b-4281-b951-d872f2087c98

Non-PNaCl builds that use libc++ also require these headers.

BUG=None
R=mallinath@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271429 0039d316-1c4b-4281-b951-d872f2087c98

BUG=94925
TBR=erg@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271428 0039d316-1c4b-4281-b951-d872f2087c98

Since narrowing conversion in the `{}` is ill-formed in C++11, it raises
compile error with G++ 4.8.2.

BUG=

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271427 0039d316-1c4b-4281-b951-d872f2087c98

BUG=371822

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271426 0039d316-1c4b-4281-b951-d872f2087c98

This is so I can introduce a new way to do PPP interfaces that are
passed to a related PPB interface, instead of using PPP_GetInterface.

Note this doesn't add support for the PNaCl shim yet, although it should
at least make it compile.

BUG=367896

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271425 0039d316-1c4b-4281-b951-d872f2087c98

This CL moves the UserContext from user.[cc|h] to its own header and
implementation files.

BUG=367847
TEST=Updated unit and browser tests

TBR=nkostylev (chrome/browser/chromeos/app_mode/kiosk_profile_loader.cc)
TBR=nkostylev (chrome/browser/chromeos/chrome_browser_main_chromeos.cc)
TBR=pamg (chrome/browser/managed_mode/chromeos/manager_password_service.[cc|h])

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271424 0039d316-1c4b-4281-b951-d872f2087c98

BUG=374297
TBR=mpearson@chromium.org,jhawkins@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271423 0039d316-1c4b-4281-b951-d872f2087c98

This is needed to provide a ContextFactory to the Compositor so that
we can create Compositors on multiple threads.

BUG=none
TEST=none
R=ben@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271422 0039d316-1c4b-4281-b951-d872f2087c98

BUG=369531
TEST=everything still works

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271421 0039d316-1c4b-4281-b951-d872f2087c98

BUG=365927

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271420 0039d316-1c4b-4281-b951-d872f2087c98

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271419 0039d316-1c4b-4281-b951-d872f2087c98

This was the only CL in this build:

Test failures: http://build.chromium.org/p/chromium.mac/builders/Mac%2010.7%20Tests%20%28dbg%29%281%29/builds/23684

browser_tests browser_tests browser_tests 381 disabled 5 flaky failed 5 ( 24 mins, 12 secs )
stdio
PRE_ThemeDataPackInvalid
ThemeDataPackInvalid
ContextMenusAPI_Basic
Shim_TestNewWindow
SpeechRecognitionAPI_HasPermissionDeny

> Blink roll 174243:174291
> 
> http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?url=/trunk&range=174243:174291&mode=html
> TBR=
> BUG=
> 
> Review URL: https://codereview.chromium.org/291923002

TBR=eseidel@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271418 0039d316-1c4b-4281-b951-d872f2087c98

Store a stacktrace of PrefService destruction in PrefChangeRegistrar to debug a
race condition: ~PrefChangeRegistrar causes a crash. The current hypothesis is
that this is caused because the PrefChangeRegistrar is registered to a Profile
that is already destroyed. This code stores stacktrace when the Profile's
PrefService is destroyed to see whether this is correct.

BUG=373435

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271416 0039d316-1c4b-4281-b951-d872f2087c98

Several systems in chromium (about:tracing, some code in blink) use
tcmalloc's heap profiler, so keep most of it around.  Do disable support
for the HEAPPROFILE envvar, and for the exit-time dump to "Exiting" (unless in
profiling=1 builds), as that causes a static initializer.

BUG=94925
R=willchan@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271415 0039d316-1c4b-4281-b951-d872f2087c98

first_notification_ was getting reset on each notification in UpdateWatch,
causing the filtering to be less effective than it should have been.


BUG=NONE

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271414 0039d316-1c4b-4281-b951-d872f2087c98

 * Allow BeginFrameArgs inside an EXPECT_EQ call.
    EXPECT_EQ(args1, args2)

 * Allow gtest to pretty print BeginFrameArgs objects.

   Output use to look like;
---------------------------------------------------
    Expected: args1
    Which is:
      16 byte object <0xFF 0xFF 0xFF 0x01 ...>

    Actual: args2
    Which is:
      16 byte object <0xFF 0xFF 0xFF 0x01 ...>
---------------------------------------------------

   Output now looks like;
---------------------------------------------------
    Expected: args1
    Which is:
     BeginFrameArgs(1, 2, 3)

    Actual: args2
    Which is:
     BeginFrameArgs(1, 2, 4)
---------------------------------------------------

BUG=371223

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271413 0039d316-1c4b-4281-b951-d872f2087c98

BUG=372754,302040
R=earthdok@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271412 0039d316-1c4b-4281-b951-d872f2087c98

Creates asynchronous versions of SetCookie, removeAllCookie and
removeSessionCookie which take callbacks.

BUG=
TEST=AndroidWebviewTest

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271411 0039d316-1c4b-4281-b951-d872f2087c98

Promises are now enabled by default

BUG=none
R=adamk@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271410 0039d316-1c4b-4281-b951-d872f2087c98

This includes the toolbar itself, the browser actions area, the location bar,
and location bar components like the EV bubble view.  The omnibox itself (i.e.
the editable control) is given a minimum width of 150 px (which matches the EV
bubble min width).

* Also fixes a bug where the EV bubble was not allowed to shrink as intended.
* Also fixes a bug where the toolbar requested 3px too much size by default when
  the "toolbar origin chip" was enabled.

BUG=58915
TEST=Try to shrink the browser width, make sure the omnibox never gets "too narrow"

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271409 0039d316-1c4b-4281-b951-d872f2087c98