GitLab

Reverting - still flaky. See http://crbug.com/303491#c104 for details.

> Re-enable PDFExtensionTest.BasicPlugin
> 
> This test was disabled because it was failing. A few changes have been made:
> -The page zoom cannot be assumed to be 100% on load as it will depend on the screen size being tested on. So testPageSize now doesn't make that assumption.
> -testAccessibilityWithPage was flaking because accessibility information was not always returned for the two text boxes. I spent a bit of time investigating, but couldn't work it out so I've left it partially disabled for now and filed crbug.com/378091.
> -The files have all been moved to a more appropriate location (in particular the test was moved out of the data directory).
> 
> BUG=303491
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273819
> 
> Review URL: https://codereview.chromium.org/290803007

TBR=raymes@chromium.org

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

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

BUG=304341

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

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

For consistency, has_next is changed to has_more. Also, has_more is easier to
understand in providing extensions context.

TEST=browser_tests, unit_test: *FileSystemProvider*
BUG=373165

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

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

The problem why it did not work is that the FIFO used in MSAP has a size smaller than 10ms buffer before, so it crashed before webrtc::AudioProcessing consumes it.

FYI, previously the APM in Chrome was only enabled for finch experiment on Canary, so the problem does not occur in stable and beta. And I recently enabled it as default.


BUG=378895
TEST= use 96000KHz as sample rate, go to apprtc.appspot.com/?&debug=loopback to make a call.

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

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

