- 28 Apr, 2014 29 commits
-
-
sergeyu@chromium.org authored
The live mode is indicated by presense of DateUTC element and unknown segment duration and size. BUG=338529 Review URL: https://codereview.chromium.org/231283005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266452 0039d316-1c4b-4281-b951-d872f2087c98
-
varkha@chromium.org authored
This prevents overwriting target bounds when reparenting layers during animation. In snapped scenario we snap a window (with a call to WindowState::OnWMEvent in WorkspaceWindowResizer::CompleteDrag()) which starts animation and then immediately reparent it (back to workspace in DockedWindowResizer::MaybeReparentWindowOnDragCompletion). Stopping (completing) BOUNDS animation allows the child window to honor previously set target bounds and keep them when it gets reparented. BUG=364517, 366993 TEST=aura_unittests --gtest_filter=WindowTest.RootWindowSetWhenReparenting Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266115 Review URL: https://codereview.chromium.org/241983003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266451 0039d316-1c4b-4281-b951-d872f2087c98
-
rtenneti@chromium.org authored
BUG=366594 R=glider@chromium.org Review URL: https://codereview.chromium.org/255793002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266450 0039d316-1c4b-4281-b951-d872f2087c98
-
davidyu@chromium.org authored
Expose kServiceInitializationStartupDelay in ChromeBrowserPolicyConnector. Extract DeviceManagementServiceConfiguration to its own files. Add a command line flag to override the consumer DM server address for testing. Fix a comment in policy_switches as it no longer applies. BUG=353050 TEST=manual Review URL: https://codereview.chromium.org/240903002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266449 0039d316-1c4b-4281-b951-d872f2087c98
-
tfarina@chromium.org authored
They do not require any extra setup, so it is easier and they do not run on mac or ios platforms. BUG=331829 TEST=gfx_unittests --gtest_filter=XForm*:Interpolated* TBR=danakj@chromium.org,ben@chromium.org Review URL: https://codereview.chromium.org/255953005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266448 0039d316-1c4b-4281-b951-d872f2087c98
-
noms@chromium.org authored
Mocks: https://docs.google.com/a/google.com/file/d/0BxMIIGI80eU-VW16NEloUFdyaXc/edit This CL is related to its Mac cousin, https://codereview.chromium.org/235833002/. Like the related CL, this one also does not implement the "round" avatar icon, nor the top-left corner info button, but it does set up the layout for the other elements. I've also added the option to be able to center the text in a textfield. BUG=353398 Review URL: https://codereview.chromium.org/249813002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266447 0039d316-1c4b-4281-b951-d872f2087c98
-
prasadv@chromium.org authored
For Windows XP platforms, GOMA service is not supported. Moreover we don't compile chorme when gs_bucket flag is set instead use builds archives, therefore ignore GOMA service for Windows XP. NOTRY=true BUG= Review URL: https://codereview.chromium.org/255963002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266446 0039d316-1c4b-4281-b951-d872f2087c98
-
b.kelemen@samsung.com authored
Need to be in google_malloc section just like every other alloc function. R=willchan@chromium.org BUG=364380 Review URL: https://codereview.chromium.org/257453002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266445 0039d316-1c4b-4281-b951-d872f2087c98
-
tyoshino@chromium.org authored
Counts the number of successful handshake completion with permessage-deflate enabled for each negotiated context_takeover parameter value. BUG=128156,359926 Review URL: https://codereview.chromium.org/244933003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266444 0039d316-1c4b-4281-b951-d872f2087c98
-
tapted@chromium.org authored
Revert of GestureRecognizerTest tests both Aura and Unified Gesture Detectors. (https://codereview.chromium.org/258633005/) Reason for revert: Breaks compile on http://build.chromium.org/p/chromium.linux/builders/Linux%20Builder%20%28dbg%29%2832%29/builds/23390 ../../ui/aura/gestures/gesture_recognizer_unittest.cc: In constructor 'aura::test::{anonymous}::GestureEventConsumeDelegate::GestureEventConsumeDelegate()': ../../ui/aura/gestures/gesture_recognizer_unittest.cc:354:17: error: 'aura::test::{anonymous}::GestureEventConsumeDelegate::wait_until_event_' will be initialized after [-Werror=reorder] ../../ui/aura/gestures/gesture_recognizer_unittest.cc:352:7: error: 'int aura::test::{anonymous}::GestureEventConsumeDelegate::flags_' [-Werror=reorder] ../../ui/aura/gestures/gesture_recognizer_unittest.cc:70:3: error: when initialized here [-Werror=reorder] Original issue's description: > GestureRecognizerTest tests both Aura and Unified Gesture Detectors. > > This is in preparation of landing the unified GR > (see https://codereview.chromium.org/251543003/). > > There are some cases where behavior differs in minor ways. > > This is sometimes dealt with by increasing the test's margin of error > (for instance, slop handling is slightly different, so we now > exceed the slop more thoroughly). > > This is sometimes dealt with by ignoring the test if the test is > run with the unified GR. > > BUG=332418 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266440 TBR=sadrul@chromium.org,tdresser@chromium.org NOTREECHECKS=true NOTRY=true BUG=332418 Review URL: https://codereview.chromium.org/254963003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266443 0039d316-1c4b-4281-b951-d872f2087c98
-
zork@chromium.org authored
Send commands from webcamPrivate API to the webcam via V4L2 ioctls BUG=346492 TBR=tommi@chromium.org TBR=kalman@chromium.org Review URL: https://codereview.chromium.org/256833003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266442 0039d316-1c4b-4281-b951-d872f2087c98
-
asanka@chromium.org authored
The @download attribute can be used to specify a suggested filename for a download. However, this suggested name should only be used if the referring document is in the same origin as the downloaded resource. This CL clears the suggested name if the final download URL after redirects is in a different origin than the original download URL. The companion Blink CL at https://codereview.chromium.org/197033005/ ensures that the suggested name for a download initiated via a @download attribute would only be non-empty if the referring document is allowed to set a suggested name for the original download URL. BUG=346744 Review URL: https://codereview.chromium.org/246893006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266441 0039d316-1c4b-4281-b951-d872f2087c98
-
tdresser@chromium.org authored
This is in preparation of landing the unified GR (see https://codereview.chromium.org/251543003/). There are some cases where behavior differs in minor ways. This is sometimes dealt with by increasing the test's margin of error (for instance, slop handling is slightly different, so we now exceed the slop more thoroughly). This is sometimes dealt with by ignoring the test if the test is run with the unified GR. BUG=332418 Review URL: https://codereview.chromium.org/258633005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266440 0039d316-1c4b-4281-b951-d872f2087c98
-
tdresser@chromium.org authored
TEST=TouchDispositionGestureFilterTest.TestDisallowedMultiFingerSwipe Review URL: https://codereview.chromium.org/252753002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266439 0039d316-1c4b-4281-b951-d872f2087c98
-
ziran.sun@samsung.com authored
password_form_conversion_utils. There are Some old Blink code implemented in Webkit that predates the Chromium/WebKit API, specifically WebPasswordFormData and WebPasswordFormUtils. At this point the split just makes this code harder to update, as you need to wait for a Blink roll when adding a new element to autofill::PasswordForm. This patch is re-writing this code in components/autofill/content/renderer/password_form_conversion_util s to make it cleaner. BUG=None Review URL: https://codereview.chromium.org/249153002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266438 0039d316-1c4b-4281-b951-d872f2087c98
-
yoshiki@chromium.org authored
This is a reland of 260688, which is reverted due to false charge of build failure. Previously, we used FileEntry.getMetadata() to retrieve filesystem metadata and getDriveEntryProperties() to get Drive metadata, so we need 2 calls for 1 file on Drive. With this patch, getDriveEntryProperties() returns not only Drive metadata but also filesystem metadata. It's enough to call only getDriveEntryProperties() and we can reduce a number of calls by half. BUG=345196 TEST=browser_test passes. R=asargent@chromium.org, hashimoto@chromium.org, hirono@chromium.org TBR=asargent@chromium.org, hashimoto@chromium.org, hirono@chromium.org Review URL: https://codereview.chromium.org/195763004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266437 0039d316-1c4b-4281-b951-d872f2087c98
-
antrim@chromium.org authored
We need explicitly call ProfileStartup, as it is done for usual flows from startup_browser_creator.cc. This will correctly initialize per-profile input method map in InputMethodManagerImpl. R=nkostylev@chromium.org BUG=364680 Review URL: https://codereview.chromium.org/253463002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266436 0039d316-1c4b-4281-b951-d872f2087c98
-
antrim@chromium.org authored
Make it possible to tab-navigate to import page and ensure tab/arrows-navigation works there. BUG=360588 Review URL: https://codereview.chromium.org/246603004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266435 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
Review URL: https://codereview.chromium.org/255983002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266434 0039d316-1c4b-4281-b951-d872f2087c98
-
navabi@google.com authored
BUG= Review URL: https://codereview.chromium.org/257743002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266433 0039d316-1c4b-4281-b951-d872f2087c98
-
satyanarayana@google.com authored
BUG= Review URL: https://codereview.chromium.org/256783004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266432 0039d316-1c4b-4281-b951-d872f2087c98
-
michaelpg@chromium.org authored
If the time has not been set automatically via network syncing, the system clock can be changed. This dialog allows the user to change the date, time and time zone when possible. The motivation is to unbrick devices with the wrong time that need to be rolled out into time-sensitive networks. Documentation of the changes to Chrome OS and Ash for this CL is here: https://docs.google.com/a/google.com/drawings/d/1T3demthtROnXf1iE31p5aIPcQzK9SmjfE9EjnJbZ4zs System Time Manual Update UI design doc: https://docs.google.com/a/google.com/document/d/1djzhBrtbx-52Gctp3Fd5MIosARbTwQh_lMmd_qUnqgo Screenshot with timezone: https://drive.google.com/a/google.com/file/d/0B6HSBrih6pNUd3p2SFBoVktjVzQ Screenshot from settings page, no timezone: https://drive.google.com/a/google.com/file/d/0B6HSBrih6pNUXzk0TjNiT0tKMTQ BUG=232066 TEST=SetTimeWebUITest, DateTimeOptionsWebUITest R=stevenjb@chromium.org,nkostylev@chromium.org,dbeam@chromium.org,derat@chromium.org,asvitkine@chromium.org TBR=sky@chromium.org # TBR for adding resources to chrome/browser/chrome_resources.grd Please review: stevenjb@chromium.org - ash/system - chromeos/dbus derat@chromium.org: - chrome/browser/chromeos - chrome/browser/ui/ash - chromeos/dbus (optional) nkostylev@chromium.org: - chrome/app - chrome/browser/chromeos - chrome/browser/resources/chromeos - chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc - chrome/browser/ui/webui/chromeos - chrome/browser/ui/webui/options/chromeos - chrome/browser/browser_resources.grd - chrome/chrome_*.gypi - chrome/common dbeam@chromium.org: - chrome/browser/resources/options - chrome/browser/ui/webui/options/*.cc sky@chromium.org: - chrome/browser/browser_resources.grd Review URL: https://codereview.chromium.org/247663003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266431 0039d316-1c4b-4281-b951-d872f2087c98
-
dbeam@chromium.org authored
R=tbreisacher@chromium.org BUG=366972 NOTRY=true Review URL: https://codereview.chromium.org/251713002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266430 0039d316-1c4b-4281-b951-d872f2087c98
-
boliu@chromium.org authored
If executeHardwareAction fails in onDetachedFromWindow, then HardwareRenderer destructor will deadlock later when being destroyed on UI thread. In this case, run the destructor on UI without deadlock but leak GL resources instead. Achieve this by running always running release gl even on UI thread, and making the allow_gl global to be a true thread local, which can be set to true on UI thread. BUG= TBR=benm Review URL: https://codereview.chromium.org/255783007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266429 0039d316-1c4b-4281-b951-d872f2087c98
-
hirono@chromium.org authored
BUG=None TEST=None TBR=yoshiki@chromium.org Review URL: https://codereview.chromium.org/251933002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266426 0039d316-1c4b-4281-b951-d872f2087c98
-
tapted@chromium.org authored
NOTRY=true NOTREECHECKS=true BUG=367600 TBR=bartfab@chromium.org, jianli@chromium.org Review URL: https://codereview.chromium.org/251093002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266423 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-release@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266421 0039d316-1c4b-4281-b951-d872f2087c98
-
tapted@chromium.org authored
Revert of Set kCanMaximizeKey/kCanResizeKey before adding to parent. (https://codereview.chromium.org/251733004/) Reason for revert: NativeWidgetAuraTest.TestPropertiesWhenAddedToLayout triggering lsan errors link: http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/1042 errors like Direct leak of 24 byte(s) in 1 object(s) allocated from: .. #1 0x97042a in views::(anonymous namespace)::NativeWidgetAuraTest_TestPropertiesWhenAddedToLayout_Test::TestBody() ui/views/widget/native_widget_aura_unittest.cc:238 Original issue's description: > Set kCanMaximizeKey/kCanResizeKey before adding to parent. > > BUG=None > TEST=NativeWidgetAuraTest.TestPropertiesWhenAddedToLayout > R=ben@chromium.org > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266386 TBR=ben@chromium.org,oshima@chromium.org NOTREECHECKS=true NOTRY=true BUG=None Review URL: https://codereview.chromium.org/257823004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266419 0039d316-1c4b-4281-b951-d872f2087c98
-
tapted@chromium.org authored
Revert of Stop passing chrome.test.fail as ErrorCallback (https://codereview.chromium.org/258523004/) Reason for revert: Suspected for making OpenAudioFiles/FileManagerBrowserTest.Test/{0,1} flaky/racy since http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%282%29/builds/10706 Errors like: [26400:26400:0426/004939:INFO:CONSOLE(0)] "[FAIL] undefined: API Test Error in undefined: Displayed data of 1st file is wrong. Actual: Beautiful Song.ogg Expected: Beautiful Song ... at chrome-extension://oobinhbdbiehknkpbpejbbpdbkdjmoco/open_audio_files.js:90:21", source: chrome-extension://oobinhbdbiehknkpbpejbbpdbkdjmoco/_generated_background_page.html (0) [26400:26400:0426/004939:INFO:CONSOLE(0)] "[FAIL] undefined: chrome.test.failure Original issue's description: > Stop passing chrome.test.fail as ErrorCallback > > chrome.test.fail cannot be used as ErrorCallback because it throws exception when the number of arguments doesn't match. > This wrong use of chrome.test.fail makes the error output useless when something goes wrong. > > BUG=None > TEST=browser_tests > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266271 TBR=hirono@chromium.org,hashimoto@chromium.org NOTREECHECKS=true NOTRY=true BUG=None Review URL: https://codereview.chromium.org/254943002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266416 0039d316-1c4b-4281-b951-d872f2087c98
-
- 27 Apr, 2014 9 commits
-
-
tonyg@chromium.org authored
It is failing and I suspect the problem is in the unittest not in the code under test, so I'm disabling rather than reverting the other change. File "E:\b\build\slave\XP_Tests__2_\build\src\tools\telemetry\telemetry\page\page_runner_unittest.py", line 240, in testPagesetRepeat os.remove(output_file) WindowsError: [Error 32] The process cannot access the file because it is being used by another process: 'c:\\docume~1\\chrome~2\\locals~1\\temp\\tmpiwpnmk' TBR=dalecurtis@chromium.org NOTRY=True NOTREECHECKS=True BUG= Review URL: https://codereview.chromium.org/257983003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266411 0039d316-1c4b-4281-b951-d872f2087c98
-
sungmann.cho@navercorp.com authored
BUG=254066 Review URL: https://codereview.chromium.org/255903002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266403 0039d316-1c4b-4281-b951-d872f2087c98
-
rbyers@chromium.org authored
CC touch hit testing now (as of http://crrev.com/264395 does a ray cast through all layers under the touched point. We included an optimization to stop searching once you reach a layer we know is opaque to hit testing. This breaks the common case of sites with a touch handler on the document because blink has an optimization to effectively disable touch hit testing in that scenario (since there's no benefit to be had to expensive rect computation) by marking just the entire root layer with a handler region. This change removes this opaque-to-hit-testing optimization. There's some risk that we'll block on main more often, but it should be rare (you'd need a scrolling layer without a handler over top of a layer with a handler). This also fixes the case where a layer has a region that is supposed to be transparent to hit-testing with the CSS pointer-events: none property. BUG=366034, 269598 Review URL: https://codereview.chromium.org/250663003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266402 0039d316-1c4b-4281-b951-d872f2087c98
-
bolian@chromium.org authored
Review URL: https://codereview.chromium.org/254803007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266401 0039d316-1c4b-4281-b951-d872f2087c98
-
scherkus@chromium.org authored
BUG=110814 Review URL: https://codereview.chromium.org/251833004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266400 0039d316-1c4b-4281-b951-d872f2087c98
-
danakj@chromium.org authored
This functionality does not exist on aura or android anymore, and is going away soon on Mac. RWHViews are always composited now on all platforms, so the active tab is always able to be copied in the browser process with CopyFromBackingStore(). This removes the ViewMsg_Snapshot and ViewHostMsg_Snapshot IPCs. R=piman, thestig BUG=251966, 251960 Review URL: https://codereview.chromium.org/259523008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266399 0039d316-1c4b-4281-b951-d872f2087c98
-
kkimlabs@chromium.org authored
In ForeignSessionHelper class, there were two places we allocate ScopedJavaLocalRef<jstring> inside a loop and doesn't delete. The Java local references are cleaned upon returning to Java, but there is 512 local reference hard-limit so we should delete it every loop to avoid hitting the limit. BUG=366659 Review URL: https://codereview.chromium.org/257513007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266398 0039d316-1c4b-4281-b951-d872f2087c98
-
mukai@chromium.org authored
BUG=365822 R=oshima@chromium.org, ben@chromium.org TEST=manually Review URL: https://codereview.chromium.org/246073009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266397 0039d316-1c4b-4281-b951-d872f2087c98
-
qinmin@chromium.org authored
We cannot guarantee that urls will not be redirected when android mediaplayer requests video streams. Always treat media urls as cross-origin for now. BUG=334204 Review URL: https://codereview.chromium.org/247573004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266396 0039d316-1c4b-4281-b951-d872f2087c98
-
- 26 Apr, 2014 2 commits
-
-
rdevlin.cronin@chromium.org authored
TBR=jochen@chromium.org for chrome/browser/chrome_notification_types.h (*removing* a notification type). BUG=358710 Review URL: https://codereview.chromium.org/252593003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266395 0039d316-1c4b-4281-b951-d872f2087c98
-
tonyg@chromium.org authored
On the chromium.perf waterfall, we have some devices that are not going offline (N4 + N5) and some that are (N7, N10, GN). Diffing the results of the provision_device step pointed out some configuration differences between the working and non-working devices. Most notably, wifi_on was set on all the devices that are going offline and not on the ones that are staying on. I hope making these settings deterministic will get these devices into better shape. BUG=366697 Review URL: https://codereview.chromium.org/251793003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266394 0039d316-1c4b-4281-b951-d872f2087c98
-