GitLab

WebGL and Canvas also should draw a fixed VideoFrame showing a Cast logo, which replaces the local video
frames in embedded <video> elements whenever a video is being played remotely.

TEST=Run http://browsertests.herokuapp.com/media/mdn_samples_webgl_sample8/index.html on Nexus 5.
BUG=284478
NOTRY=true

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

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

Rather than forwarding the touch event to the input router and incurring the
attendant overhead, short-circuit touch forwarding from ContentViewCoreImpl
when no touch handler exists.  This aligns with existing Aura behavior, and
saves ~20us/event when no touch handler exists.

BUG=341613

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

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

This is a follow-up to
https://src.chromium.org/viewvc/chrome?revision=256913&view=revision that makes
error messages more descriptive.

It was automatically produced with

git grep -l 'DCHECK(.*CurrentlyOn' extensions chrome/*/extensions|xargs sed -i -e 's/DCHECK(BrowserThread::CurrentlyOn(\([^)]*\)))/DCHECK_CURRENTLY_ON(\1)/' -e 's/DCHECK(content::BrowserThread::CurrentlyOn(\([^)]*\)))/DCHECK_CURRENTLY_ON(\1)/'

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

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

Remove outdated comment and enable OOM tests under sanitizers. Also, document
the fact that the newly introduced UncheckedMalloc()/UncheckedCalloc() don't
work as intended in sanitizer builds.

BUG=357732
R=thakis@chromium.org

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

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

This code has been unused for around 2 years.

BUG=355145

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

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

DevToolsAgentMsg_PauseWorkerContextOnStart was send to the worker process from the browser process in order to pause the worker context when it starts.
Before http://crbug.com/329786 this message was sent between WorkerProcessMsg_CreateWorker and WorkerMsg_StartWorkerContext.
So the worker was successfully paused before started.
But after http://crbug.com/329786 the worker context may be started before the worker process receives this message because WorkerMsg_StartWorkerContext is no longer sent from the browser process.
So we have to add pause_on_start flag to WorkerProcessMsg_CreateWorker_Params to pause the worker context correctly.

BUG=329786

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

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

This cl adds one UMA stats.
- SharedWorker.RendererSurviveForWorkerTime

BUG=327256

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

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

TBR=cbentzel@chromium.org, cpu@chromium.org
BUG=357915

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

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

We can re-enable them after the Shadow DOM construction
dust settles

TBR=rsadam
BUG=357946

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

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

36 is both the square of 6 and a triangular number, making it a square triangular number. It is the smallest square triangular number other than 1, and it is also the only triangular number other than 1 whose square root is also a triangular number.

36 is the number of degrees in the interior angle of each tip of a regular pentagram..

The number of possible outcomes (not summed) in the roll of two distinct dice.

The number of domino tilings of a 4×4 checkerboard is 36.

The sum of the integers from 1 to 36 is 666.

There are 36 inches in a yard.

36 is the number of characters required to store the display name of a UUID or GUID.

36 is the perfect score on the ACT.

Jewish tradition holds that the number 36 has had special significance since the beginning of time.

36 is the ASCII code for the dollar symbol.

R=dharani@google.com, laforge@chromium.org

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

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

"Make mediaPlayerRequestFullscreen() user gesture bypass explicit"
(Blink r170080) makes the setting redundant.

BUG=355439

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

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

R=darin@chromium.org

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

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

As of r259048, WebMediaPlayerImpl now notifies the compositor of new
frames directly on the compositing thread. If
VideoFrameProviderClientImpl::DidRecieveFrame() is called before
VideoLayerImpl::DidBecomeActive() is called,
VideoFrameProviderClientImpl may attempt to call
VideoLayerImpl::SetNeedsRedraw() using an uninitialized pointer.

BUG=357145
R=enne@chromium.org

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

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

It's impossible for VideoFrameCompositor to know why the compositor may not have requested the frame, so it shouldn't be counted towards dropped frames if the rest of the media stack was capable of producing and delivering frames on time.

BUG=335345
R=xhwang@chromium.org

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

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

No functionality change, just moving things around.
- Move functions so that it is more convenient to read .cc file top to bottom.
- replace ANDROID with OS_ANDROID. Looking at other places in the code OS_ANDROID is much more popular way to check for android.

R=rlarocque@chromium.org

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

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

The Service Worker can reject an install event by passing a promise to
waitUntil that is rejected. This patch allows handling for this situation.

This patch depends on the Blink change:
https://codereview.chromium.org/210833004/

BUG=285976
R=jam@chromium.org, kinuko@chromium.org, tkent@chromium.org, tsepez@chromium.org

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

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

(Explicitly disallow octal and hexadecimal ordinals.)

R=darin@chromium.org

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

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

BUG=347044
TEST=Manual, see bug.
NOTRY=true

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

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

Note the missing trailing comma in the second-last entry (invalidating the last
two entries). (zlib is allowed by chrome/browser/DEPS while ui is allowed by
chrome/DEPS.)

R=fischman@chromium.org

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

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

As we've observed a "cloned install" effect in the wild, 
doing this will de-clone such installs. (Which will work 
in conjunction with other measures to prevent new instances 
of cloned installs appearing). 

Renames client id and low entropy source prefs so that they 
are re-generated the next time Chrome runs. Makes the code 
clear the old prefs, so that they are cleaned up in 
existing local state files (in a future milestone, we can 
remove them completely - http://crbug.com/357704).

(Note: Does not rename kMetricsPermutedEntropyCache as 
that pref is reset when the low entropy id is re-generated.) 

BUG=357618
TEST=Running an UMA-enabled build should result in local state 
getting new values for user_experience_metrics.low_entropy_source2 
and user_experience_metrics.client_id2 and removal of prefs 
low_entropy_source and client_id.
NOTRY=true

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

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

When a cloned install is detected the metrics client id and low entropy source are reset on the next start of Chrome. This is done by setting a pref in local state. The next time either the client id or low entropy source are loaded from the local state, they are first reset.

BUG=343273
NOTRY=true

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

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

TBR=darin@chromium.org

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

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

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

Replace DCHECK(BrowserThread::CurrentlyOn) with DCHECK_CURRENTLY_ON in content/browser/renderer_host.

This is a follow-up to
https://src.chromium.org/viewvc/chrome?revision=256913&view=revision that makes
error messages more descriptive.

It was automatically produced with

git grep -l 'DCHECK(.*CurrentlyOn' content/browser/renderer_host/|xargs sed -i -e 's/DCHECK(BrowserThread::CurrentlyOn(\([^)]*\)))/DCHECK_CURRENTLY_ON(\1)/' -e 's/DCHECK(content::BrowserThread::CurrentlyOn(\([^)]*\)))/DCHECK_CURRENTLY_ON(\1)/'

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

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

This supports registration and unregistration from the UI thread.

BUG=355131

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

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

Click-to-play was broken when the plugin resizing code was
changed to update plugin style instead of width/height fields
directly. Problem is that 
remoting.ClientPlugin.showPluginForClickToPlay() was still using
trying to set width/height which are overridden by the CSS size.

NOTRY=true

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

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

SIMD requires aligned loads or it'll crash.

BUG=357840
TEST=new unittest. played video back in crashes.
R=rileya@chromium.org, scherkus@chromium.org
TBR=scherkus

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

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

BUG=357593
NOTRY=true

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

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

The download item code tried to be helpful and make sure Blink could
navigate to download items dropped in a web contents by setting a file
URL in the drop data. Unfortunately, this doesn't work because
RenderViewHostImpl::DragTargetDragEnter filters out file URLs and
rewrites it to about:blank, and Blink happily navigates there instead of
the file.

As it turns out, this is unnecessary because Blink already converts the
filename to a URL when it attempts to turn drops into navigations so we
get this behavior for free by omitting the call to SetURL.

It also confuses some Linux file managers, causing them to link to the
file instead of making a copy when the item is dragged from the download
bar.

BUG=356794
TEST=drag download bar items to the desktop (should copy) and to another
     Chrome window (should navigate)

R=thakis@chromium.org

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

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

This fixes a bug where we failed to set the proper channel count on
AudioBuffers with the CHANNEL_LAYOUT_DISCRETE channel layout.

BUG=357801
R=dalecurtis@chromium.org
TBR=dmichael

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

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

NOTRY=true

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

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

This patch allows using OutputConfigurator and helpers to support configration on platforms other than X11 (such as Ozone).

This CL depends on https://codereview.chromium.org/214513006/

BUG=333413

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

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

Undoing the revert, since the MultiProfileFileManagerBrowserTest failure was
caused by Blink 170311 instead.  Sorry for the disruption.

> Revert 260198 "[VideoPlayer] Browser tests for new separated vid..."
> 
> I suspect it was the cause of these file_manager_browsertest.cc failures:
> http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%282%29/builds/22673
> 
> > [VideoPlayer] Browser tests for new separated video player app
> > 
> > BUG=357621
> > TEST=manually tested
> > 
> > R=hirono@chromium.org
> > TBR=arv@chromium.org
> > # TBRing for adding a resource.
> > 
> > Review URL: https://codereview.chromium.org/209853011
> 
> TBR=yoshiki@chromium.org
> 
> Review URL: https://codereview.chromium.org/216183006

TBR=yoshiki@chromium.org

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

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

Undoing the revert, since MultiProfileFileManagerBrowserTest failure was
caused by Blink 170311 instead.  Sorry for the disruption.

> Revert 260222 "Separate audio player tests from special file tes..."
> 
> Still trying to track down cause of MultiProfileFileManagerBrowserTest failures:
> http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%282%29/builds/22674
> 
> 
> > Separate audio player tests from special file test of Files.app (2nd)
> > 
> > I have a plan to add more audio player tests. Before that, make audio player tests
> > separated for readability.
> > 
> > This is the 2nd (last) part of the work of crrev.com/214463002. First part is committed as crrev.com/260069.
> > 
> > BUG=none
> > TEST=browser_test passes
> > TBR=hirino@chromium.org
> > 
> > Review URL: https://codereview.chromium.org/212123008
> 
> TBR=yoshiki@chromium.org
> 
> Review URL: https://codereview.chromium.org/217743002

TBR=yoshiki@chromium.org

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

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

BUG=322664

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

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

R=markusheintz@chromium.org
BUG=356741

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

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

R=sky@chromium.org

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

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

It works for me on my Linux box. It only supports binary tests, not
Python. :/

R=darin@chromium.org

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

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

1. Rename a folder name if it is a normal folder.
2. Maximum folder name length.
3. Folder name not editable for oem folder.

BUG=354627
R=xiyuan@chromium.org

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

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

BUG= 357703

TESTED=try jobs passed
NOTRY=true

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

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