New issue
Advanced search Search tips
Starred by 6 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 377847

Blocking:
issue 536263


Show other hotlists

Hotlists containing this issue:
Relevant-for-Bootstrap-4


Sign in to add a comment
link

Issue 749848: CSS table borders not overlapping despite margin-right:-1px

Reported by cvrebert@google.com, Jul 27 2017 Project Member

Issue description

Chrome Version: 60.0.3112.78 (Official Build) (64-bit)
OS: macOS 10.12.6

What steps will reproduce the problem?
(1) Open http://jsbin.com/bihila/1/edit?html,css,output in macOS Chrome.

What is the expected result?
The visual border between the "Go!" button and the <input> in the upper example should be 1px wide, just like it is in the lower example.

What happens instead?
The visual border between the "Go!" button and the <input> in the upper example is 2px wide.

Please provide any additional information below. Attach a screenshot if possible.
In the JS Bin, we have two sets of two elements with 1px borders, and these elements are either display:table-cell or are children of elements that are display:table-cell. The expectation is that using a margin-right:-1px (or margin-left:-1px , as appropriate) on one member of a pair of the elements should cause the adjacent border portions of the elements to overlap, thus preventing an unsightly double-width border between the members of a pair. This works fine in the margin-left case, but bizarrely doesn't work in the margin-right case.

I could not reproduce the bug on Linux. I don't have a Windows machine handy to test with at the moment.

This was originally reported as  Issue 534750 , which was ultimately duped to  Issue 227192 , which claims to have been fixed in Chrome 58+.
Original Bootstrap issue: https://github.com/twbs/bootstrap/issues/14237


