Bug 146775

Summary: Web Inspector: Copy Rule in the context menu copies hidden properties in the rule
Product: WebKit Reporter: Devin Rousso <hi>
Component: Web InspectorAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description Devin Rousso 2015-07-08 22:19:01 PDT
* STEPS TO REPRODUCE:
1. Paste the following into a rule in the CSS styles sidebar:

margin: 0;
padding: 0;

2. Right click the selector area for that rule and click "Copy Rule".
3. Paste the text.  It should look something like this:

<selector> {
    margin: 0;
    padding: 0;
    margin-top: 0px;
    margin-right: 0px;
    margin-bottom: 0px;
    margin-left: 0px;
    padding-top: 0px;
    padding-right: 0px;
    padding-bottom: 0px;
    padding-left: 0px;
}

Everything after "padding: 0;" shouldn't have been copied as those rules are not actually visible in the rule in the sidebar.
Comment 1 Radar WebKit Bug Importer 2015-07-08 22:19:39 PDT
<rdar://problem/21740039>
Comment 2 Devin Rousso 2015-07-08 22:21:18 PDT
Created attachment 256458 [details]
Patch
Comment 3 WebKit Commit Bot 2015-07-08 23:42:18 PDT
Comment on attachment 256458 [details]
Patch

Clearing flags on attachment: 256458

Committed r186585: <http://trac.webkit.org/changeset/186585>
Comment 4 WebKit Commit Bot 2015-07-08 23:42:21 PDT
All reviewed patches have been landed.  Closing bug.