- 28 Jan, 2016 40 commits
-
-
megjablon authored
BUG=581807 Review URL: https://codereview.chromium.org/1646703002 Cr-Commit-Position: refs/heads/master@{#371974}
-
oshima authored
BUG=581753 TEST=manual Review URL: https://codereview.chromium.org/1646543002 Cr-Commit-Position: refs/heads/master@{#371973}
-
wangxianzhu authored
BUG=574938 TEST=PaintLayerPaintTest.PaintPhaseFloat Review URL: https://codereview.chromium.org/1581593003 Cr-Commit-Position: refs/heads/master@{#371972}
-
tapted authored
Nothing uses these on Mac. They're used a lot on ChromeOS. On Windows, it shows a default browser selection dialog. On Linux it shows the certificate viewer. On Mac I think it was once used for a webui task manager. BUG=None Review URL: https://codereview.chromium.org/1640043002 Cr-Commit-Position: refs/heads/master@{#371971}
-
pkasting authored
Unconditionally requesting the browser's toolbar view causes a null deref. Try to guard by checking if the toolbar is visible first. This should fix the crash unless at this point the toolbar feature is turned on but the toolbar view itself has not been created. Hopefully that's not the case. BUG=581673 TEST=none TBR=sky Review URL: https://codereview.chromium.org/1649493003 Cr-Commit-Position: refs/heads/master@{#371970}
-
rob authored
BUG=575234,578828 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation Review URL: https://codereview.chromium.org/1629013002 Cr-Commit-Position: refs/heads/master@{#371969}
-
stevenjb authored
For extended desktop (non mirrored) layouts, |displays_| should only provide input parameters to the UI. Iteration over display layouts should occur in DisplayLayoutManager. This is necessary to support fake display layouts for testing (and provides a cleaner abstraction in general). This also fixes a couple of bugs related to changing the number of displays introduced in other recent re-factoring. BUG=576375 Review URL: https://codereview.chromium.org/1648493002 Cr-Commit-Position: refs/heads/master@{#371968}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/497b8970..e85d78f6 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. TBR=hablich@chromium.org,machenbach@chromium.org,yangguo@chromium.org,vogelheim@chromium.org Review URL: https://codereview.chromium.org/1641793005 Cr-Commit-Position: refs/heads/master@{#371967}
-
haibinlu authored
Otherwise, ACK buffer will buildup in BlimpMessageOutputBuffer. Review URL: https://codereview.chromium.org/1643813002 Cr-Commit-Position: refs/heads/master@{#371966}
-
kraush authored
This change alters SdkMediaCodecBridgeTest.DoNormal in order to not have it assume that the AudioOutput will get dequeued in three batches. While this is the maximum number, some devices batch output (e.g. MT 8127 and MT 8135 chipsets), thus reducing the total number of dequeues required. Since we also know the expected size of the output, this test now also verifies the full expected output stream has been dequeued. BUG=578881 Review URL: https://codereview.chromium.org/1601863002 Cr-Commit-Position: refs/heads/master@{#371965}
-
zhenw authored
The sync_id was int. This CL changes its type to string so that it is more flexible and consistent with DevTools API's preference. Now the tracing controller uses base::GenerateGUID() to generate the ID. BUG=542390 Review URL: https://codereview.chromium.org/1644653002 Cr-Commit-Position: refs/heads/master@{#371964}
-
kelvinp authored
Revert of Fails QUnit.test() after a timeout of 5 seconds. (patchset #2 id:80001 of https://codereview.chromium.org/1638753003/ ) Reason for revert: Test failing on Linux and Mac. Original issue's description: > Fails QUnit.test() after a timeout of 5 seconds. > > Summary of changes: > 1. Fails QUnit.test() it doesn't return within 5 seconds. > Timing dependent tests should use fake timers instead. > 2. Adds an option to disable the timeout when debugging failed tests > locally. > 3. Logs the current test to the console to ease debugging. > > NOTRY=true > > Trybots failing on tests that has already been deleted. > > Committed: https://crrev.com/ed48a6910aace9ad86ef10c2ecdfeae1bb31a40c > Cr-Commit-Position: refs/heads/master@{#371890} TBR=jamiewalch@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review URL: https://codereview.chromium.org/1642643003 Cr-Commit-Position: refs/heads/master@{#371963}
-
tkent authored
This CL has no behavior changes. BUG=537815 NOTRY=true Review URL: https://codereview.chromium.org/1640293005 Cr-Commit-Position: refs/heads/master@{#371962}
-
skia-deps-roller authored
https://chromium.googlesource.com/skia.git/+log/bbb6dc80fdfe..1c971faab7b2 $ git log bbb6dc80f..1c971faab --date=short --no-merges --format='%ad %ae %s' 2016-01-27 mtklein SkPngFilters: use normal pointer types. CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel TBR=msarett@google.com Review URL: https://codereview.chromium.org/1641953002 Cr-Commit-Position: refs/heads/master@{#371961}
-
jbauman authored
This was intended only for use with NPAPI (now dead), and causes issues by thinking it can activate the DirectComposition child window. BUG=580938 Review URL: https://codereview.chromium.org/1630873003 Cr-Commit-Position: refs/heads/master@{#371960}
-
kcarattini authored
BUG=561867 Review URL: https://codereview.chromium.org/1624853005 Cr-Commit-Position: refs/heads/master@{#371959}
-
yhirano authored
The spec says that retry field is valid when the value consists of only ASCII digits. Currently Chrome accepts "retry: 12a", "retry: 5", etc. This change fixes it. BUG=578709 Review URL: https://codereview.chromium.org/1601553004 Cr-Commit-Position: refs/heads/master@{#371958}
-
ochang authored
TBR=phajdan.jr@chromium.org R=mmoroz@chromium.org BUG=581762 Review URL: https://codereview.chromium.org/1644743002 Cr-Commit-Position: refs/heads/master@{#371957}
-
sky authored
https://codereview.chromium.org/1639223003/ makes it so clients can use whatever ids they want for windows. This means when surfaces resolve ids they need to resolve the id against the client that submitted the frame. This patch does that. BUG=566155 TEST=covered by tests R=fsamuel@chromium.org Review URL: https://codereview.chromium.org/1640253003 Cr-Commit-Position: refs/heads/master@{#371956}
-
reillyg authored
net::IOBuffer should be forward declared. This requires a number of source files to add their own includes where they use net::IOBuffer. BUG=None Review URL: https://codereview.chromium.org/1642493005 Cr-Commit-Position: refs/heads/master@{#371955}
-
yzshen authored
By default, enums are not extensible, which means any unknown value will fail validation. If an enum may grow in the future, it needs to have [Extensible=True] attribute specified. In that case, the user code is responsible for handling unknown values properly. This CL doesn't add corresponding validation for JS or Java. BUG=404186 Review URL: https://codereview.chromium.org/1618963006 Cr-Commit-Position: refs/heads/master@{#371954}
-
erg authored
[This is the same patch as before, but this fixes the win x64 gn bot which is a main waterfall tree closer, but doesn't have a default trybot run.] One common pattern that's coming up multiple times is that we want to read or write the full contents of a file. The first attempt at an interface to that had was put on the File object. However, file seeking behaviour differs between platforms. Performing a seek on an empty file is safe on posix and errors on Windows. And when dealing with arbitrary File objects, we want to seek to the beginning just in case there was any previous usage on the File object. It was also cumbersome. The user was still responsible for opening the file and closing it once they were done with it. Putting these operations on Directory not only removes a bug, but also simplifies the interface. BUG=557405 First Review URL: https://codereview.chromium.org/1634293002 TBR=sky@chromium.org Review URL: https://codereview.chromium.org/1646673002 Cr-Commit-Position: refs/heads/master@{#371953}
-
pfeldman authored
BUG=580337 Review URL: https://codereview.chromium.org/1648463002 Cr-Commit-Position: refs/heads/master@{#371952}
-
danakj authored
These are named in kFooBar style, which doesn't need an underscore. The name should not change if we moved it to an enum, and this is the overwhelming (always used) style for nested constants in chromium. R=dcheng BUG=580746 Review URL: https://codereview.chromium.org/1644803002 Cr-Commit-Position: refs/heads/master@{#371951}
-
oshima authored
* Scale drag/drop events to Viewport in RenderViewHostImpl. * Scale the drag offset to Window/DIP in RenderViewImpl. * Use the screen info's scale factor to tag the drag image. * Make sure that the target image size passed to clampedImageScale is in DIP. BUG=485650 TEST=manual. I'll add a unit test after https://codereview.chromium.org/1586923002/ is landed. Review URL: https://codereview.chromium.org/1605143003 Cr-Commit-Position: refs/heads/master@{#371950}
-
thakis authored
Seems cleaner. libc++ recently added a few headers that shadow C headers, and shouldn't pick those up when building C files. No intended behavior change. BUG=none Review URL: https://codereview.chromium.org/1647493002 Cr-Commit-Position: refs/heads/master@{#371949}
-
estade authored
BUG=561133 TEST=on cros, right click on shelf > shelf position Review URL: https://codereview.chromium.org/1637973002 Cr-Commit-Position: refs/heads/master@{#371948}
-
scheib authored
Initial high level documentation for device/bluetooth. Review URL: https://codereview.chromium.org/1619173002 Cr-Commit-Position: refs/heads/master@{#371947}
-
sebsg authored
For phone fields that have a maxLength attribute of 10 (city code + phone number), if the value of the phone number is larger, fill the field with the last 10 digits. BUG=579674 TEST=AutofillFieldTest Review URL: https://codereview.chromium.org/1639563002 Cr-Commit-Position: refs/heads/master@{#371946}
-
dgrogan authored
Classes inside core/layout/ that need a reference to an InlineBox's layoutObject are being converted to use lineLayoutItem() so that we can rationalize access to the Layout tree. There are no functional changes. BUG=499321 Review URL: https://codereview.chromium.org/1640173005 Cr-Commit-Position: refs/heads/master@{#371945}
-
marpan authored
https://chromium.googlesource.com/webm/libvpx.git/+log/c0307e6cea0f..d6996849f0c6 $ git log c0307e6ce..d6996849f --date=short --no-merges --format='%ad %ae %s' 2016-01-26 yunqingwang Fix a clang/win build error 2016-01-25 aconverse vp9: Be more responsive to massive overshoot. 2016-01-22 zoeliu Cleaned a code in define_gf_group() for firstpass 2016-01-20 aconverse Short circuit flat blocks when coding screen content at realtime speed. 2016-01-22 slavarnway VP9: Do not set up/left available in vp9_first_pass() 2016-01-22 yaowu Fix a minor typo 2016-01-21 marpan vp9 pickmode: Fix the shift on negative value. 2016-01-21 jzern vp9_aq_cyclicrefresh: fix double->float warning 2016-01-21 yaowu Fix a typo in ctrl_set_render_size() 2016-01-20 marpan vp9-aqmode=3: Reduce condition below which we turn off delta-qp. 2016-01-20 huisu Fix build with --enable-internal-stats 2016-01-20 aconverse Remove the vestigal skip_eob feature from tokenization. 2016-01-20 marpan vp9-svc: Allow frame-dropping in svc datarate control tests. 2016-01-20 marpan vp8-denoiser: Fix threshold for skin map. 2016-01-20 yaowu Remove duplicate definitions 2016-01-15 marpan Vidyo patch: Changes to the scalability code. 2016-01-19 slavarnway VP9: Eliminate MB_MODE_INFO 2016-01-18 rsbultje Use input framerate as timebase if none was explicitly specified. 2016-01-15 aconverse Tie the bit cost scale to a define. 2016-01-15 aconverse Deduplicate sign cost for ONE/TWO/THREE/FOUR tokens (...) R=johannkoenig@google.com Review URL: https://codereview.chromium.org/1641773002 Cr-Commit-Position: refs/heads/master@{#371944}
-
catapult-deps-roller authored
https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/8283c4a90352..e8c9e565cd60 $ git log 8283c4a90..e8c9e565c --date=short --no-merges --format='%ad %ae %s' CQ_INCLUDE_TRYBOTS= TBR=catapult-sheriff@chromium.org Review URL: https://codereview.chromium.org/1644683003 Cr-Commit-Position: refs/heads/master@{#371943}
-
lgarron authored
- ecobee.com: "I just noticed that this is actually a problem for us as we have some external sub-domains that are unable to support HTTPS at this time." - emilstahl.dk: Has only been on the list one week; asked to be removed. - liftcannabis.ca: "Unfortunately we found a few edge cases where HSTS is causing problems." - mbdb.jp: https://crbug.com/580764 BUG=527947, 580764 TBR=agl@chromium.org Review URL: https://codereview.chromium.org/1640963002 Cr-Commit-Position: refs/heads/master@{#371942}
-
pdr authored
Performance requires that some opaque chunks (such as the background) are drawn using a layer set to be opaque. This patch implements this idea by adding a bit to DrawingDisplayItem for whether the item is known to be opaque. The "knownToBeOpaque" bit is used to mark layers as explicitly opaque which lets us pass 2x more tests. BUG=580355 Review URL: https://codereview.chromium.org/1632263002 Cr-Commit-Position: refs/heads/master@{#371941}
-
fs authored
This implements the parsing algorithm for "list of floating point numbers" and uses it to parse the 'coords' attribute on <area>. The fractional part of numbers are no longer discarded. The old parsing code in platform/Length.cpp is no longer used and is removed. BUG=578114 Review URL: https://codereview.chromium.org/1636333003 Cr-Commit-Position: refs/heads/master@{#371940}
-
dyen authored
This reverts commit be50aafa. Previously our crash symbol parsing was not taking into account other files that are also loaded in (especially in debug builds). R=nednguyen@google.com BUG=563716 Review URL: https://codereview.chromium.org/1615173003 Cr-Commit-Position: refs/heads/master@{#371939}
-
scottmg authored
And make GetHostDesktopTypeForNativeView/Window and GetActiveDesktop() no-ops. R=sky@chromium.org BUG=558054 Review URL: https://codereview.chromium.org/1637943003 Cr-Commit-Position: refs/heads/master@{#371938}
-
davidben authored
This removes all uses of base::hash_*, BASE_HASH_NAMESPACE, and base::ScopedPtrHashMap in favor of the C++11 versions. BUG=576864, 579229 CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel Review URL: https://codereview.chromium.org/1587283002 Cr-Commit-Position: refs/heads/master@{#371937}
-
hans authored
This build includes changes from https://codereview.chromium.org/1640873003/. The only difference between the new and the old archives is the addition of an empty file (the stub for the new library). Existing libraries were not rebuilt. BUG=520257 Review URL: https://codereview.chromium.org/1640983002 Cr-Commit-Position: refs/heads/master@{#371936}
-
dgrogan authored
Classes inside core/layout/ that need a reference to an InlineBox's layoutObject are being converted to use lineLayoutItem() so that we can rationalize access to the Layout tree. There are no functional changes. BUG=499321 Review URL: https://codereview.chromium.org/1645753002 Cr-Commit-Position: refs/heads/master@{#371935}
-