- 22 Oct, 2016 40 commits
-
-
dgrogan authored
This will give an estimate of the % of iframes that we can defer loading. We can defer offscreen frames unless they are probably used for cross-origin communication. Reland of https://codereview.chromium.org/2389023002 BUG=635105 TBR=isherman@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2437193002 Cr-Commit-Position: refs/heads/master@{#426979}
-
pkasting authored
NOTRY=true BUG=657839 TEST=none TBR=msarett Review-Url: https://chromiumcodereview.appspot.com/2439183003 Cr-Commit-Position: refs/heads/master@{#426978}
-
drbasic authored
When the debugger session in Visual Studio finished, the console window closed by default. For console applications, such as unit tests, it is possible not to close the console window immediately. This is controlled by the property "SubSystem" on "Link" page in project settings. This patch set property "SubSystem" in generated *.vcxproj file based on the "ldflags" field. R=brettw@chromium.org BUG= Review-Url: https://chromiumcodereview.appspot.com/2428013002 Cr-Commit-Position: refs/heads/master@{#426977}
-
juncai authored
This CL removes period for some of the single line strings that are used by WebBluetooth chooser on Android. BUG=658097, 658093 Review-Url: https://chromiumcodereview.appspot.com/2440213002 Cr-Commit-Position: refs/heads/master@{#426976}
-
skobes authored
Inertness of a scroller can change without causing it to be laid out, if a dialog is added or removed from Document::m_topLayerElements. Checking inertness in PLSA::updateScrollableAreaSet led to stale compositing state. This patch updates PLSA to ignore inertness. This means scrollers will remain in FrameView::m_scrollableAreas even when covered by a dialog. That should be ok, since the ::backdrop element prevents them from actually receiving input. BUG=633520 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://chromiumcodereview.appspot.com/2437303002 Cr-Commit-Position: refs/heads/master@{#426975}
-
pdfium-deps-roller authored
https://pdfium.googlesource.com/pdfium.git/+log/5532835aeb88..3cad596c55cd $ git log 5532835ae..3cad596c5 --date=short --no-merges --format='%ad %ae %s' 2016-10-21 npm Add CPDF_Document::GetPage() unittests 2016-10-21 tsepez Remove dead code in CPDF_CustomAccess BUG=638513 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls TBR=dsinclair@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2440263002 Cr-Commit-Position: refs/heads/master@{#426974}
-
esprehn authored
This code is just copy pasta from v8, this patch copies a newer version of the pasta. https://chromium.googlesource.com/v8/v8/+/374a4da83e6f4e05d31640b23b5ca92f0bbf0586/src/strtod.cc#249 BUG=637044, 629034 Review-Url: https://chromiumcodereview.appspot.com/2438383003 Cr-Commit-Position: refs/heads/master@{#426973}
-
kbr authored
GPU rasterization has been disabled on this configuration. BUG=653538 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=zmo@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2442963002 Cr-Commit-Position: refs/heads/master@{#426972}
-
estark authored
We'd like to get an idea of what MITM proxies tend to do with the TLS Feature Extension (colloquially known as Must-Staple). If MITM proxies blindly copy the extension into generated certificates, then deploying Must-Staple will cause a lot of breakage, due to MITM proxies generating Must-Staple certificates but not stapling OCSP responses. This CL adds an UMA metric for the presence of the TLS Feature Extension in certificates that chain to private roots, as a baby step in this investigation. (Note that this conflates misbehaving MITM proxies with private PKIs that are using Must-Staple, so it's only a starting point for an investigation.) A new asn1::HasTLSFeatureExtension() function is used to record this histogram. BUG=633732 Review-Url: https://chromiumcodereview.appspot.com/2436233002 Cr-Commit-Position: refs/heads/master@{#426971}
-
watk authored
We no longer need this data and/or the historical data should be sufficient if we need it in the future. BUG=644396 Review-Url: https://chromiumcodereview.appspot.com/2447463002 Cr-Commit-Position: refs/heads/master@{#426970}
-
danakj authored
And clean up uses of this interface. DidAbortSwapBuffers becomes DidLoseCompositorFrameSink. Android uses this to tell the browser that it has no pending frames and can make a new frame. Ash was using this to end crossfade maximize animations. I've removed this and dropped "aborting" compositing from the Compositor observer. DidPostSwapBuffers becomes DidSubmitCompositorFrame which represents itself better. RenderWidgetCompositor was telling RenderWidget about this just to make a trace event, so deleted that. DidReceiveCompositorFrameAck is duplicated in LayerTreeHostClient and was not actually called on this interface anymore, so just deleted it. R=piman@chromium.org TBR=sky BUG=606056 CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel Review-Url: https://chromiumcodereview.appspot.com/2444563002 Cr-Commit-Position: refs/heads/master@{#426969}
-
zmo authored
https://chromium.googlesource.com/angle/angle.git/+log/336b147..f017315 BUG=chromium:657859,chromium:653454,chromium:653694,chromium:656485 TEST=bots TBR=kbr@chromium.org CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Review-Url: https://chromiumcodereview.appspot.com/2442143002 Cr-Commit-Position: refs/heads/master@{#426968}
-
luoe authored
Review-Url: https://chromiumcodereview.appspot.com/2435153003 Cr-Commit-Position: refs/heads/master@{#426967}
-
reillyg authored
This change exchanges the 2nd and 3rd parameters to std::memset() in a number of places where the value to set and the length of the buffer were inverted. The resulting incompletely initialized buffers triggered MSan failures. BUG=375297 TBR=dmurph@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2447473002 Cr-Commit-Position: refs/heads/master@{#426966}
-
pdfium-deps-roller authored
https://pdfium.googlesource.com/pdfium.git/+log/a804fd48b716..5532835aeb88 $ git log a804fd48b..5532835ae --date=short --no-merges --format='%ad %ae %s' 2016-10-21 tsepez Re-enable CHECK() than only 0-numbered objects are released. Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls TBR=dsinclair@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2440153003 Cr-Commit-Position: refs/heads/master@{#426965}
-
pkasting authored
BUG=658479 TEST=none TBR=wangxianzhu Review-Url: https://chromiumcodereview.appspot.com/2432043010 Cr-Commit-Position: refs/heads/master@{#426964}
-
Rebaseline Bot authored
https://chromium.googlesource.com/chromium/src/+/f1449449b BUG=657839 TBR=pkasting@chromium.org Review URL: https://codereview.chromium.org/2440253002 . Cr-Commit-Position: refs/heads/master@{#426963}
-
chrome-cron authored
Cr-Commit-Position: refs/heads/master@{#426962}
-
japhet authored
A command line flag will take priority. If none is present, the Feature will be used to determine the policy. BUG=651646 Review-Url: https://chromiumcodereview.appspot.com/2445453002 Cr-Commit-Position: refs/heads/master@{#426961}
-
kozyatinskiy authored
Async call stack with async function always contains (async function) label and async function as top frame for following stack. We can merge label with this frame for better user experience. R=dgozman@chromium.org,littledan@chromium.org BUG=654018,621515 Review-Url: https://chromiumcodereview.appspot.com/2384533002 Cr-Commit-Position: refs/heads/master@{#426960}
-
jcivelli authored
Splitting the Mus demo code so it can be executed from outside of a service. Also making the BitmapUploader use the async version of the GPU service EstablishGpuChannel API to prevent a deadlock when embedded. This is in preparation of running the Mus demo from Android content shell. BUG=657499 Review-Url: https://chromiumcodereview.appspot.com/2440453002 Cr-Commit-Position: refs/heads/master@{#426959}
-
puthik authored
CTS Verifier want us to also support multiple signal strength when we declared that multi-advertisement is supported. Disable multiadvertisement until that is done. BUG=658387,658385 TEST=CTS Verifier LE signal strength test won't show up Review-Url: https://chromiumcodereview.appspot.com/2438373002 Cr-Commit-Position: refs/heads/master@{#426958}
-
imcheng authored
erase() by key takes input param as a const ref. Invoking erase() by key using data that will be destroyed during the erase() call will result in undefined behavior. BUG=656073 Review-Url: https://chromiumcodereview.appspot.com/2436403003 Cr-Commit-Position: refs/heads/master@{#426957}
-
ccameron authored
This has no obviously correct replacement TBR=zmo BUG=650837 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://chromiumcodereview.appspot.com/2443783002 Cr-Commit-Position: refs/heads/master@{#426956}
-
danakj authored
R=enne@chromium.org BUG=606056 CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel Review-Url: https://chromiumcodereview.appspot.com/2444603002 Cr-Commit-Position: refs/heads/master@{#426955}
-
allada authored
Added navigation row highlighting to canvas in network timeline expiriment. See: http://imgur.com/a/GwrDe R=dgozman BUG=653738 Review-Url: https://chromiumcodereview.appspot.com/2436953003 Cr-Commit-Position: refs/heads/master@{#426954}
-
jamescook authored
Several webui dialogs spawned by the ash system tray can be shown either as children of the general settings webui window or by themselves at the login screen. In the latter case we need to place them in the right ash window container. Under mash, chrome is not aware of the ash window hierarchy, so it needs to use a container ID rather than a native window parent. Wire up the "set time" dialog as an example. BUG=657021 TEST=manual, spawn set time dialog from settings and from lock screen TBR=stevenjb@chromium.org for function rename touching c/b/ui/webui/chromeos Review-Url: https://chromiumcodereview.appspot.com/2441133002 Cr-Commit-Position: refs/heads/master@{#426953}
-
danakj authored
When deleting DelegatingRenderer::SwapBuffers I failed to notice that the TRACE_EVENT in there had the benchmark category. This category is used by perf tests, and without that event they had no way to count the number of renderer frames produced. This adds benchmark to the DrawLayers trace event in LayerTreeHostImpl instead and points the perf tests at that. Also removes the benchmark category from the GLRenderer and SoftwareRenderer classes as they are never used in the renderer and are not used for perf tests (tools/ no longer refers to the name "SwapBuffers" for perf tests). R=enne@chromium.org TBR=dtu BUG=656947 CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel Review-Url: https://chromiumcodereview.appspot.com/2442453003 Cr-Commit-Position: refs/heads/master@{#426952}
-
ericwilligers authored
Observer lists now support range-based for loops. BUG=655021 Review-Url: https://chromiumcodereview.appspot.com/2435533004 Cr-Commit-Position: refs/heads/master@{#426951}
-
chrishtr authored
Review-Url: https://chromiumcodereview.appspot.com/2446433002 Cr-Commit-Position: refs/heads/master@{#426950}
-
erikchen authored
GL default state must be set before the offscreen resources are created. BUG=657912 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://chromiumcodereview.appspot.com/2442073003 Cr-Commit-Position: refs/heads/master@{#426949}
-
pkasting authored
NOTRY=true BUG=657839 TEST=none TBR=msarett Review-Url: https://chromiumcodereview.appspot.com/2442943002 Cr-Commit-Position: refs/heads/master@{#426948}
-
davidben authored
BUG=658341 Review-Url: https://chromiumcodereview.appspot.com/2442883002 Cr-Commit-Position: refs/heads/master@{#426947}
-
kojii authored
This patch fixes max preferred width for LayoutText when break-all preferred width ("word-break: break-all" or "word-break: break-word") is used. The problem only appears when the sum of character widths is not equal to the word width. However, when the sum is larger (e.g., kerning,) the difference is hardly noticeable. Also an optimization in LayoutText::width() hides the problem when !preserveNewline(), so it is visible only when "white-space: pre-wrap" or "white-space: pre-line". BUG=655590 Review-Url: https://chromiumcodereview.appspot.com/2438683006 Cr-Commit-Position: refs/heads/master@{#426946}
-
zhongyi authored
BUG= Review-Url: https://chromiumcodereview.appspot.com/2438863003 Cr-Commit-Position: refs/heads/master@{#426945}
-
tapted authored
On 10.11+, when navigating away from the NTP with the bookmarks bar "detached", AppKit currently complains about a constraint violation when setting the bookmarks bar height to zero. It seems to occur because the subview previously asked for 6 DIP of padding on the top and bottom which won't fit in a height of 0 DIP. The subview height is set to 0 DIP shortly after. BookmarkBarView currently says it has a flexible height and fixed padding. Change it to a fixed height and flexible padding to stop AppKit complaining. While investigating, improve the encapsulation of -layoutSubviews slightly. BrowserWindowController called it after setting the view's frame manually and unnecessarily tweaking (a different set of) layout constraints. DCHECK instead. BUG=655112 Review-Url: https://chromiumcodereview.appspot.com/2435593008 Cr-Commit-Position: refs/heads/master@{#426944}
-
caseq authored
Review-Url: https://chromiumcodereview.appspot.com/2434393002 Cr-Commit-Position: refs/heads/master@{#426943}
-
dskiba authored
BUG=none Review-Url: https://chromiumcodereview.appspot.com/2409703002 Cr-Commit-Position: refs/heads/master@{#426942}
-
dschuyler authored
This CL removes the Learn more row button in the Content settings and puts that functionality in a new help button in the upper right of the subpage. BUG=658060 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://chromiumcodereview.appspot.com/2432383005 Cr-Commit-Position: refs/heads/master@{#426941}
-
catapult-deps-roller authored
https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/fe41618f8d39..b7d8fe883c16 $ git log fe41618f8..b7d8fe883 --date=short --no-merges --format='%ad %ae %s' 2016-10-21 bpastene devil: Add some logging to device_monitor. 2016-10-21 jbudorick Revert of [devil] Roll to ADB 1.0.36. (patchset #1 id:1 of https://codereview.chromium.org/2436373002/ ) 2016-10-21 aiolos Add solidSelectedDots option to chart_series config. 2016-10-21 benjhayden Set text-overflow:ellipsis on Name column cells in histogram-set-table. 2016-10-21 benjhayden Fix and test metrics side panel. 2016-10-21 rnephew [Android][Telemetry] Show output of adb.ForwardList when error detected. 2016-10-21 rnephew [Devil] Fix more forwarder logging. 2016-10-21 jbudorick [devil] Roll to ADB 1.0.36. BUG=649481,623989,655653,655625,623989 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=catapult-sheriff@chromium.org Review-Url: https://chromiumcodereview.appspot.com/2445473002 Cr-Commit-Position: refs/heads/master@{#426940}
-