The changes are:
- Show a supervised user icon. Note that this has a different look than on CrOS, to fit in with the "lock" icon on desktop. (The final icon resource is still missing, but since this must land in time for M37 I'm already sending the code for review now, and will switch out the icon later.)
- Use the supervised user deletion warning text from CrOS also for supervised users on desktop.

Mocks at https://folio.googleplex.com/chrome-ux/mocks/064-chrome-away-from-chrome/supervised/mirror

BUG=372812, 310640

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

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

Providing extensions will very often mount provided file systems on startup.
When an extension is installed, the background is executed, and hence the file
system registered.

Since providing extensions should be event pages, they will also register
all of the request events, so the extension is woken up once there is a request
sent to it.

However, the background page is not run after a reboot. All of the registered
events are remembered in preferences, but mounted file systems not. As a result
after a reboot, the file systems are lost.

To overcome this issue, this CL introduces storing mounted file systems to
preferences, so they are remounted automatically after a reboot, once the
extensions are loaded. This is consistent with remembering registered
event handlers in preferences.

Note, that if the extension is gone after a reboot, then remounting will not
be performed, since it is done after the extension is loaded.

All of the mounted file systems are written to preferences during shutdown.

TEST=TBD
BUG=248427

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

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

Some methods not used in product code keep to call GetMetadataDatabase directly.


BUG=347425
TEST=./unit_tests --gtest_filter="DriveBackendSyncTest.*:SyncEngine*:Conflict*:RegisterApp*:RemoteTo*:LocalTo*:ListChange*:MetadataDatabase*:TaskDep*"

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

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

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.

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.
  - Responsible for communication with java class and accessing gamepad data.
  - It adds methods for communication with singleton java GamepadList class
to get gamepads data.

* 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/133943002

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

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

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

BUG=377383,373252
TBR=grunell@chromium.org

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

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

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

TBR=mvstanton@chromium.org

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

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

BUG=343891

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

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

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

Revert of Update V8 to version 3.27.17 (based on bleeding_edge revision r21586) (https://codereview.chromium.org/303263007/)

Reason for revert:
gn config doesn't work

Original issue's description:
> Update V8 to version 3.27.17 (based on bleeding_edge revision r21586)
> 
> R=mvstanton@chromium.org
> TBR=mvstanton@chromium.org
> BUG=none
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274195

TBR=mvstanton@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=none

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

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

> Depend on v8:v8 instead of v8:mksnapshot
> 
> BUG=none
> R=brettw@chromium.org
> 
> Review URL: https://codereview.chromium.org/302783009

TBR=jochen@chromium.org

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

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

BUG=none
R=brettw@chromium.org

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

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

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

R=mvstanton@chromium.org
TBR=mvstanton@chromium.org
BUG=none

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

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

BUG=n/a
TEST=n/a

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

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

are covered

This a preparatory CL to land https://chromiumcodereview.appspot.com/309693002/ which introduces new values for Blink cache policies.

BUG=376025

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

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

- Adhere to the 80-column limit
- Don't show the partition if there is no registrations nor versions.

BUG=358657

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

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

This test was disabled because it was failing. A few changes have been made:
-The page zoom cannot be assumed to be 100% on load as it will depend on the screen size being tested on. So testPageSize now doesn't make that assumption.
-testAccessibilityWithPage was flaking because accessibility information was not always returned for the two text boxes. I spent a bit of time investigating, but couldn't work it out so I've left it partially disabled for now and filed crbug.com/378091.
-The files have all been moved to a more appropriate location (in particular the test was moved out of the data directory).

BUG=303491

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273819

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

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

The icon should be left-aligned, to the left of the name. I refactored the
layout code of the autofill popup view to better support both image alignment,
as well as RTL alignment.

This CL depends on https://codereview.chromium.org/286243002/, which has not
yet landed.

See the associated bug for visual confirmation of the correctness of the layout
logic.

BUG=139154

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

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

This adds a tip on best practices to the documentation for getAuthToken,
and also updates the tip on launchWebAuthFlow to correct an inaccuracy.

BUG=372279

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

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

Convert the video app into a class. No functionality change.

BUG=171191
TEST=browser test passes.
R=mtomasz@chromium.org

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

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

Update last_modified when closing cache file.

BUG=346625
TEST=unit_tests

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

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

The browser side code of the new WebSocket stack needs to
know the ID of the render frame where the corresponding
WebSocketBridge is created in order to handle certificate errors.

This CL just ports the code used by the old WebSocket
implementation to new one.

Blink side change: https://codereview.chromium.org/304113002/

BUG=364361

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

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

Constants for https, http already exists in our_constants. This patch makes gurl use those constants instead of hrad-coded values. Also it adds constants for WS and WSS.

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

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

Webviews and OOP iframes now have non-empty parentId property pointing to the target they are nested into. Also added 'webview' and 'iframe' target types.

TBR=bulach,mnaganov (for dev_tools_server.cc aw_dev_tools_server.cc)
BUG=378457

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

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

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

Re-enable the failing expectations for the Pixel.CSS3DBlueBox test.
The <meta viewport> tag added to this test is causing failures on some
desktop bots. See Issue 368495.

> Remove failing expectations for pixel tests.
> 
> This is a follow-up patch for r273755.
> 
> BUG=368495
> TBR=kbr@chromium.org
> 
> Review URL: https://codereview.chromium.org/304183008

TBR=alokp@chromium.org

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

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

Replace GetResourceListCallback with FileListCallback.
Rename GetAllResourceList and GetResourceListInDirectory to GetAllFileList and GetFileListInDirectory.
Rename FakeDriveService counter variables.
Remove GetOldestCreatedFolderResource in drive_backend_util.cc and merged the code to folder_creator.cc

BUG=231125
TEST=unit_tests

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

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

Before, the net::IOBuffer was being passed as a pointer, from the
FileStreamReader::Read to operations::ReadFile. However, there are PostTask
invocations on the way. When FileStreamReader is deleted after a PostTask is
fired from FileStreamReader::Read, and the operation is not completed, then
the net::IOBuffer will become invalid. When the operation is completed,
operation::ReadFile would cause a segmentation fault.

Since net::IOBuffer is ref counted, we should pass it as ref counted object
to be sure, that the buffer is always valid.

This patch migrates from passing IOBuffer as a pointer, to scoped_refptr.

TEST=Tested manually, that crashing doesn't occur anymore.
BUG=248427

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

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

Implement IsNonNativeLocalPathDirectory for supporting non-local non-Drive directory access in apps.

What the CL does is:
* Almost mechanically moves out the local function CheckIfDirectoryExists
from open_util.cc to fileapi_util.cc and just use it for the desired implementation.
* Removes Drive-specific helper code for the feature.
* Allows file_system_api test setup code to refer the info of File Manager app.
(Because the new general implementation checks the directory existence in the
permission level of the file manager, the test needs access to it.)

BUG=367028

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

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

Revert of Fix legacy dummy window visibility inheritance. RenderWidgetHostViewAura may be shown(::Show()) bef… (https://codereview.chromium.org/272173002/)

Reason for revert:
Regression: makes frameless packaged app windows
unmovable in Aero. Confirmed with local git revert.

BUG=374681
TBR=ben@chromium.org

Original issue's description:
> Fix legacy dummy window visibility inheritance. RenderWidgetHostViewAura may be shown(::Show()) before legacy window creation, in such case dummy window was later created hidden which leads to wrong behaviour. E.g. NPAPI Flash in windowless mode tracks associated window visibility and slows down draw invalidation.
> 
> R=ben@chromium.org
> BUG=
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270739

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

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

Changes:
1) Run clang-format through source and header files.
3) Fix data member variables to use unix_hacker_ style.
4) Rename file name to mock_web_user_media_client.
5) Rename class name to MockWebUserMediaClient.

BUG=331299
TEST=content_unittests, content_shell. No functional changes
TBR=dpranke@chromium.org

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

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

App launch times can (rarely) be equal, so the comparison should have
been less than or equal to.

BUG=379132
TEST=unit_tests (EphemeralAppServiceTest.*)

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

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

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

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

Prior to this change, the browser compositor allocated
buffers on the UI thread while the buffer allocation for
renderers was done on the IO thread.

This moves all allocation to the UI thread, which makes life
easier for GpuMemoryBuffer implementations and avoids
potentially expensive allocation costs on the IO thread.

BUG=

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

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