For graphics-related bugs, please copy/paste the contents of the chrome://gpu
page at the end of this report.
Graphics Feature Status
Canvas: Hardware accelerated
CheckerImaging: Disabled
Flash: Hardware accelerated
Flash Stage3D: Hardware accelerated
Flash Stage3D Baseline profile: Hardware accelerated
Compositing: Hardware accelerated
Multiple Raster Threads: Enabled
Native GpuMemoryBuffers: Hardware accelerated
Rasterization: Hardware accelerated
Video Decode: Hardware accelerated
Video Encode: Hardware accelerated
WebGL: Hardware accelerated
WebGL2: Hardware accelerated
Driver Bug Workarounds
add_and_true_to_loop_condition
adjust_src_dst_region_for_blitframebuffer
decode_encode_srgb_for_generatemipmap
disable_framebuffer_cmaa
disable_multimonitor_multisampling
disable_webgl_rgb_multisampling_usage
emulate_abs_int_function
get_frag_data_info_bug
init_two_cube_map_levels_before_copyteximage
msaa_is_slow
pack_parameters_workaround_with_pack_buffer
rebind_transform_feedback_before_resume
regenerate_struct_names
remove_invariant_and_centroid_for_essl3
rewrite_texelfetchoffset_to_texelfetch
scalarize_vec_and_mat_constructor_args
set_zero_level_before_generating_mipmap
unfold_short_circuit_as_ternary_operation
unpack_alignment_workaround_with_unpack_buffer
unpack_image_height_workaround_with_unpack_buffer
use_intermediary_for_copy_texture_image
use_unused_standard_shared_blocks
avoid_stencil_buffers
Problems Detected
Multisampling is buggy on OSX when multiple monitors are connected: 237931
Applied Workarounds: disable_multimonitor_multisampling
Unfold short circuit on Mac OS X: 307751
Applied Workarounds: unfold_short_circuit_as_ternary_operation
Always rewrite vec/mat constructors to be consistent: 398694
Applied Workarounds: scalarize_vec_and_mat_constructor_args
Mac drivers handle struct scopes incorrectly: 403957
Applied Workarounds: regenerate_struct_names
On Intel GPUs MSAA performance is not acceptable for GPU rasterization: 527565
Applied Workarounds: msaa_is_slow
glGenerateMipmap fails if the zero texture level is not set on some Mac drivers: 560499
Applied Workarounds: set_zero_level_before_generating_mipmap
Pack parameters work incorrectly with pack buffer bound: 563714
Applied Workarounds: pack_parameters_workaround_with_pack_buffer
Alignment works incorrectly with unpack buffer bound: 563714
Applied Workarounds: unpack_alignment_workaround_with_unpack_buffer
copyTexImage2D fails when reading from IOSurface on multiple GPU types.: 581777
Applied Workarounds: use_intermediary_for_copy_texture_image
Multisample renderbuffers with format GL_RGB8 have performance issues on Intel GPUs.: 607130
Applied Workarounds: disable_webgl_rgb_multisampling_usage
Limited enabling of Chromium GL_INTEL_framebuffer_CMAA: 535198
Applied Workarounds: disable_framebuffer_cmaa
glGetFragData{Location|Index} works incorrectly on Max: 638340
Applied Workarounds: get_frag_data_info_bug
glResumeTransformFeedback works incorrectly on Intel GPUs: 638514
Applied Workarounds: rebind_transform_feedback_before_resume
Result of abs(i) where i is an integer in vertex shader is wrong: 642227
Applied Workarounds: emulate_abs_int_function
Rewrite texelFetchOffset to texelFetch for Intel Mac: 642605
Applied Workarounds: rewrite_texelfetchoffset_to_texelfetch
Rewrite condition in for and while loops for Intel Mac: 644669
Applied Workarounds: add_and_true_to_loop_condition
Decode and encode before generateMipmap for srgb format textures on macosx: 634519
Applied Workarounds: decode_encode_srgb_for_generatemipmap
Init first two levels before CopyTexImage2D for cube map texture on Intel Mac 10.12: 648197
Applied Workarounds: init_two_cube_map_levels_before_copyteximage
Insert statements to reference all members in unused std140/shared blocks on Mac: 618464
Applied Workarounds: use_unused_standard_shared_blocks
Tex(Sub)Image3D performs incorrectly when uploading from unpack buffer with GL_UNPACK_IMAGE_HEIGHT greater than zero on Intel Macs: 654258
Applied Workarounds: unpack_image_height_workaround_with_unpack_buffer
adjust src/dst region if blitting pixels outside read framebuffer on Mac: 644740
Applied Workarounds: adjust_src_dst_region_for_blitframebuffer
Mac driver GL 4.1 requires invariant and centroid to match between shaders: 639760, 641129
Applied Workarounds: remove_invariant_and_centroid_for_essl3
Disable KHR_blend_equation_advanced until cc shaders are updated: 661715
Certain Apple devices leak stencil buffers: 713854
Applied Workarounds: avoid_stencil_buffers
Checker-imaging has been disabled via finch trial or the command line.
Disabled Features: checker_imaging
Version Information
Data exported	7/27/2017, 3:05:33 PM
Chrome version	Chrome/60.0.3112.78
Operating system	Mac OS X 10.12.6
Software rendering list version	13.8
Driver bug list version	10.93
ANGLE commit id	3e6a61fecba9
2D graphics backend	Skia/60 a20ae70af542208b06c21413f13c4c86269c0b84-
Command Line	/Applications/Google Chrome.app/Contents/MacOS/Google Chrome -psn_0_61455 --restore-last-session --flag-switches-begin --flag-switches-end
Driver Information
Initialization time	59
In-process GPU	false
Passthrough Command Decoder	false
Supports overlays	false
Sandboxed	true
GPU0	VENDOR = 0x8086, DEVICE= 0x0d26 *ACTIVE*
Optimus	false
Optimus	false
AMD switchable	false
Driver vendor	
Driver version	10.25.17
Driver date	
Pixel shader version	4.10
Vertex shader version	4.10
Max. MSAA samples	8
Machine model name	MacBookPro
Machine model version	11.4
GL_VENDOR	Intel Inc.
GL_RENDERER	Intel Iris Pro OpenGL Engine
GL_VERSION	4.1 INTEL-10.25.17
GL_EXTENSIONS	GL_ARB_blend_func_extended GL_ARB_draw_buffers_blend GL_ARB_draw_indirect GL_ARB_ES2_compatibility GL_ARB_explicit_attrib_location GL_ARB_gpu_shader_fp64 GL_ARB_gpu_shader5 GL_ARB_instanced_arrays GL_ARB_internalformat_query GL_ARB_occlusion_query2 GL_ARB_sample_shading GL_ARB_sampler_objects GL_ARB_separate_shader_objects GL_ARB_shader_bit_encoding GL_ARB_shader_subroutine GL_ARB_shading_language_include GL_ARB_tessellation_shader GL_ARB_texture_buffer_object_rgb32 GL_ARB_texture_cube_map_array GL_ARB_texture_gather GL_ARB_texture_query_lod GL_ARB_texture_rgb10_a2ui GL_ARB_texture_storage GL_ARB_texture_swizzle GL_ARB_timer_query GL_ARB_transform_feedback2 GL_ARB_transform_feedback3 GL_ARB_vertex_attrib_64bit GL_ARB_vertex_type_2_10_10_10_rev GL_ARB_viewport_array GL_EXT_debug_label GL_EXT_debug_marker GL_EXT_framebuffer_multisample_blit_scaled GL_EXT_texture_compression_s3tc GL_EXT_texture_filter_anisotropic GL_EXT_texture_sRGB_decode GL_APPLE_client_storage GL_APPLE_container_object_shareable GL_APPLE_flush_render GL_APPLE_object_purgeable GL_APPLE_rgb_422 GL_APPLE_row_bytes GL_APPLE_texture_range GL_ATI_texture_mirror_once GL_NV_texture_barrier
Disabled Extensions	GL_KHR_blend_equation_advanced GL_KHR_blend_equation_advanced_coherent
Window system binding vendor	
Window system binding version	
Window system binding extensions	
Direct rendering	Yes
Reset notification strategy	0x0000
GPU process crash count	0
Compositor Information
Tile Update Mode	Zero-copy
Partial Raster	Enabled
GpuMemoryBuffers Status
ATC	Software only
ATCIA	Software only
DXT1	Software only
DXT5	Software only
ETC1	Software only
R_8	GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT
RG_88	Software only
BGR_565	Software only
RGBA_4444	Software only
RGBX_8888	Software only
RGBA_8888	GPU_READ, SCANOUT, SCANOUT_CPU_READ_WRITE
BGRX_8888	GPU_READ, SCANOUT, SCANOUT_CPU_READ_WRITE
BGRA_8888	GPU_READ, SCANOUT, SCANOUT_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT
RGBA_F16	GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT
YVU_420	Software only
YUV_420_BIPLANAR	GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT
UYVY_422	GPU_READ_CPU_READ_WRITE, GPU_READ_CPU_READ_WRITE_PERSISTENT
 
