- 03 Aug, 2012 29 commits
-
-
rvargas@google.com authored
BUG=6944 TEST=sbox_integration_tests Review URL: https://chromiumcodereview.appspot.com/10823134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149782 0039d316-1c4b-4281-b951-d872f2087c98
-
mallinath@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10824143 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149781 0039d316-1c4b-4281-b951-d872f2087c98
-
tedchoc@chromium.org authored
For classes with long package/class names, this can extend over 80 characters when preceded by a #ifndef, etc.. BUG= Review URL: https://chromiumcodereview.appspot.com/10850030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149780 0039d316-1c4b-4281-b951-d872f2087c98
-
jhawkins@chromium.org authored
CID_COUNT=2 CID=104562,104563 BUG=none R=khorimoto@chromium.org TBR=rch Review URL: https://chromiumcodereview.appspot.com/10828138 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149778 0039d316-1c4b-4281-b951-d872f2087c98
-
dennisjeffrey@chromium.org authored
testNewTabDocs loads Google Docs and then waits until an expected substring appears on the page before continuing. Previously, the test waited for a string that only appears when viewing the "my drive" folder, because it was assumed that we'd always see that folder when opening google docs. However, sometimes the "shared with me" folder appears on page load. This appears to be because google docs opens the last folder opened by the account. Since we have another perf test that clicks on the "shared with me" folder, then the current test will sometimes fail to realize the page loaded, since an unexpected screen is displayed after page load. The fix is to have this test wait for a different substring on the page, which appears regardless of which folder shows up when loading google docs. BUG=None TEST=Verified the test runs for 10 iterations whereas previously it failed after only 1 or 2 iterations. NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10832127 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149777 0039d316-1c4b-4281-b951-d872f2087c98
-
mazda@chromium.org authored
Reverting because r149766 broke "Linux ChromiumOS Tests (2)". http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%282%29/builds/5610 Dependent upon https://chromiumcodereview.appspot.com/10568014 TBR=brettw@chromium.org BUG=130212 TEST=included Review URL: https://chromiumcodereview.appspot.com/10656052 TBR=mitchellwrosen@chromium.org Review URL: https://chromiumcodereview.appspot.com/10827151 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149776 0039d316-1c4b-4281-b951-d872f2087c98
-
mazda@chromium.org authored
BUG=125043 Review URL: https://chromiumcodereview.appspot.com/10823067 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149775 0039d316-1c4b-4281-b951-d872f2087c98
-
munjal@chromium.org authored
Remove WebAuthFlowWindow and its implementation on all platforms. We don't need it anymore since we now use browser popups. TBR=ben Review URL: https://chromiumcodereview.appspot.com/10823149 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149774 0039d316-1c4b-4281-b951-d872f2087c98
-
jhawkins@chromium.org authored
CID_COUNT=1 CID=104605 BUG=none R=khorimoto@chromium.org TBR=aa,sky Review URL: https://chromiumcodereview.appspot.com/10826127 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149773 0039d316-1c4b-4281-b951-d872f2087c98
-
tzik@chromium.org authored
BUG=140401 Review URL: https://chromiumcodereview.appspot.com/10824160 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149772 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149771 0039d316-1c4b-4281-b951-d872f2087c98
-
jhawkins@chromium.org authored
It appears the sizeof change was to make the code more future proof: sizeof(float) -> sizeof(audio_data[0]) However, audio_data[0] is a float*, not the intended float. Adding the deref to fix this. CID_COUNT=1 CID=104561 BUG=none R=scherkus Review URL: https://chromiumcodereview.appspot.com/10832123 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149770 0039d316-1c4b-4281-b951-d872f2087c98
-
dimich@chromium.org authored
TBR= BUG= Review URL: https://chromiumcodereview.appspot.com/10850032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149769 0039d316-1c4b-4281-b951-d872f2087c98
-
kochi@chromium.org authored
Parent means a folder containing a file in Drive v2 API. If a file does not contain any reference to its parent, it should be interpreted as the file is under the root directory. Changed existing unit test for checking the case that parent is missing. BUG=chromium:127728 TEST=unit_tests --gtest_filter="DriveAPIParserTest.*" Review URL: https://chromiumcodereview.appspot.com/10829139 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149768 0039d316-1c4b-4281-b951-d872f2087c98
-
mitchellwrosen@chromium.org authored
Mapping from extension.location(): INTERNAL -> 'normal' LOAD -> 'development' EXTERNAL_POLICY_DOWNLOAD -> 'admin' EXTERNAL* -> 'sideload' (all others) -> 'other' BUG=135564 TBR=jam@chromium.org Review URL: https://chromiumcodereview.appspot.com/10750010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149767 0039d316-1c4b-4281-b951-d872f2087c98
-
mitchellwrosen@chromium.org authored
Dependent upon https://chromiumcodereview.appspot.com/10568014 TBR=brettw@chromium.org BUG=130212 TEST=included Review URL: https://chromiumcodereview.appspot.com/10656052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149766 0039d316-1c4b-4281-b951-d872f2087c98
-
jhawkins@chromium.org authored
CID_COUNT=1 CID=104610 BUG=none R=khorimoto@chromium.org TBR=sky Review URL: https://chromiumcodereview.appspot.com/10829154 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149765 0039d316-1c4b-4281-b951-d872f2087c98
-
mpearson@chromium.org authored
I forgot to re-enable it after tuning parameters in http://codereview.chromium.org/10782017/ BUG= Review URL: https://chromiumcodereview.appspot.com/10823152 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149764 0039d316-1c4b-4281-b951-d872f2087c98
-
rvargas@google.com authored
SpawnCleanup may trigger the destruction of a partially created TargetProcess. BUG=139898 TEST=none Review URL: https://chromiumcodereview.appspot.com/10831133 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149763 0039d316-1c4b-4281-b951-d872f2087c98
-
rvargas@google.com authored
BUG=none TEST=none TBR=gavinp@chromium.org Review URL: https://chromiumcodereview.appspot.com/10850035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149762 0039d316-1c4b-4281-b951-d872f2087c98
-
simonjam@chromium.org authored
This is often uninitialized in UI and browser tests. This blocks issue 10829031 from landing. BUG=139093 Review URL: https://chromiumcodereview.appspot.com/10831104 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149761 0039d316-1c4b-4281-b951-d872f2087c98
-
nkostylev@chromium.org authored
Revert 149610 - Extension resources should only load in contexts the extension has permission to access. See http://codereview.chromium.org/10792008/ for background. BUG=139592 Review URL: https://chromiumcodereview.appspot.com/10828067 TBR=mkwst@chromium.org Review URL: https://chromiumcodereview.appspot.com/10826134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149760 0039d316-1c4b-4281-b951-d872f2087c98
-
shishir@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10834037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149759 0039d316-1c4b-4281-b951-d872f2087c98
-
bradnelson@google.com authored
default_defines is no longer set in untrusted.gypi Using normal define merging instead. R=bbudge@chromium.org BUG=None TEST=None Review URL: https://chromiumcodereview.appspot.com/10825145 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149758 0039d316-1c4b-4281-b951-d872f2087c98
-
acolwell@chromium.org authored
BUG=136438 TBR=acolwell@chromium.org Review URL: https://chromiumcodereview.appspot.com/10830155 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149757 0039d316-1c4b-4281-b951-d872f2087c98
-
sky@chromium.org authored
observers properly if the resizer stayed visible after a resize. BUG=139227 TEST=covered by unit tests R=ben@chromium.org Review URL: https://chromiumcodereview.appspot.com/10830148 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149756 0039d316-1c4b-4281-b951-d872f2087c98
-
gman@chromium.org authored
certain things needed to be done. Hopefully this fixes these bugs. BUG=140116,139791 Review URL: https://chromiumcodereview.appspot.com/10830147 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149755 0039d316-1c4b-4281-b951-d872f2087c98
-
ronghuawu@chromium.org authored
Use empty string as the mid before it's ready from WebKit. Label is not the correct mid, so we shouldn't use it, otherwise there will be a mismatch of mid and mline_index. BUG= http://code.google.com/p/webrtc/issues/detail?id=717 TEST=Test page from http://code.google.com/p/webrtc/issues/detail?id=717. Review URL: https://chromiumcodereview.appspot.com/10834137 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149754 0039d316-1c4b-4281-b951-d872f2087c98
-
dimich@chromium.org authored
Started to fail in Chrome r149732 TBR=lambroslambrou BUG=140397 Review URL: https://chromiumcodereview.appspot.com/10836089 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149753 0039d316-1c4b-4281-b951-d872f2087c98
-
- 02 Aug, 2012 11 commits
-
-
dennisjeffrey@chromium.org authored
Change 1: fix a problem where the docs test fails when the UI unexpectedly and automatically reloads. The fix is to identify when this happens and invoke some setup code that allows the test scenario to work properly in the event that it does happen. Change 2: modify two xpaths in the plus test to account for UI changes. BUG=None TEST=Verified that both tests run for at least several minutes with these changes. NOTRY=true Review URL: https://chromiumcodereview.appspot.com/10825166 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149751 0039d316-1c4b-4281-b951-d872f2087c98
-
brettw@chromium.org authored
This sets 2048 to select the high-latency path rather than the low one. This allows us to make Flash use power-of-two buffers and still get the high latency path, avoiding bugs in the current low-latency path. BUG= Review URL: https://chromiumcodereview.appspot.com/10830153 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149750 0039d316-1c4b-4281-b951-d872f2087c98
-
jvoung@google.com authored
before deciding to cache a translated nexe (current cache mechanism won't work in incognito). Use this to decide whether or not to cache translations for now. For now, the linker always writes to a self-deleting temp file, instead of to the cache directly. We then copy the data from the temp file to the cache as a separate step. Took about 60ms to copy a 4.6MB nexe in debug mode with a 128kb buffer and 15ms with a 1mb buffer... BUG= http://code.google.com/p/nativeclient/issues/detail?id=2683 TEST= try compiling an app that doesn't use the File system in incognito mode -- we should add a test though... we should add a test though Review URL: https://chromiumcodereview.appspot.com/10826107 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149749 0039d316-1c4b-4281-b951-d872f2087c98
-
stevenjb@google.com authored
BUG=none Review URL: https://chromiumcodereview.appspot.com/10831141 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149748 0039d316-1c4b-4281-b951-d872f2087c98
-
akalin@chromium.org authored
Plumb the registrations from the UI thread to the sync thread, and plumb the invalidations back. Add ObjectIdSet <-> ObjectIdPayloadMap conversions. Rename syncapi_unittest.cc to sync_manager_impl_unittest.cc. BUG=137087 TEST= Review URL: https://chromiumcodereview.appspot.com/10805002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149747 0039d316-1c4b-4281-b951-d872f2087c98
-
bryeung@chromium.org authored
This also fixes a problem with the way error cases were returned: RunImpl should not call SendResponse when returning false. TEST=updated api test BUG=134583 Review URL: https://chromiumcodereview.appspot.com/10816023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149746 0039d316-1c4b-4281-b951-d872f2087c98
-
ivankr@chromium.org authored
BUG=None Review URL: https://chromiumcodereview.appspot.com/10830132 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149745 0039d316-1c4b-4281-b951-d872f2087c98
-
ivankr@chromium.org authored
This only fixes the avatar-related part of the bug. UserImageLoader now always passes raw data to the callback. BUG=139887 Review URL: https://chromiumcodereview.appspot.com/10830111 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149744 0039d316-1c4b-4281-b951-d872f2087c98
-
saintlou@chromium.org authored
BUG=140336 TBR=oshima@chromium.org Review URL: https://chromiumcodereview.appspot.com/10823154 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149743 0039d316-1c4b-4281-b951-d872f2087c98
-
viettrungluu@chromium.org authored
We found that, with Pepper plugins (e.g., Pepper Flash), even 50 ms buffers weren't enough to avoid occasional clicks (with 3 wave out buffers). (On my Vista test machine with 4 wave out buffers we could manage ~43 ms buffers.) Review URL: https://chromiumcodereview.appspot.com/10830150 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149741 0039d316-1c4b-4281-b951-d872f2087c98
-
hallielaine@chromium.org authored
Fixed Create/Manage Profile dialog spacing due to Mac bug. Updated margin to account for added space due to border added. Review URL: https://chromiumcodereview.appspot.com/10832120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149740 0039d316-1c4b-4281-b951-d872f2087c98
-