- 22 Aug, 2014 40 commits
-
-
rdevlin.cronin@chromium.org authored
BUG=166959 Review URL: https://codereview.chromium.org/495193004 Cr-Commit-Position: refs/heads/master@{#291410} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291410 0039d316-1c4b-4281-b951-d872f2087c98
-
xiyuan@chromium.org authored
Bluetooth detection on Mac triggers 500ms adapter polling and causes performance regression. Since Easy unlock is only offered on ChromeOS at the moment, disable Bluetooth detection for other platforms while the polling problem is investigated. BUG=404482,399067 Review URL: https://codereview.chromium.org/498453002 Cr-Commit-Position: refs/heads/master@{#291409} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291409 0039d316-1c4b-4281-b951-d872f2087c98
-
bartfab@chromium.org authored
If the recommended locales for a public session are invalid/unavailable, public sessions should default to the current UI language, not the locale that happens to be first in alphabetic order (Arabic for Chrome OS running in English). BUG=403550 TEST=Extended unit and browser tests Review URL: https://codereview.chromium.org/483523005 Cr-Commit-Position: refs/heads/master@{#291408} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291408 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
Cr-Commit-Position: refs/heads/master@{#291407} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291407 0039d316-1c4b-4281-b951-d872f2087c98
-
derat@chromium.org authored
BUG=403571 TBR=davemoore@chromium.org Review URL: https://codereview.chromium.org/470953005 Cr-Commit-Position: refs/heads/master@{#291406} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291406 0039d316-1c4b-4281-b951-d872f2087c98
-
sky@chromium.org authored
BUG=392974 TEST=none R=darin@chromium.org Review URL: https://codereview.chromium.org/480663003 Cr-Commit-Position: refs/heads/master@{#291405} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291405 0039d316-1c4b-4281-b951-d872f2087c98
-
mkwst@chromium.org authored
TBR=finnur@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/500543002 Cr-Commit-Position: refs/heads/master@{#291404} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291404 0039d316-1c4b-4281-b951-d872f2087c98
-
weiliangc@chromium.org authored
Move RenderPass::Id out of RenderPass and make it an isolated class called RenderPassId, so RenderPassDrawQuad and AppendQuadsData will not need to depend on entire RenderPass class. BUG=344962 Review URL: https://codereview.chromium.org/404563005 Cr-Commit-Position: refs/heads/master@{#291403} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291403 0039d316-1c4b-4281-b951-d872f2087c98
-
grunell@chromium.org authored
The stats is intended for the low latency case, hence removing other cases. NOTRY=true Review URL: https://codereview.chromium.org/486633003 Cr-Commit-Position: refs/heads/master@{#291402} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291402 0039d316-1c4b-4281-b951-d872f2087c98
-
avi@chromium.org authored
Revert of Keep a copy of page id in RenderViewHost. (patchset #1 of https://codereview.chromium.org/493853002/) Reason for revert: This hit canary and we're getting instrumented crashes. That's what we were looking for so let's back it out. Original issue's description: > Keep a copy of page id in RenderViewHost. > > This is an instrumented version of the patch that will be reverted in a few days. > This is meant to catch crashes in edge cases and log enough for us to repro > them. > > BUG=99379, 369661 > TEST=this is gonna crash for a few people > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=291077 TBR=rsesek@chromium.org,creis@chromium.org NOTREECHECKS=true NOTRY=true BUG=99379, 369661 Review URL: https://codereview.chromium.org/500533002 Cr-Commit-Position: refs/heads/master@{#291401} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291401 0039d316-1c4b-4281-b951-d872f2087c98
-
pkotwicz@chromium.org authored
BUG=405957 TEST=SplitViewControllerTest.SplitModeActivation Review URL: https://codereview.chromium.org/488153003 Cr-Commit-Position: refs/heads/master@{#291400} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291400 0039d316-1c4b-4281-b951-d872f2087c98
-
dconnelly@chromium.org authored
BUG=328847 Review URL: https://codereview.chromium.org/422233009 Cr-Commit-Position: refs/heads/master@{#291399} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291399 0039d316-1c4b-4281-b951-d872f2087c98
-
vasilii@chromium.org authored
Credential Management API is Javascript API that can be used by sites to sign user in or correctly store the credentials in the password manager. BUG=400674 Review URL: https://codereview.chromium.org/459103005 Cr-Commit-Position: refs/heads/master@{#291398} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291398 0039d316-1c4b-4281-b951-d872f2087c98
-
finnur@chromium.org authored
Revert 291214 "Fixes three crashes" in an attempt to find the cause of Athena related unit test failures on the ASAN bot. http://build.chromium.org/p/chromium.memory/buildstatus?builder=Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%283%29&number=2862 > Fixes three crashes > * AppListViewDelegate was accessing deleted search_provider. > * DeviceSocketListener::StopListening can happen after DeviceSocketManager is deleted > * Explicitly delete FilePathWatcher. This was causing > recursive callback to FilePathWatcherImpl::Cancel from > FilePathWatcherImpl::CancelOnMessageLoopThread, which > caused crash. > > > Clean ups > * change OnIO to OnFILE as they run on FILE thread. > * removed unused singleton related code. > > BUG=None > TEST=Run athena_main on desktop and close window. > > Review URL: https://codereview.chromium.org/490033003 TBR=oshima@chromium.org Review URL: https://codereview.chromium.org/467003003 Cr-Commit-Position: refs/heads/master@{#291394} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291394 0039d316-1c4b-4281-b951-d872f2087c98
-
mlamouri@chromium.org authored
The manifest_parser.{h,cc} files are colliding with new files I'm creating for the Web Manifest feature. Appcache manifest files should be prefixed by "appcache". BUG=None NOTRY=true TBR=torne (for third_party_files_whitelist.txt) Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=291388 Review URL: https://codereview.chromium.org/469413003 Cr-Commit-Position: refs/heads/master@{#291393} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291393 0039d316-1c4b-4281-b951-d872f2087c98
-
rob@robwu.nl authored
Include the stack trace of the code that triggered the error. Refactored the error generation method to prevent extension code from breaking the state of the internals. BUG=404406 Review URL: https://codereview.chromium.org/482603002 Cr-Commit-Position: refs/heads/master@{#291392} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291392 0039d316-1c4b-4281-b951-d872f2087c98
-
pauljensen@chromium.org authored
Fails check_licenses > Rename appcache/manifest_parser* to appcache/appcache_manifest_parser*. > > The manifest_parser.{h,cc} files are colliding with new files I'm > creating for the Web Manifest feature. Appcache manifest files should > be prefixed by "appcache". > > BUG=None > NOTRY=true > > Review URL: https://codereview.chromium.org/469413003 TBR=mlamouri@chromium.org Review URL: https://codereview.chromium.org/493383003 Cr-Commit-Position: refs/heads/master@{#291391} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291391 0039d316-1c4b-4281-b951-d872f2087c98
-
mkwst@chromium.org authored
This patch does three things: 1. Implements the Blink platform interface WebCredentialManagerClient by adding a CredentialManagerClient class to the password manager component. 2. Wires that implementation up to Blink's platform layer by creating an instance of the new client, held in ChromeContentRendererClient, and setting it as each new RenderView's client. 3. Stubs out IPCs for the renderer to pass messages up to the browser in order to do the heavy lifting of actually answering Blink's requests by generating and delivering Credential objects. BUG=400674 TBR=tkent@chromium.org Review URL: https://codereview.chromium.org/464883002 Cr-Commit-Position: refs/heads/master@{#291390} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291390 0039d316-1c4b-4281-b951-d872f2087c98
-
dconnelly@chromium.org authored
Screenshots available on the bug. BUG=394303 Review URL: https://codereview.chromium.org/487193003 Cr-Commit-Position: refs/heads/master@{#291389} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291389 0039d316-1c4b-4281-b951-d872f2087c98
-
mlamouri@chromium.org authored
The manifest_parser.{h,cc} files are colliding with new files I'm creating for the Web Manifest feature. Appcache manifest files should be prefixed by "appcache". BUG=None NOTRY=true Review URL: https://codereview.chromium.org/469413003 Cr-Commit-Position: refs/heads/master@{#291388} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291388 0039d316-1c4b-4281-b951-d872f2087c98
-
n.bansal@samsung.com authored
Currently setZoom() doesn't check whether zoom factor provided falls within range or not. It is possible to pass big zoom level through 'zoom' open pdf parameter which can make browser unresponsive. This patch updates the zoom factor provided in setZoom() to fall within range. BUG=303491 Review URL: https://codereview.chromium.org/475933004 Cr-Commit-Position: refs/heads/master@{#291387} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291387 0039d316-1c4b-4281-b951-d872f2087c98
-
jansson@chromium.org authored
Added warning_ function that highlights messages in orange in the debug message panel. MediaStreamTrack.onmute and MediaStreamTrack.onunmute are now highlighted in red as an error and in orange as a warning. BUG=None TEST=Manually triggered the events directly. NOTRY=TRUE Review URL: https://codereview.chromium.org/493073002 Cr-Commit-Position: refs/heads/master@{#291386} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291386 0039d316-1c4b-4281-b951-d872f2087c98
-
hirono@chromium.org authored
The class is extracted from EventRouter of Files.app. It is going to manage device states and dispatch the device events. BUG=360946, 396258 TEST=add unit tests Review URL: https://codereview.chromium.org/472603004 Cr-Commit-Position: refs/heads/master@{#291384} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291384 0039d316-1c4b-4281-b951-d872f2087c98
-
earthdok@chromium.org authored
BUG=354881 TBR=ygorshenin@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/494223003 Cr-Commit-Position: refs/heads/master@{#291383} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291383 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-devtools://devtools/remote/opendgozman@chromium.org authored
BUG=370025 Review URL: https://codereview.chromium.org/489893002 Cr-Commit-Position: refs/heads/master@{#291382} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291382 0039d316-1c4b-4281-b951-d872f2087c98
-
mtomasz@chromium.org authored
Closing a file should not be abortable, since we could end up on files which once opened, are never closed. TEST=browser_tests: *FileSystemProvider*WriteFile* BUG=405821 Review URL: https://codereview.chromium.org/497673002 Cr-Commit-Position: refs/heads/master@{#291381} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291381 0039d316-1c4b-4281-b951-d872f2087c98
-
earthdok@chromium.org authored
BUG=405985 TBR=phajdan.jr@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/485893009 Cr-Commit-Position: refs/heads/master@{#291380} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291380 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
Cr-Commit-Position: refs/heads/master@{#291379} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291379 0039d316-1c4b-4281-b951-d872f2087c98
-
machenbach@chromium.org authored
TBR=yangguo@chromium.org Review URL: https://codereview.chromium.org/495313002 Cr-Commit-Position: refs/heads/master@{#291378} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291378 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
Cr-Commit-Position: refs/heads/master@{#291377} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291377 0039d316-1c4b-4281-b951-d872f2087c98
-
blink-deps-roller@chromium.org authored
http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?url=/trunk&range=180754:180760&mode=html TBR=wibling@chromium.org,gavinp@chromium.org BUG= Review URL: https://codereview.chromium.org/498663002 Cr-Commit-Position: refs/heads/master@{#291376} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291376 0039d316-1c4b-4281-b951-d872f2087c98
-
sanjoy.pal@samsung.com authored
BUG=406236 Review URL: https://codereview.chromium.org/485103005 Cr-Commit-Position: refs/heads/master@{#291375} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291375 0039d316-1c4b-4281-b951-d872f2087c98
-
wuchengli@chromium.org authored
- sizeimage DCHECK: The video device can adjust the buffer size to be bigger than we need as long as the data can fit. Theoretically sizeimage should be bigger than PlaneAllocationSize. But Exynos driver has a 256 byte padding for sizeimage. It is rounded up when calculating coded_size. So sizeimage can be smaller than PlaneAllocationSize. For example, say format.fmt.pix_mp.width and height are 1920x1088. format.fmt.pix_mp.plane_fmt[0].sizeimage is 2089216 (1920*1088+256). |sizeimage| is rounded up and |coded_size| becomes 1920x1089. So format.fmt.pix_mp.plane_fmt[i].sizeimage (2089216) is less than PlaneAllocationSize (1920*1090=2092800). Also, we should not round down because the coded size is passed VideoEncodeAccelerator::Client::RequireBitstreamBuffers. The client will allocate buffers according to the coded size. - bytesperline DCHECK: bytesperline of different planes should be checked against media::VideoFrame::row_bytes, not the width of coded size. BUG=387701 TEST=Run Hangout in debug builds. Review URL: https://codereview.chromium.org/406893002 Cr-Commit-Position: refs/heads/master@{#291374} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291374 0039d316-1c4b-4281-b951-d872f2087c98
-
jamesr@chromium.org authored
R=viettrungluu@chromium.org Review URL: https://codereview.chromium.org/495223002 Cr-Commit-Position: refs/heads/master@{#291373} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291373 0039d316-1c4b-4281-b951-d872f2087c98
-
owenlin@chromium.org authored
It was introduced by http://codereview.chromium.org/426873004/ BUG=406160 TEST=Run the VDA_Unittest on ARM CrOS device Review URL: https://codereview.chromium.org/483093005 Cr-Commit-Position: refs/heads/master@{#291372} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291372 0039d316-1c4b-4281-b951-d872f2087c98
-
yoshiki@chromium.org authored
Show "This video could not be loaded" error on playing unsupported video. BUG=405860 TEST=manually testted Review URL: https://codereview.chromium.org/495003002 Cr-Commit-Position: refs/heads/master@{#291371} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291371 0039d316-1c4b-4281-b951-d872f2087c98
-
finnur@chromium.org authored
Reason: Failure on ASAN build bots. https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20(3)/builds/2854/steps/athena_unittests/logs/OneAppActivity > Athena: Adding basic resource management framework (un-/re-loading) of V2 applications > > Functionality: > > The |AppRegistry| has for each running application an |AppActivityRegistry|. > > The |AppActivityRegistry| knows all activities associated with the application it represents. > It can furthermore shut the app entirely down upon resource manager request. It will then create > an |AppActivityProxy| for the overview mode which shows a placeholder for an unloaded app. This > placeholder can then ask the |AppActivityRegistry| to restart the application again. > > A shutdown request for the application is only performed when all activities were marked for > UNLOAD. > > If there were multiple activities upon shutdown for one app, the app has to take care of > re-creating all windows and thus re-creating all activities. Since an activity match cannot > be performed, the |AppActivityProxy| will only be shown once and it will show in the location > of the most recently used activity of that app. If we later on find an app which really uses > multiple windows and it is imperative to keep the history for all of them tact & the app is > recreating them properly, (a lot of if's) we can revisit the single |AppActivityProxy| and > try to address it in a cleaner way, but at this time that seems rather un-useful since it is > not known if required. > > BUG=388085 > TEST=AppActivityTest.* > > Review URL: https://codereview.chromium.org/477523002 TBR=skuhne@chromium.org Review URL: https://codereview.chromium.org/497013002 Cr-Commit-Position: refs/heads/master@{#291370} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291370 0039d316-1c4b-4281-b951-d872f2087c98
-
tfarina@chromium.org authored
And while I'm at it, change the includes to use the qualified path for the grit generated header file ($gen_dir/content/app/resources/grit/content_resources.h). BUG=338338 TEST=None R=avi@chromium.org,jamesr@chromium.org,thestig@chromium.org,tony@chromium.org TBR=darin@chromium.org Review URL: https://codereview.chromium.org/486573002 Cr-Commit-Position: refs/heads/master@{#291369} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291369 0039d316-1c4b-4281-b951-d872f2087c98
-
nednguyen@google.com authored
BUG=405955 NOTRY=true Review URL: https://codereview.chromium.org/484353003 Cr-Commit-Position: refs/heads/master@{#291368} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291368 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=401588 Review URL: https://codereview.chromium.org/469993006 Cr-Commit-Position: refs/heads/master@{#291367} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291367 0039d316-1c4b-4281-b951-d872f2087c98
-