GitLab

Apparently libtool on Mac doesn't understand the "@" syntax.

BUG=
R=thakis@chromium.org

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

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

* Remove null check
* Simplify NestedAcceleratorDelegate interface.

BUG=None
R=sky@chromium.org

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

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

https://codereview.chromium.org/291933002 solved
these suppressions.

BUG=364675,158510

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

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

This breaks the remaining dependency from src/extensions to chrome resources files.

BUG=368334

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

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

Currently, subprocess_mu_ is held the entire time the ld subprocess is
starting. This blocks a larger refactor that I'd like to make as part of nexe
loading.

I'd like to change ServiceRuntime::LoadModule() to be asynchronous to simplify
threading behavior on the plugin side when loading nexe modules. However, this
requires that many methods are made asynchronous, including
Plugin::LoadHelperNaClModule().

PnaclTranslateThread will need to use a pattern similar to WaitForSelLdrStart()
to resume execution on a background thread. However, I don't like the idea of
introducing another mutex and condvar to deal with while holding
subprocess_mu_.

This change narrows the time that subprocess_mu_ is held to make that refactor
possible. This follows a similar change made for the llc subprocess.

BUG=333950
R=bbudge@chromium.org

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

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

Broke BookmarkAppHelperExtensionServiceTest.CreateBookmarkApp on Linux Clang (dbg) and other bots: 
http://build.chromium.org/p/chromium.linux/buildstatus?builder=Linux%20Clang%20%28dbg%29&number=60356 

[ RUN ] BookmarkAppHelperExtensionServiceTest.CreateBookmarkApp 
[26813:26813:0603/095344:697268828:FATAL:render_process_host_impl.cc(501)] Check failed: is_self_deleted_. RenderProcessHostImpl is destroyed by something other than itself 
#0 0x7fb30098e8de base::debug::StackTrace::StackTrace() 
#1 0x7fb300a21b85 logging::LogMessage::~LogMessage() 
#2 0x7fb308595006 content::RenderProcessHostImpl::~RenderProcessHostImpl() 
#3 0x7fb3085955a9 content::RenderProcessHostImpl::~RenderProcessHostImpl() 
#4 0x000000eef1df (anonymous namespace)::BookmarkAppHelperExtensionServiceTest::TearDown() 
#5 0x0000039dd3c3 testing::internal::HandleSehExceptionsInMethodIfSupported\u003C>() 
#6 0x0000039ca68e testing::internal::HandleExceptionsInMethodIfSupported\u003C>() 
#7 0x0000039c159a testing::Test::Run() 
#8 0x0000039c1c6b testing::TestInfo::Run() 
#9 0x0000039c225a testing::TestCase::Run() 
#10 0x0000039c6868 testing::internal::UnitTestImpl::RunAllTests() 
#11 0x0000039d60d3 testing::internal::HandleSehExceptionsInMethodIfSupported\u003C>() 
#12 0x0000039cc26e testing::internal::HandleExceptionsInMethodIfSupported\u003C>() 
#13 0x0000039c64f4 testing::UnitTest::Run() 
#14 0x00000594af51 RUN_ALL_TESTS() 
#15 0x000005949ff7 base::TestSuite::Run() 
#16 0x00000595dc4d content::UnitTestTestSuite::Run() 
#17 0x000003a80772 base::internal::RunnableAdapter\u003C>::Run() 
#18 0x000003a806dc base::internal::InvokeHelper\u003C>::MakeItSo() 
#19 0x000003a8068a base::internal::Invoker\u003C>::Run() 
#20 0x000002b6f0de base::Callback\u003C>::Run() 
#21 0x00000594404e base::(anonymous namespace)::LaunchUnitTestsInternal() 
#22 0x000005943d40 base::LaunchUnitTests() 
#23 0x000003a80450 main 
#24 0x7fb2f7c6076d __libc_start_main 
#25 0x000000710379 \u003Cunknown> 

> Add generateAppForLink function in chrome.management
> 
> This function takes a url and a title as input, generates and
> returns a bookmark app.
> 
> R=benwells@chromium.org,kalman@chromium.org,calamity@chromium.org
> BUG=370350
> 
> Review URL: https://codereview.chromium.org/266353006

TBR=wjywbs@gmail.com

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

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

BUG=379128

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

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

This function takes a url and a title as input, generates and
returns a bookmark app.

R=benwells@chromium.org,kalman@chromium.org,calamity@chromium.org
BUG=370350

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

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

BUG=368495
TBR=kbr@chromium.org

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

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

As discussed with cpu@, transition the ownership to waffles@.

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

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

This interface will be used by controller's UI to pass through pairing.

BUG=375191

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

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

Fix for https://codereview.chromium.org/293693003

BUG=369409
TBR=oshima@chromium.org

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

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

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

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

BUG=

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

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

Create AccountServiceFlagFetcher which downloads an account's Gaia service flags, using https://accounts.google.com/GetUserInfo.

BUG=372381

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

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

If a commit aborts, then PictureLayer::update_source_frame_number_ will
be stale since only a finished commit would update the
LTH::source_frame_number_.  This could lead to an issue where a commit
aborts and then on the next frame a layer's size is updated *and* it is
set to not draw.  This causes it to skip being updated (meaning the pile
size is not updated) but the layer bounds have changed and trips an
assert in PushPropertiesTo that tries to make sure that the pile and
layer bounds are the same when a layer has been updated that frame.

