- 31 Jan, 2015 8 commits
-
-
sudarsana.nagineni authored
This CL allows app_shell to use chrome.alarms. Also it includes the following changes related to unit tests: - Moved chrome.alarms tests to extensions/ - Moved a helper function CreateEmptyExtensionWithLocation() from extension_function_test_utils.h to api_test_utils.h. - Moved CreateBackgroundPage() from extension_api_unittest.h to api_unittest.h. - Creates WebContents for the extension that can be used to associate a RenderViewHost in extension unit tests. BUG=443622 Review URL: https://codereview.chromium.org/789643004 Cr-Commit-Position: refs/heads/master@{#314040}
-
tfarina authored
Not in gcl, which probably these days nobody working on Chromium uses. BUG=None R=maruel@chromium.org TBR=maruel@chromium.org NOPRESUBMIT=true Review URL: https://codereview.chromium.org/887033002 Cr-Commit-Position: refs/heads/master@{#314039}
-
vadimt authored
BUG=422489 TBR=mmenke Review URL: https://codereview.chromium.org/889653004 Cr-Commit-Position: refs/heads/master@{#314038}
-
maruel authored
$ git log d863df383..c698ea2a7 --date=short --format='%ad %ae %s' | sed 's/@chromium\.org//' 2015-01-30 maruel Add back -H since it's actively used. 2015-01-28 maruel Remove 1_local.py as it was not relevant anymore. 2015-01-28 maruel Rename --hash to --isolated and -i to -s. R=vadimsh@chromium.org BUG= Review URL: https://codereview.chromium.org/884473007 Cr-Commit-Position: refs/heads/master@{#314037}
-
jiayl authored
BUG=424762 Review URL: https://codereview.chromium.org/880393002 Cr-Commit-Position: refs/heads/master@{#314036}
-
fsamuel authored
This allows all GuestView types to have access to a color picker. BUG=451737 Review URL: https://codereview.chromium.org/889873004 Cr-Commit-Position: refs/heads/master@{#314035}
-
tedchoc authored
Revert of media: Enable Opus support in Clank <video> and MSE (patchset #5 id:100001 of https://codereview.chromium.org/866573004/) Reason for revert: This is breaking downstream builders on L: C 1029.038s Main [FAIL] MediaCanPlayTypeTest.CodecSupportTest_webm: C 1029.038s Main [ERROR:unix_domain_server_socket_posix.cc(106)] Not implemented reached in virtual int net::UnixDomainServerSocket::GetLocalAddress(net::IPEndPoint*) const C 1029.039s Main [WARNING:proxy_service.cc(898)] PAC support disabled because there is no system implementation C 1029.039s Main ../../content/browser/media/media_canplaytype_browsertest.cc:278: Failure C 1029.039s Main Value of: CanPlay("'video/webm; codecs=\"vp8, opus\"'") C 1029.039s Main Actual: "probably" C 1029.039s Main Expected: kOpusProbably C 1029.039s Main Which is: "" C 1029.039s Main ../../content/browser/media/media_canplaytype_browsertest.cc:279: Failure C 1029.039s Main Value of: CanPlay("'video/webm; codecs=\"vp8.0, opus\"'") C 1029.039s Main Actual: "probably" C 1029.039s Main Expected: kOpusProbably C 1029.039s Main Which is: "" C 1029.039s Main ../../content/browser/media/media_canplaytype_browsertest.cc:285: Failure C 1029.039s Main Value of: CanPlay("'video/webm; codecs=\"vp9, opus\"'") C 1029.039s Main Actual: "probably" C 1029.039s Main Expected: VP9AndOpusProbably C 1029.039s Main Which is: "" C 1029.039s Main ../../content/browser/media/media_canplaytype_browsertest.cc:287: Failure C 1029.039s Main Value of: CanPlay("'video/webm; codecs=\"vp9.0, opus\"'") C 1029.039s Main Actual: "probably" C 1029.040s Main Expected: VP9AndOpusProbably C 1029.040s Main Which is: "" C 1029.040s Main ../../content/browser/media/media_canplaytype_browsertest.cc:296: Failure C 1029.040s Main Value of: CanPlay("'audio/webm; codecs=\"opus\"'") C 1029.040s Main Actual: "probably" C 1029.040s Main Expected: kOpusProbably C 1029.040s Main Which is: "" C 1029.040s Main ../../content/browser/media/media_canplaytype_browsertest.cc:297: Failure C 1029.040s Main Value of: CanPlay("'audio/webm; codecs=\"opus, vorbis\"'") C 1029.040s Main Actual: "probably" C 1029.040s Main Expected: kOpusProbably C 1029.040s Main Which is: "" Spoke with vigneshv@ and reverting for now is the best thing to do while a fix is prepared. Original issue's description: > media: Enable Opus support in Clank <video> and MSE > > Opus audio codec is supported by the android platform starting from Lollipop. > This CL enables canPlayType() support for Opus on Clank and MSE playback of Opus > in Clank. This brings Opus feature parity with Desktop Chromium. > > BUG=318436 > > Committed: https://crrev.com/95fcf214b5a9ba718d53efbf509c9d92e2514e4f > Cr-Commit-Position: refs/heads/master@{#313549} TBR=tomfinegan@chromium.org,wolenetz@chromium.org,rsleevi@chromium.org,qinmin@chromium.org,jschuh@chromium.org,vigneshv@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=318436 Review URL: https://codereview.chromium.org/889053003 Cr-Commit-Position: refs/heads/master@{#314034}
-
Scott Hess authored
Sometimes CRLF sneak into SQLite's code, and then causes patch errors when landing in Chromium. Add an import step for cleaning this up and clean things up. None of these files are in the Chromium build. They just gum up the works. BUG=340757 TBR=michaeln@chromium.org NOTRY=true Review URL: https://codereview.chromium.org/880533004 Cr-Commit-Position: refs/heads/master@{#314033}
-
- 30 Jan, 2015 32 commits
-
-
rch authored
Review URL: https://codereview.chromium.org/893693002 Cr-Commit-Position: refs/heads/master@{#314032}
-
vchigrin authored
Review URL: https://codereview.chromium.org/874033003 Cr-Commit-Position: refs/heads/master@{#314031}
-
jamuraa authored
Profiles in bluez are not specific to a device. When client code tries to connect to two different devices, only one profile is created, managed by BluetoothAdapterChromeOS. dbus messages are multiplexed based on the device desired using new class BluetoothAdapterProfileChromeOS. This is a resubmit with bugfixes after https://codereview.chromium.org/851123002/ was reverted by https://codereview.chromium.org/868753006/ BUG=421207 Review URL: https://codereview.chromium.org/888913002 Cr-Commit-Position: refs/heads/master@{#314030}
-
skare authored
BUG=453955 Review URL: https://codereview.chromium.org/888993002 Cr-Commit-Position: refs/heads/master@{#314029}
-
blink-deps-roller authored
Summary of changes available at: https://chromium.googlesource.com/chromium/blink/+log/ae50934..395f4d8 TBR=junov@chromium.org,vsevik@chromium.org Review URL: https://codereview.chromium.org/886143002 Cr-Commit-Position: refs/heads/master@{#314028}
-
zmo authored
It is not used at all. Note that I didn't change any logic at all except 1) Get rid of NonCachedProgramInfoManager. 2) Merge CachedProgramInfoManager into ProgramInfoManager. 3) Get rid of the Create() function. 4) Add ~VertexAttrib() ~UniformInfo() ~Program() as required by compiler. BUG=453127 TEST=no change at all R=piman@chromium.org Review URL: https://codereview.chromium.org/893713002 Cr-Commit-Position: refs/heads/master@{#314027}
-
sergeyu authored
Previously PolicyWatcher allowed initialization on a thread different from the one it runs on. It's no longer necessary. Also for It2Me it was initialized on UI thread, but was given task runner for the network thread, which causes the DCHECK in the linked bug. Now PolicyWatcher is marked as NonThreadSafe. It's initialized on UI thread for It2Me host and on NetworkThread for the me2me host. Also the file thread is used for blocking operation when reading policies. BUG=453615 Review URL: https://codereview.chromium.org/886913002 Cr-Commit-Position: refs/heads/master@{#314026}
-
jamiewalch authored
The 256x256 and 512x512 pngs are not used anywhere, so I deleted them. BUG=453510 Review URL: https://codereview.chromium.org/890883005 Cr-Commit-Position: refs/heads/master@{#314025}
-
jbauman authored
The precise order (or number) of some of these messages doesn't matter, so just handle them in a loop. Review URL: https://codereview.chromium.org/888693004 Cr-Commit-Position: refs/heads/master@{#314024}
-
ckehoe authored
BUG=453523 Review URL: https://codereview.chromium.org/870053008 Cr-Commit-Position: refs/heads/master@{#314023}
-
dbeam authored
This is the Accessibility Developer Tools suggested color for a 4.5 contrast ratio (minimum for AA rating[1] / audit to stop barking). R=kalman@chromium.org BUG=439227 [1] http://www.w3.org/WAI/WCAG2AA-Conformance Review URL: https://codereview.chromium.org/893603006 Cr-Commit-Position: refs/heads/master@{#314022}
-
stuartmorgan authored
This causes crashes if the shutdown path is used on iOS, because on iOS there's already a higher-level autorelease pool by the time this code can be run. BUG=None Review URL: https://codereview.chromium.org/891843002 Cr-Commit-Position: refs/heads/master@{#314021}
-
mseaborn authored
This pulls in the following Native Client changes: f02226b: (bbudge@chromium.org) Disable mmap_unittest.cc tests on Windows 64 in Debug... 6bc00f6: (dschuff@chromium.org) Update upstream compiler_rt to HEAD and use official git mirror 55d212c: (phosek@chromium.org) Updates the tools Makefile to work with Git ccc9c66: (shyamsundarr@chromium.org) Add all remaining avx2 instructions except vgather. 9595661: (jfb@chromium.org) Enable -Werror for LLVM build aa51c35: (pcc@google.com) Disable -flto when compiling nacl_bootstrap_lib. 476c662: (jfb@chromium.org) PNaCl: Update Subzero and LLVM revision in pnacl/COMPONENT_REVISIONS 2f388a0: (dschuff@chromium.org) Enable SCons testing of arm-nacl-clang on the bots fea15c4: (bbudge@chromium.org) Disable mmap_unittest.cc tests on Windows 64 in Debug... BUG=none TEST=browser_tests and nacl_integration CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_compile,linux_nacl_sdk_build Review URL: https://codereview.chromium.org/867463008 Cr-Commit-Position: refs/heads/master@{#314020}
-
kmarshall authored
Fix buffer overflow due to unbounded strlen over the non-null terminated audio policy string. Caught by asan. BUG= Review URL: https://codereview.chromium.org/890683002 Cr-Commit-Position: refs/heads/master@{#314019}
-
Aurimas Liutikas authored
This reverts commit 3952cc42. Retvield messed up in https://codereview.chromium.org/885743003/ and only reverted part of the CL https://chromiumcodereview.appspot.com/889683002/ TBR=newt@chromium.org,tedchoc@chromium.org NOTRY=true BUG=407781 Review URL: https://codereview.chromium.org/892573006 Cr-Commit-Position: refs/heads/master@{#314018}
-
dtseng authored
Currently, when a caller requests a desktop tree and then opens a new windowthe new WebViewmight have an unloaded child root web area because the tree id sent on construction may be stale by the time automation gets it. Adding a notification in WebView::RenderViewHostChanged appears to work properly. This also allows ChromeVox to add one set of listeners on the desktop node. TEST=With ChromeVox Next set to use automation, - navigate within a tab - switch to several tabs and navigate - open new windows and navigate Result should be that events fire properly when tabbing within the new page. Review URL: https://codereview.chromium.org/880063002 Cr-Commit-Position: refs/heads/master@{#314017}
-
dbeam authored
Also adds better support for zoom/wrapping. BUG=438815 R=kalman@chromium.org Review URL: https://codereview.chromium.org/887803003 Cr-Commit-Position: refs/heads/master@{#314016}
-
scottmg authored
Adds functionality to create_installer_archive.py to generate a .d file that includes all of the files that it packages into chrome[.packed].7z. This is done by tracking the source of all the files that are copied into the staging dir that is given to 7za to package the .7z file. The depfile is then used in a gyp 'depfile' to correctly specify the implicit dependencies for the mini_installer target. Previously it used a fake output of 'xxx2.out' to cause it to rebuild every time (which is what we're trying to avoid in this change). Additionally, switch from a rule to an action in gyp -- gyp currently only supports 'depfile' on actions, and as there's only one file being built here, they're equivalent. BUG=342974,451499 Review URL: https://codereview.chromium.org/887673003 Cr-Commit-Position: refs/heads/master@{#314015}
-
bnc authored
This CL changes the argument of GetCanonicalSuffix() from HostPortPair to std::string (both const&) to better document the fact that GetCanonicalSuffix only operates on that field. BUG=392575 Review URL: https://codereview.chromium.org/868753007 Cr-Commit-Position: refs/heads/master@{#314014}
-
kkimlabs authored
- Top level folders are now all the sub folders of Bookmark Bar, Mobile, and Others nodes. - We no longer use Uncategorized. BUG=453024 Review URL: https://codereview.chromium.org/869193008 Cr-Commit-Position: refs/heads/master@{#314013}
-
ariblue authored
BUG=452266 Review URL: https://codereview.chromium.org/887763003 Cr-Commit-Position: refs/heads/master@{#314012}
-
xunjieli authored
This CL implements the following four methods: - getHeaderFields() - getHeaderField(String key) - getHeaderField(int pos) - getHeaderFieldKey(int pos) BUG=398997 Review URL: https://codereview.chromium.org/784853004 Cr-Commit-Position: refs/heads/master@{#314011}
-
morrita authored
Without this, ChannelMojo doesn't work. R=inferno@chromium.org, tsepez@chromium.org TBR=jamesr@chromium.org BUG=453571 Review URL: https://codereview.chromium.org/867903006 Cr-Commit-Position: refs/heads/master@{#314010}
-
jbauman authored
If the bitmap wasn't allocated, return false instead. BUG=411505 Review URL: https://codereview.chromium.org/879143002 Cr-Commit-Position: refs/heads/master@{#314009}
-
scottmg authored
Avoids ninja expecting an import .lib to be generated so that it doesn't continually rebuild because the import lib is "missing". To be landed after gyp roll at https://codereview.chromium.org/890043003/ lands. R=robertshield@chromium.org BUG=342974,451499 Review URL: https://codereview.chromium.org/886123002 Cr-Commit-Position: refs/heads/master@{#314008}
-
felt authored
This updates the permission prompt (infobar and permission bubble) icons with the new icons provided by sgabriel@: https://drive.google.com/a/google.com/folderview?id=0B-Vmdj5n- YddWktRd1BMeHZYWWs&usp=sharing BUG=451490 Review URL: https://codereview.chromium.org/893603003 Cr-Commit-Position: refs/heads/master@{#314007}
-
sdefresne authored
ScheduleAndForget is a thin wrapper around ScheduleTask that hides a base::Bind() call and passing the HistoryBackend pointer as the first parameter. As they are templated function, they prevent splitting HistoryService into a pure virtual interface and a implementation interface that would improve the componentization velocity. BUG=370850 R=droger@chromium.org Review URL: https://codereview.chromium.org/865443005 Cr-Commit-Position: refs/heads/master@{#314006}
-
fsamuel authored
For GuestView types in the browser, such as <webview> in WebUI and <extensionoptions> in WebUI, we would live status messages to appear in the browser. This CL plumbs that information out to the embedder. ContentsMouseEvent, LoadingStateChanged, and UpdateTargetURL are calls out to the content embedder that are used by Browser to update status indicators such as the spinner, and status text. This CL plumbs those calls out from the guest to the embedder. This is a bit racy because guest status and embedder status can interleave but that's OK because these status texts are transient and eventually they'll settle and disappear so there's no long lasting raciness of consequence. BUG=453216 TBR=lazyboy@chromium.org Review URL: https://codereview.chromium.org/880983006 Cr-Commit-Position: refs/heads/master@{#314005}
-
mathp authored
No longer rely on DidFailProvisionalLoad, solving other issues. Includes a rename of URLRequestMockHTTPJob::AddUrlHandler BUG=373455 TEST=NewTabPageInterceptorTest Review URL: https://codereview.chromium.org/845973005 Cr-Commit-Position: refs/heads/master@{#314004}
-
meacer authored
This CL adds Wi-Fi SSID to the captive portal interstitial on Mac and Linux, if the connection is Wi-Fi. WiFiService isn't available on other platforms, so we will be using other mechanisms such as NetworkChangeNotifier on those. BUG=451272 Review URL: https://codereview.chromium.org/887573002 Cr-Commit-Position: refs/heads/master@{#314003}
-
alexclarke authored
This is needed to make the webkit_unit_tests pass when the HTML parseing tasks are posted on the loading task queue. Required for patch https://codereview.chromium.org/876623002 BUG=391005 Review URL: https://codereview.chromium.org/878413002 Cr-Commit-Position: refs/heads/master@{#314002}
-
pmeenan authored
This is in preparation for adding preconnect support which goes through the same underlying plumbing. dns_prefetch was too restrictive of a component name and was only exposing the single method. There are no functionality changes in this CL, just the rename (the namespace was also renamed) BUG=450682 Review URL: https://codereview.chromium.org/848303005 Cr-Commit-Position: refs/heads/master@{#314001}
-