<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>98924</bug_id>
          
          <creation_ts>2012-10-10 10:18:18 -0700</creation_ts>
          <short_desc>Web Inspector: [CSS Regions] Move the Regions support out of experiments</short_desc>
          <delta_ts>2014-08-03 18:40:09 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Web Inspector (Deprecated)</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>AdobeTracked</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>90786</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Mihai Balan">mibalan</reporter>
          <assigned_to name="Mihai Balan">mibalan</assigned_to>
          <cc>apavlov</cc>
    
    <cc>burg</cc>
    
    <cc>bweinstein</cc>
    
    <cc>joepeck</cc>
    
    <cc>keishi</cc>
    
    <cc>loislo</cc>
    
    <cc>mvujovic</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>syoichi</cc>
    
    <cc>WebkitBugTracker</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>739061</commentid>
    <comment_count>0</comment_count>
    <who name="Mihai Balan">mibalan</who>
    <bug_when>2012-10-10 10:18:18 -0700</bug_when>
    <thetext>The purpose of this patch is to have the WebInspector features related to regions available outside of the experiments tab.

I don&apos;t have an answer yet as to how this should interact with the runtime flag for regions.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739171</commentid>
    <comment_count>1</comment_count>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2012-10-10 11:34:46 -0700</bug_when>
    <thetext>You can introduce a new command &quot;CSS.supportsRegions&quot; and query it post-load or upon context menu.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739654</commentid>
    <comment_count>2</comment_count>
    <who name="Alexander Pavlov (apavlov)">apavlov</who>
    <bug_when>2012-10-11 00:11:10 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; You can introduce a new command &quot;CSS.supportsRegions&quot; and query it post-load or upon context menu.

The most common way to do this is through the concept of &quot;capabilities&quot;. See:
- Settings.js, var Capabilities = {...} - the default values (yours should be &quot;false&quot;)
- inspector.js, WebInspector.doLoadedDone() queries agents for their supported capabilities and initializes their values according to the response. The last one should provide bound WebInspector._doLoadedDoneWithCapabilities as the callback. Your call will most probably look similar to:
CSS.supportsRegions(WebInspector._initializeCapability.bind(WebInspector, &quot;cssRegionsSupported&quot;, null)); // not the last capability initializer

Afterwards, you just check:
if (Capabilities.cssRegionsSupported) {
    // do something
}</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739724</commentid>
    <comment_count>3</comment_count>
    <who name="Alexander Pavlov (apavlov)">apavlov</who>
    <bug_when>2012-10-11 01:56:41 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (In reply to comment #1)
&gt; &gt; You can introduce a new command &quot;CSS.supportsRegions&quot; and query it post-load or upon context menu.
&gt; 
&gt; The most common way to do this is through the concept of &quot;capabilities&quot;.

Pavel has rightly pointed out that this approach will add up to the total inspector loading time. So it is better to query the agent when you do need the real value and cache it once received.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>900371</commentid>
    <comment_count>4</comment_count>
    <who name="Michelangelo De Simone">michelangelo</who>
    <bug_when>2013-06-13 20:22:58 -0700</bug_when>
    <thetext>This seems to be a Chromium bug, closing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>900533</commentid>
    <comment_count>5</comment_count>
    <who name="Mihai Balan">mibalan</who>
    <bug_when>2013-06-14 06:16:37 -0700</bug_when>
    <thetext>Reopening as the Web Inspector code is still in trunk and usable and the code isn&apos;t Chrome-only.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>