crbugshot.png
8.7 KB View Download

Comment 1 by manoranj...@chromium.org, Jul 27 2017

Labels: Needs-Triage-M61 Needs-Bisect
Able to reproduce this issue on Latest Canary#62.0.3168.0 for OS X 10.12.6.

Comment 2 by kkaluri@chromium.org, Jul 28 2017

Labels: -Type-Bug -Pri-3 M-62 Pri-1 Type-Bug-Regression
Able to reproduce this issue on Mac 10.12.5 with chrome version #60.0.3112.78 and also in current canary version #62.0.3168.0
Issue is broken in M47.

Bisect Info:
===========
Good build : 47.0.2516.0 ,  Revision Range -350069
Bad build  : 47.0.2518.0 ,  Revision Range -350532

After executing the old bisect script , i got the following CL's between good and bad build versions
===========================================
https://chromium.googlesource.com/chromium/src/+log/4de1bab9bbd6344709382a368cb452556544095e..2727752b5f7db5711ce1942828a55773dcbdce88


Unable to find the Culprit CL from above CL's, hence untriaging this issue for further triage from dev team.

Note: Issue is not observed on Windows 10, Ubunutu 14.04

Requesting a dev team to look into this issue and assign this issue to the right owner.

Comment 3 by ajha@chromium.org, Jul 31 2017

Cc: ajha@chromium.org
Labels: -Needs-Bisect -Needs-Triage-M61 hasbisect
Owner: msten...@opera.com
Status: Assigned (was: Untriaged)
As per the bisect provided in C#2 there are 3 Blink CLs in the above regression range:

https://chromium.googlesource.com/chromium/blink/+log/ee47094..9839535 [#1]
https://chromium.googlesource.com/chromium/blink/+log/b1a2077..ee47094 [#2]
https://chromium.googlesource.com/chromium/blink/+log/9fe9f73..b1a2077 [#3]

Suspecting https://codereview.chromium.org/1360753002 from [#3].

mstensho@: Could you please take a look at this.

Thank you!

Comment 4 by msten...@opera.com, Jul 31 2017

Cc: msten...@opera.com
Labels: -Pri-1 Pri-3
Owner: ----
Status: Available (was: Assigned)
I find it hard to believe that's the culprit. It only changes pagination / multicol code. Anyway: I don't have a Mac, so I cannot work on this.

Comment 5 by bugsnash@chromium.org, Aug 1 2017

Labels: Update-Weekly

Comment 6 by e...@chromium.org, Aug 30 2017

Components: -Blink>Layout>Subpixel -Blink>CSS Blink>Layout>Table
Owner: dgro...@chromium.org
Status: Assigned (was: Available)
To dgrogan for triage.

Comment 7 Deleted

Comment 8 by cvrebert@google.com, Aug 30 2017

Labels: allpublic

Comment 9 by dgro...@chromium.org, Aug 30 2017

Blockedon: 377847
I see the same issue on mac canary 62. But only on windows by zooming around. Strongly suspect that it's because tables still use integers instead of LayoutUnit for layout (as mentioned in https://github.com/twbs/bootstrap/issues/14237#issuecomment-185410177).

Comment 10 by dgro...@chromium.org, Aug 30 2017

Labels: -hasbisect -Type-Bug-Regression -Update-Weekly -M-62 Type-Bug

Comment 11 by dgro...@chromium.org, May 4 2018

Cc: -ajha@chromium.org -msten...@opera.com
Labels: -OS-Mac
Owner: ----
Status: Available (was: Assigned)

Sign in to add a comment