The fix here is to update the source frame number even for aborted
commits.  This causes the PushPropertiesTo assert to properly be
ignored.

BUG=375675

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

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

BUG=178409
TBR=phajdan.jr@chromium.org
NOTRY=true

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

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

BUG=379986

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

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

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

This change pulls out more OpenManifestEntry code from service_runtime, making
PPB_NaCl_Private a smallter interface, and preparing for pulling all of that
logic out of the trusted plugin.

BUG=239656
R=dmichael@chromium.org

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

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

This CL introduces ProfilerMetricsProvider, which takes the profiler data
recording responsibilities that were previously part of MetricsLog.
ProfilerMetricsProvider records profiler data into a cached proto and then
merges this data into the actual UMA proto in RecordGeneralMetrics().

BUG=379881

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

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

R=rmcilroy@chromium.org, viettrungluu@chromium.org

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

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

The version script was being ignored in non-gcc (clang) builds.

BUG=313751
TBR=glider@chromium.org
NOTRY=true

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

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

BUG=380081

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

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

BUG=378713
TEST=manual test:
1Launch chrome and navigate to chrome://webrtc-internals/. 
2.Expand "Create Dump" option >> Check the checkbox option of "enable Diagnostic audio recordings". 
3.With "Save File" overlay opened, again uncheck the same checkbox.

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

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

This refactors GetDeviceIdentifier function into GetDeviceIdentifier
and GetSaltedString. GetSaltedString is a utility function that can
be used elsewhere.

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

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

BUG=376860

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

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

This CL changes the Enumeration to be based on 
a callback passed to the VideoCaptureDeviceFactory.
It is used for QTKit enumeration on UI thread:
this is speculatively better than trying to run
-performSelectorOnMainThread:withObject:waitUntilDone:

-- Description --

Currently VCM::EnumerateDevices() issues an internal 
PostTaskAndReplyWithResult where the first part
is GetAvailableDevicesInfoOnDeviceThread() on 
DeviceThread and the second is 
OnDevicesInfoEnumerated() in IO thread.

This CL changes this to a simple PostTask to the 
VCDFactory::EnumerateDeviceNames() with a callback
to VCM::ConsolidateDevicesInfoOnDeviceThread().
This method has a jump at the end to 
VCM::OnDevicesInfoEnumerated() on IO thread. So
what used to be a simple back-and-forth jump is
now a 3 hop process. 

This is used for QTKit to be able to override the
VCDF::EnumerateDeviceNames and run the device
enumeration in UI thread. 

Note 1: The UI thread is currently passed in Create()
method for all implementations. This is changed to
earlier: on VCDF::CreateFactory(); injected from 
MediaStreamManager; and only used in VCDFLinux 
(for ChromeOS) and in VCDFMac for QTKit enumeration.
This approach is cleaner since we don't send UI thread
references to derived classes that don't need it.

Note 2: VideoCaptureDeviceFactoryMac::Create()
used to have a search-and-find for |device_id| that
is removed; the search was a precaution for devices
that disappeared between first enumeration and use.
However, using a |device_name| that is gone would
just cause a fail on AllocateAndStart() anyhow, and
enumerating twice is very expensive.
 For compatibility reasons, this behaviour is kept
for AVFoundation devices and blacklisted QTKit
devices (that behave as a subset of AVF-type ones).



BUG=255552, 115327

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

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

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

Please reply to the V8 sheriff mvstanton@chromium.org in case of problems.

TBR=mvstanton@chromium.org
NOTRY=true

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

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

BUG=371824
TBR=blundell

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

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

BufferedDataSource should accept the 'url' and the 'CORS mode' parameters
in the constructor instead via the initialize() method. This patch makes
this change and now only the 'initializeCallBack' is passed to initialize()
method. Also now the WebMediaPlayerImpl directly queries the BufferedDataSource
whether the resource is local or not, instead of finding it out itself.

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

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

NOTRY=true

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

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

A optionally larger number of buffers makes recording audio in a pnacl module
more reliable when latency is not an issue.

BUG=330851

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

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

The InfoBarManager::Observer() is used instead.

GoogleURLTrackerMapEntry now makes actual calls on the InfoBarManager (to
register and unregister as an observer). This has some consequences:
- the |infobar_manager_| pointer is no longer const
- the unittest has to create actual instances of InfoBarManager instead of
using placeholder integers.

BUG=373243

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

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

BUG=379574

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

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

Mock: https://folio.googleplex.com/chrome-ux/mocks/064-chrome-away-from-chrome/supervised/mirror#%2FPlaypen_210514_mirror-05.png

BUG=372812, 310640

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

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

PreloadAddressValidator implements the same interface as Chromium's
current AddressValidator but calls libaddressinput AddressValidator to
perform the actual work, to simplify switching Chromium Autofill over to
using libaddressinput. All existing test cases are ported.

The suggestions functionality is currently stubbed out.

R=rouslan@chromium.org
BUG=

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

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

This CL do followings to ensure to update thumbnails after the app saves images.

* Add '?nocache=xxx' to FileEntry's URL to ignore browser caches.
* Stop to use drive thumbnails for files that are on the drive but cached
  locally. These files may be edited locally and be out of sync with the local
  cache.
* Specify the current time as modificationTime of the metadata that is
  dispatched when the edited file is saved.

BUG=378648
TEST=manually

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

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

Previously this had the default app icon (rounded rectangle).

In future we may show the icon of the specific apps that are running.

BUG=333429

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

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