| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
Cr-Commit-Position: refs/branch-heads/2420@{#4}
Cr-Branched-From: 39d27afab20c3f8559bde9207d8c0260cc94322d-refs/heads/master@{#332301}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
case for NV12."
A likely accidental removal of NV12 case from this method broke HW video decode
on CrOS ARM, as plane size calculations for NV12 were using incorrect
BytesPerElement result.
TBR=mcasas@chromium.org,dalecurtis@chromium.org
BUG=494912
TEST=video playback, apprtc
Review URL: https://codereview.chromium.org/1163973002
Cr-Commit-Position: refs/heads/master@{#332566}
(cherry picked from commit 0360c9825afe3740669d707cf75295b763cbcba4)
Review URL: https://codereview.chromium.org/1162993004
Cr-Commit-Position: refs/branch-heads/2420@{#3}
Cr-Branched-From: 39d27afab20c3f8559bde9207d8c0260cc94322d-refs/heads/master@{#332301}
|
|
|
|
|
| |
Cr-Commit-Position: refs/branch-heads/2420@{#2}
Cr-Branched-From: 39d27afab20c3f8559bde9207d8c0260cc94322d-refs/heads/master@{#332301}
|
|
|
|
|
| |
Cr-Commit-Position: refs/branch-heads/2420@{#1}
Cr-Branched-From: 39d27afab20c3f8559bde9207d8c0260cc94322d-refs/heads/master@{#332301}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the omnibox, when typing the phrase "this is a search phrase",
TemplateURL::FindSearchTermsInURL() does around 120,000-130,000 heap
allocations. This change drops that by roughly 23,000. This change drops
TemplateURLRef::ParseParameter() from 4 to 2 allocations per call
(saving ~20,000), and TemplateURLRef::ParseHostAndSearchTermKey() from
12 to 11 (saving ~3,000).
Review URL: https://codereview.chromium.org/1162833002
Cr-Commit-Position: refs/heads/master@{#332301}
|
|
|
|
|
|
|
|
|
| |
R=grt, wfh
BUG=none
Review URL: https://codereview.chromium.org/1164733002
Cr-Commit-Position: refs/heads/master@{#332300}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously password manager will not autofill in iframe regardless of its
origin. This change will allow autofill in iframes that have the same origin as
the enclosing page.
BUG=345371
Review URL: https://codereview.chromium.org/1159513002
Cr-Commit-Position: refs/heads/master@{#332299}
|
|
|
|
|
|
|
|
|
|
|
|
| |
I plan to add checks to see that the directory tree in the LHS of Files app is
updated when we create/delete/share a folder.
BUG=None
TEST=browser_tests -gtest_filter=*CreateNewFolder/FileManagerBrowserTest*
Review URL: https://codereview.chromium.org/1157663005
Cr-Commit-Position: refs/heads/master@{#332298}
|
|
|
|
|
|
|
|
|
|
| |
This is useful for testing, for example, we need to expose GC for layout/perf testing.
BUG=478249
Review URL: https://codereview.chromium.org/1151153007
Cr-Commit-Position: refs/heads/master@{#332297}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is not intended to change anything for current VS2013 users.
The 2015 code here will not work until a change in depot_tools side
lands. (It might not work after that either, but this is
the first step in getting a 2015 fyi bot up.)
R=dpranke@chromium.org
BUG=492774
Review URL: https://codereview.chromium.org/1163723003
Cr-Commit-Position: refs/heads/master@{#332296}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/1149893019
Cr-Commit-Position: refs/heads/master@{#332295}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The last time we tried to roll asan on the lkgr bot it caused bot timeouts.
Since we don't see that on our trunk bots, I'm trying this again -- it's likely
this will cause timeouts again and will have to be reverted again.
Since regular clang is now pinned, the asan bot can use the regular pinned
version instead of having its own pinned version.
If this sticks, we can reenable the oilpan plugin in asan configurations again.
BUG=487929,489123,483065
Review URL: https://codereview.chromium.org/1162923002
Cr-Commit-Position: refs/heads/master@{#332294}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This delegate is consulted whenever a CRWBrowsingDataStore's mode wants to
become |ACTIVE| or |INACTIVE|.
The default behavior of stashing and restoring browsing data does not fit all
use cases and this decision needs to be delegated over.
Added a TODO to actually consult the delegate when a mode change occurs.
BUG=480654
Review URL: https://codereview.chromium.org/1151063004
Cr-Commit-Position: refs/heads/master@{#332293}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I've accumulated lots of pairings through testing various versions that the
list of paired clients is too large to fit in the dialog, and the Close and
Delete All buttons are not clickable. Although I don't anticipate many users
having this problem, this CL makes the list scrollable.
I've also moved the existing CSS rule under css/html/ since it's not needed
for app remoting.
Review URL: https://codereview.chromium.org/1162263006
Cr-Commit-Position: refs/heads/master@{#332292}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The optimization for avoiding unncessary glEGLImageTargetTexture2DOES
does not apply when the workaround to never re-use textures is in effect
since texture is never re-used and must always be bound again.
BUG=492315
Review URL: https://codereview.chromium.org/1145893006
Cr-Commit-Position: refs/heads/master@{#332291}
|
|
|
|
|
|
|
|
| |
BUG=493856
Review URL: https://codereview.chromium.org/1167593002
Cr-Commit-Position: refs/heads/master@{#332290}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
It was added in https://codereview.chromium.org/3531019 to account for False
Start corking logic. That logic has since been removed, so there is no need to
account for it anymore.
Also remove a TODO to use SSL_RestartHandshakeAfterCertReq. There's not much
point since we'll still need a timeout with new socket codepath anyway.
BUG=none
Review URL: https://codereview.chromium.org/1162323002
Cr-Commit-Position: refs/heads/master@{#332289}
|
|
|
|
|
|
|
|
|
|
| |
person'.
BUG=478443
Review URL: https://codereview.chromium.org/1151573013
Cr-Commit-Position: refs/heads/master@{#332288}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The "title" file_handler attribute in the manifest is deprecated. Moreover
unmounting was broken.
TEST=Tested manually according to the bug repro steps.
BUG=481670
Review URL: https://codereview.chromium.org/1162433004
Cr-Commit-Position: refs/heads/master@{#332287}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
extensions\common\permissions\api_permission.h had a value added (on
May 11 for bug 485227) that threw off the numbering. Move it to the
bottom. We should probably also merge this to the branch.
BUG=485227
NOPRESUBMIT=true
Review URL: https://codereview.chromium.org/1153503003
Cr-Commit-Position: refs/heads/master@{#332286}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/1150853008
Cr-Commit-Position: refs/heads/master@{#332285}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Update sad tab layout to new design and match the other platforms.
Add a button for reloading the page.
Fixed BlueLabelButton to enable LCD font smoothing when layer backed.
BUG=490139
Review URL: https://codereview.chromium.org/1145063005
Cr-Commit-Position: refs/heads/master@{#332284}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Tests failed due to event_page_tracker_ nullptr in MediaRouterMojoImpl.
There is actually no reason we should delay instantiation of that field
until BindToMojoRequest. So the change that would also fix the test is
to initialize event_page_tracker_ on construction. It also eliminates
a ForTest method since now tests can inject mock/fake version of
EventPageTracker via the common constructor.
TEST=*_rel trybots,asan trybots. Also run the tests locally.
Original patch description: (crrev.com/1139203003)
This patch has two parts:
- MediaRouterWebUIMessageHandler handles calls coming
from WebUI/JS. Also sends up-to-date data (e.g. MediaSinks)
to WebUI.
- MediaRouterUI interfaces with the MediaRouter to handle calls
from the WebUI (such as CreateRoute) and listen for sink /
route updates.
Implemented handler logic for the following from Media Router WebUI:
OnGetInitialSettings
OnCreateRoute
OnActOnIssue - skeleton impl
OnCloseRoute
OnCloseDialog
Implemented WebUI logic for the following (to update the UI):
UpdateSinks
UpdateRoutes
UpdateCastModes
AddRoute
UpdateIssue - skeleton impl
Some Issues related APIs are left unimplemented until more resources
have been upstreamed.
BUG=464216, 464208
BUG=
Review URL: https://codereview.chromium.org/1161483003
Cr-Commit-Position: refs/heads/master@{#332283}
|
|
|
|
|
|
|
|
|
|
|
| |
Since we manipulate pending state in a menu controller we need to reset it before the menu controller goes away. This became broken in https://codereview.chromium.org/1129203003/ and only affects the unit tests added in that CL.
BUG=493321
TEST=MenuControllerTest.NextSelectedItem
Review URL: https://codereview.chromium.org/1153193012
Cr-Commit-Position: refs/heads/master@{#332282}
|
|
|
|
|
|
|
|
|
|
|
| |
(The bug has been fallow since 2011 and I couldn't reproduce it with --gtest_repeat=100 in my checkout.)
BUG=88228
R=rsleevi@chromium.org
Review URL: https://codereview.chromium.org/1159183005
Cr-Commit-Position: refs/heads/master@{#332281}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #5 id:80001 of https://codereview.chromium.org/989493002/)
Reason for revert:
Broke video-related WebGL conformance tests; see https://code.google.com/p/chromium/issues/detail?id=449197#c5 .
Original issue's description:
> android video: use SkImage instead of texture backed SkBitmap.
>
> Since getting rid of texture backed bitmaps is one of the Skia team's goals,
> remove texture backed SkBitmap in android media code.
>
> TEST=blink layout tests in fast/canvas
> BUG=449197
>
> Committed: https://crrev.com/8e1ec43e4f2fc6925c74db45ee4407de47fd4152
> Cr-Commit-Position: refs/heads/master@{#331974}
TBR=sievers@chromium.org,bsalomon@chromium.org,reed@google.com,bsalomon@google.com,dongseong.hwang@intel.com
BUG=449197
Review URL: https://codereview.chromium.org/1162003005
Cr-Commit-Position: refs/heads/master@{#332280}
|
|
|
|
|
|
|
|
|
| |
BUG=493960
TEST=Switch the UI language to Hebrew and see if Hebrew characters are rendered with Noto Sans Hebrew instead of Arimo (Arial) or other fonts.
Review URL: https://codereview.chromium.org/1153923006
Cr-Commit-Position: refs/heads/master@{#332279}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch fixes size_t to int truncations in animation/ and debug/
directories in cc.
R=danakj
BUG=167187
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1156483007
Cr-Commit-Position: refs/heads/master@{#332278}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a start of 3-side patch to remove interface in Blink
to get notification about IME candidate window change events.
More code cleanups on Chromium side will follow once
the interface is removed from Blink's WebWidgets.h.
Blink side change is
https://codereview.chromium.org/1162853002/
BUG=494856
Review URL: https://codereview.chromium.org/1150953003
Cr-Commit-Position: refs/heads/master@{#332277}
|
|
|
|
|
|
|
|
|
|
|
| |
In particular, CookieMonster::InitStore, DiskCacheBasedSSLHostInfo, and
HttpBasicStream::LogNumRttVsBytesMetrics were removed a while back.
BUG=42942, 68553, 87423
Review URL: https://codereview.chromium.org/1161023004
Cr-Commit-Position: refs/heads/master@{#332276}
|
|
|
|
|
|
|
|
| |
BUG=493856
Review URL: https://codereview.chromium.org/1166563002
Cr-Commit-Position: refs/heads/master@{#332275}
|
|
|
|
|
|
|
|
|
|
| |
Also introduce Oobe.showUserPods to avoid 4 copies of the same code.
BUG=478427
Review URL: https://codereview.chromium.org/1161753006
Cr-Commit-Position: refs/heads/master@{#332274}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change users of the methods to not call them unless MP4 support
is included.
TEST=EME tests run
BUG=
Review URL: https://codereview.chromium.org/1159153002
Cr-Commit-Position: refs/heads/master@{#332273}
|
|
|
|
|
|
|
|
| |
Clean up some technical debt related to removing children. Currently we do an expensive traversal to clear the unique name and frame id maps of stale references to children being removed. Stash some state on HistoryNodes to make it faster and more readable.
Review URL: https://codereview.chromium.org/1138543002
Cr-Commit-Position: refs/heads/master@{#332272}
|
|
|
|
|
|
|
|
|
|
|
| |
Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/fb3ee7d..4974d05
TBR=cbiesinger@chromium.org,pfeldman@chromium.org
Review URL: https://codereview.chromium.org/1146133003
Cr-Commit-Position: refs/heads/master@{#332271}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This time, with more throttled plugins!
I'm adding 2 different benchmarks:
1) some popular sites that have throttled foreground plugins
2) a control site that we should also do well on
and creating tests that will run both benchmarks with and without PPS.
R=sullivan@chromium.org
BUG=478918
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:linux_perf_bisect;tryserver.chromium.perf:mac_perf_bisect;tryserver.chromium.perf:win_perf_bisect;tryserver.chromium.perf:android_nexus5_perf_bisect
Review URL: https://codereview.chromium.org/1144383005
Cr-Commit-Position: refs/heads/master@{#332270}
|
|
|
|
|
|
|
|
|
|
| |
method for testing.
Also rename SetConnectivity() to SetConnected() to be more consistent with Connected()
Review URL: https://codereview.chromium.org/1162213003
Cr-Commit-Position: refs/heads/master@{#332269}
|
|
|
|
|
|
|
|
|
|
|
| |
Sufficient time has passed that the vast majority
of users have migrated.
BUG=357704
Review URL: https://codereview.chromium.org/1161133004
Cr-Commit-Position: refs/heads/master@{#332268}
|
|
|
|
|
|
|
|
| |
BUG=465862
Review URL: https://codereview.chromium.org/1151573014
Cr-Commit-Position: refs/heads/master@{#332267}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The tests were disabled because of two failures:
* The test server was failing to start. There were some CLs to fix this
landed quite a while back, and the fact that
URLRequestTestFTP.UnfsafePort, which (needlessly) started the FTP server
was still passing indicates that this isn't a problem any longer.
* Tests were having trouble accessing some test files. This CL modifies
the test to use a file in the url_request_unittest directory, like the
other URLRequestTest subclasses, which should hopefully fix this.
Additionally, FTP is no longer enabled in the TestURLRequestContext by
default, so this CL also sets up FTP support in the URLRequestTestFTP
test fixture.
BUG=25045,495220
Review URL: https://codereview.chromium.org/1160273003
Cr-Commit-Position: refs/heads/master@{#332266}
|
|
|
|
|
|
|
|
|
|
|
| |
https://chromium.googlesource.com/angle/angle.git/+log/4031b99..fd216c4
BUG=
TEST=bots
Review URL: https://codereview.chromium.org/1148003004
Cr-Commit-Position: refs/heads/master@{#332265}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As we plan to standardize on GUID strings for device
instance identifiers across the device module, this
change replaces unique_id() with guid().
Lots of mostly uninteresting changes elsewhere to
accommodate this change, but the extensions USB API
did require a new service (UsbGuidMap) to map between
these GUIDs and the integer IDs its methods use.
BUG=None
R=reillyg@chromium.org
Review URL: https://codereview.chromium.org/1148893006
Cr-Commit-Position: refs/heads/master@{#332264}
|
|
|
|
|
|
|
|
| |
BUG=493878
Review URL: https://codereview.chromium.org/1165753002
Cr-Commit-Position: refs/heads/master@{#332263}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Minor rebaselines needed for three tests. Suppressing to unblock the
Skia roll.
BUG=skia:3877
TBR=caryclark@google.com
NOTRY=true
Review URL: https://codereview.chromium.org/1161313005
Cr-Commit-Position: refs/heads/master@{#332262}
|
|
|
|
|
|
|
|
|
|
| |
Proposal is here: https://docs.google.com/document/d/1ucnVkZSkOyY0R7kyryyLlvNq7rkN8EyAtvE6WsCxcFY/edit#heading=h.w8inspeo32bj
BUG=491369
Review URL: https://codereview.chromium.org/1143303004
Cr-Commit-Position: refs/heads/master@{#332261}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When creating a new Channel ID, an X.509 certificate is created, even though
only the keypair is used. This CL removes the generation and storage of
certificates for Channel ID and updates the callsites to use the modified
API that only exposes a crypto::ECPrivateKey.
BUG=457566
Review URL: https://codereview.chromium.org/1076063002
Cr-Commit-Position: refs/heads/master@{#332260}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
on Windows.
Current guess is that timer on Windows is low resolution
(~16 msec).
This can cause request.creation_time to be equal to
last_connection_change_. This was causing following
2 tests to fail on Windows:
(1) NetworkQualityEstimatorTest.TestPeakKbpsFastestRTTUpdates
(2) URLRequestTestHTTP.NetworkQualityEstimator
If the guess is correct, these tests could potentially fail
on other platforms too.
Trying a fix which makes this check less stringent. Now,
if request creation time is exactly equal to time of last
connection change, the request is still used to
estimate the network quality (unlike before, when the
former was required to be strictly larger than the latter).
BUG=492410
Review URL: https://codereview.chromium.org/1153023003
Cr-Commit-Position: refs/heads/master@{#332259}
|
|
|
|
|
|
|
|
|
|
|
|
| |
If the callback isn't answered, the DisplayConfigurator will be waiting
to finish the current display configuration run before starting the next
one.
BUG=none
Review URL: https://codereview.chromium.org/1154063004
Cr-Commit-Position: refs/heads/master@{#332258}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
For software compositing, this will reuse resources when possible
to avoid rastering the full tile. This improves the time to
raster a single animating tab loading spinner from 0.283474 ms to
0.0729661 ms per frame, for a 74% reduction in raster time.
To reproduce use the --disable-gpu command line flag.
R=reveman, vmpstr
BUG=487387
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1157103006
Cr-Commit-Position: refs/heads/master@{#332257}
|