- 13 May, 2014 40 commits
-
-
ericu@chromium.org authored
this is the biggest chunk I can pull out vaguely cleanly. It does contain a couple of fake calls to keep the compiler happy. This CL also makes SetUpMetadata a member in order to ease testing in a later CL. This depends on https://codereview.chromium.org/261843004/. BUG=108012 Review URL: https://codereview.chromium.org/266333002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270016 0039d316-1c4b-4281-b951-d872f2087c98
-
raymes@google.com authored
There was a small regression which http://build.chromium.org/p/chromium/builders/Linux%20x64/builds/64929 in the range r270006-r270012 which seems to be normal growth. This updates expectations as per http://www.chromium.org/developers/tree-sheriffs/perf-sheriffs?pli=1#TOC-Updating-Performance-Expectations See also the perf dashboard at https://chromeperf.appspot.com/report?masters=Chromium&bots=chromium-rel-linux-64&tests=sizes&rev=270012&checked=core BUG=none TBR=bokan@chromium.org,qyearsley@chromium.org Review URL: https://codereview.chromium.org/279413003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270015 0039d316-1c4b-4281-b951-d872f2087c98
-
nbarth@chromium.org authored
Blink try bots are failing due to failure of Tools/Scripts/lint-test-expectations These were broken in previous CL, b/c the lint test weren't run: Suppressions for DirectWrite tests (line thickness) https://codereview.chromium.org/283503003 TBR=borenet, robertphillips BUG=372497 NOTRY=true NOTREECHECKS=true Review URL: https://codereview.chromium.org/288493002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270014 0039d316-1c4b-4281-b951-d872f2087c98
-
bajones@chromium.org authored
BUG=344451 Review URL: https://codereview.chromium.org/286553002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270012 0039d316-1c4b-4281-b951-d872f2087c98
-
teravest@chromium.org authored
This does some small cleanup of a bunch of little things without any logic changes. BUG=239656 Review URL: https://codereview.chromium.org/270863006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270010 0039d316-1c4b-4281-b951-d872f2087c98
-
binji@chromium.org authored
* CHROME_PATH_ESCAPE was wrong (Windows doesn't need space escaping) * run.py wasn't running a debugger * Chrome wasn't being run with --no-sandbox when debugging Windows. Turns out that is the only case where it is needed. BUG=364223 R=sbc@chromium.org Review URL: https://codereview.chromium.org/272243006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270009 0039d316-1c4b-4281-b951-d872f2087c98
-
mlerman@chromium.org authored
The avatar icons should not look skewed in any setting. This CL is a partial revert of https://codereview.chromium.org/212603011/ (specifically, the portions that changed the dimensions of the avatar icons). BUG=367022,369495,341608 Review URL: https://codereview.chromium.org/268073005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270008 0039d316-1c4b-4281-b951-d872f2087c98
-
munjal@chromium.org authored
BUG=345759 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269445 Review URL: https://codereview.chromium.org/254083007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270007 0039d316-1c4b-4281-b951-d872f2087c98
-
kalman@chromium.org authored
incognito and/or given access to the file:// scheme. BUG=372047 R=rdevlin.cronin@chromium.org,isherman@chromium.org Review URL: https://codereview.chromium.org/273253002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270006 0039d316-1c4b-4281-b951-d872f2087c98
-
qinmin@chromium.org authored
Currently we rely on webmediaplayer to inform webview about fullscreen change. With https://codereview.chromium.org/277823002/, we no longer need to do so. When entering fullscreen, the page will be resized to fullscreen, and that will call Will/DidEnterFullscreen for us. When leaving the fullscreen, ToggleFullscreenForTab() will request the tab to exit fullscreen. That should trigger the Resize() call and send Will/DidExitFullscreen for us. BUG=367346 Review URL: https://codereview.chromium.org/275163002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270005 0039d316-1c4b-4281-b951-d872f2087c98
-
shess@chromium.org authored
Previously cached gethash responses were valid either if the last update was recent enough, or if the item itself were recent enough. Switch to supporting independent item lifetimes to prepare for API v3 (which sends lifetime information from the server). BUG=357763 Review URL: https://codereview.chromium.org/280013002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270003 0039d316-1c4b-4281-b951-d872f2087c98
-
chrishtr@chromium.org authored
BUG= Review URL: https://codereview.chromium.org/282653002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270002 0039d316-1c4b-4281-b951-d872f2087c98
-
pkotwicz@chromium.org authored
BUG=366310 TEST=Manual, see bug Review URL: https://codereview.chromium.org/285503002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269998 0039d316-1c4b-4281-b951-d872f2087c98
-
nsatragno@chromium.org authored
This change separates the logic into two different classes. It also - Brings back a bounce animation when there is only one window that was lost at some point in the past. - Removes the minimized animation if there is only one window while alt tabbing. Basically I brought back from the past two classes, WindowCycleController and WindowCycleList and adapted them to work with the new codebase. Also, I moved some tests from WindowSelectorUnittest to WindowCycleControllerUnittest, and removed the ones that are now irrelevant. Finally, we are also dropping support for alt-tabbing while in overview mode in favour of cancelling the latter. BUG=365786 TEST=none Review URL: https://codereview.chromium.org/260883005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269997 0039d316-1c4b-4281-b951-d872f2087c98
-
tfarina@chromium.org authored
This parameter is unused, so we don't need to pass it down to DidChangePriority() anymore. This is another small chunk from r267947 unlikely to regress anything. BUG=265753, 338338, 237249 TEST=content_shell works fine + content_unittests --gtest_filter=ResourceDispatcherTest.* R=jam@chromium.org Review URL: https://codereview.chromium.org/272343003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269996 0039d316-1c4b-4281-b951-d872f2087c98
-
sungmann.cho@navercorp.com authored
Currently, the lint warns at: Error: Field requires API level 18 (current min is 14): android.provider.Settings#EXTRA_ACCOUNT_TYPES [InlinedApi] Error: Field requires API level 16 (current min is 14): android.view.View#SYSTEM_UI_FLAG_FULLSCREEN [InlinedApi] So we need to add @TargetApi to suppress these lint error. BUG=327768 Review URL: https://codereview.chromium.org/273313003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269995 0039d316-1c4b-4281-b951-d872f2087c98
-
alokp@chromium.org authored
Review URL: https://codereview.chromium.org/284703002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269994 0039d316-1c4b-4281-b951-d872f2087c98
-
ananta@chromium.org authored
Send the device scale factor from Windows 8 ASH during initialization via the MetroViewerHostMsg_SetTargetSurface IPC message. cpu, please review everything. sky, please review the ash/chrome/ui portions. I added a dependency on gfx in the metro_driver project for a subsequent change to send the correct device scale factor on Windows 8 as the metro API does not appear to work correctly causing text, etc to appear very small on high dpi monitors with scales above 125%. BUG=371219 R=cpu,sky Review URL: https://codereview.chromium.org/271543009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269993 0039d316-1c4b-4281-b951-d872f2087c98
-
borenet@google.com authored
BUG=372497 Review URL: https://codereview.chromium.org/283503003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269988 0039d316-1c4b-4281-b951-d872f2087c98
-
leng@chromium.org authored
There is a small chance that the permission bubble manager for a given web contents has been deleted before an asychronous permission request has been handled. BUG=371085 Review URL: https://codereview.chromium.org/282693002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269987 0039d316-1c4b-4281-b951-d872f2087c98
-
zelidrag@chromium.org authored
BUG=372098 TEST=none TBR=sky Review URL: https://codereview.chromium.org/279393002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269986 0039d316-1c4b-4281-b951-d872f2087c98
-
estade@chromium.org authored
"Native" theme is sometimes synonymous with "system" theme, and sometimes not. Change some uses of "native" to "system" to avoid confusion with the class called NativeTheme. BUG=none Review URL: https://codereview.chromium.org/244893004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269985 0039d316-1c4b-4281-b951-d872f2087c98
-
dbeam@chromium.org authored
TBR=markusheintz@chromium.org BUG=356741 Review URL: https://codereview.chromium.org/288443003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269983 0039d316-1c4b-4281-b951-d872f2087c98
-
ycheo@chromium.org authored
The test is 30% failed in Galaxy Nexus, JellyBean at DCHECK(!needs_establish_peer_) of WebMediaPlayerAndroid::play(). We need to defer the steam texture creation also when we have the switch, kForceUseOverlayEmbeddedVideo. BUG=372174 Review URL: https://codereview.chromium.org/281523003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269982 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-release@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269979 0039d316-1c4b-4281-b951-d872f2087c98
-
michaeln@chromium.org authored
BUG=364318 Review URL: https://codereview.chromium.org/279683002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269977 0039d316-1c4b-4281-b951-d872f2087c98
-
zysxqn@google.com authored
BUG= Review URL: https://codereview.chromium.org/268673007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269976 0039d316-1c4b-4281-b951-d872f2087c98
-
jbauman@chromium.org authored
The callbacks are overridden by WebTestProxy in layout tests, and layout tests should be the only times that they're called, so they're not useful anymore. BUG=362164 Review URL: https://codereview.chromium.org/279373002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269975 0039d316-1c4b-4281-b951-d872f2087c98
-
mtomasz@chromium.org authored
This is a split of https://codereview.chromium.org/279213002/, to avoid merging issues. It is almost impossible to land a CL adding histograms via CQ, without a merging conflict. TBR=isherman@chromium.org TEST=Just histograms, no actual changes. BUG=248427 Review URL: https://codereview.chromium.org/280783002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269973 0039d316-1c4b-4281-b951-d872f2087c98
-
cjhopman@chromium.org authored
New in this version: - Runs checklicenses.py against the code when running update_dom_distiller_js.sh - Picked up a couple new changes (including fix for checklicenses.py failure) Original Description: The update script should build `ant package` and copy the entire out/package directory. This moves the path to domdistiller.js to .../package/js so update that reference. The old (non-proto) API is still supported, so this keeps using that. Picked up changes: a15dc85 Change json protoc generation directory f9c9a21 Add GENERATED FILE header to proto-plugin files e99285f add *.pyc to .gitignore fdb023d Fix Proto module's path 44e9366 This adds two simple protoc plugins. Review URL: https://codereview.chromium.org/282663002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269972 0039d316-1c4b-4281-b951-d872f2087c98
-
brettw@chromium.org authored
These are required by Blink platform which I'm working on now. Also adds a missing enum to the GN target name description function. BUG= Review URL: https://codereview.chromium.org/276263003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269971 0039d316-1c4b-4281-b951-d872f2087c98
-
wjywbs@gmail.com authored
The ToString() overloading function of inline and reference enums will conflict because they exist in both global namespace and local classes. This patch chooses the correct ToString() function in different namespaces. R=kalman@chromium.org BUG=371042 Review URL: https://codereview.chromium.org/279833002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269969 0039d316-1c4b-4281-b951-d872f2087c98
-
mtomasz@chromium.org authored
This is a split of https://codereview.chromium.org/284443002/, to avoid merging issues. It is almost impossible to land a CL adding histograms via CQ, without a merging conflict. TBR=isherman@chromium.org TEST=Just histograms, no actual changes. BUG=248427 Review URL: https://codereview.chromium.org/288483002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269968 0039d316-1c4b-4281-b951-d872f2087c98
-
bajones@chromium.org authored
Only fails with D3D9 ANGLE. Didn't notice earlier because D3D11 and Conformance 1.0.3 updates happened at roughly the same time. BUG=372511 TBR=kbr@chromium.org,zmo@chromium.org Review URL: https://codereview.chromium.org/285583002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269966 0039d316-1c4b-4281-b951-d872f2087c98
-
vitalybuka@chromium.org authored
BUG=372423 Review URL: https://codereview.chromium.org/284543002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269965 0039d316-1c4b-4281-b951-d872f2087c98
-
vangelis@chromium.org authored
> use accessTopLayer instead of (DEPRECATED) getTopDevice > > NOTRY=True > > the win_chromium_rel failure must be a flake, since my CL only edits mac-only files. > > the mac gpu failure is unrelated to this CL > > Review URL: https://codereview.chromium.org/248113005 TBR=reed@google.com Review URL: https://codereview.chromium.org/275813005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269964 0039d316-1c4b-4281-b951-d872f2087c98
-
benwells@chromium.org authored
The tip is to suggest auth flows are initiated from app UI explaining why. Without this, if a signed out user launches an app that immediately initiates an interactive auth session, the first thing the user sees is a chrome signin screen. BUG=372279 Review URL: https://codereview.chromium.org/273423005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269963 0039d316-1c4b-4281-b951-d872f2087c98
-
teravest@chromium.org authored
There's some logic in ServiceRuntime to support a future implementation for CloseManifestEntry, but since we don't take any useful action, it's much simpler to always return true. This makes ServiceRuntime even smaller. This also removes an unused struct in ServiceRuntime. BUG=239656 Review URL: https://codereview.chromium.org/280613002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269962 0039d316-1c4b-4281-b951-d872f2087c98
-
vandebo@chromium.org authored
BUG=NONE Review URL: https://codereview.chromium.org/269603002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269960 0039d316-1c4b-4281-b951-d872f2087c98
-
benwells@chromium.org authored
This is important to make sure apps handle this, particularly when we remove the sign in requirements. BUG=372279 Review URL: https://codereview.chromium.org/277313002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269959 0039d316-1c4b-4281-b951-d872f2087c98
-