<?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>39721</bug_id>
          
          <creation_ts>2010-05-26 03:50:19 -0700</creation_ts>
          <short_desc>[chromium] Upstream layout tests expectations for Geolocation.</short_desc>
          <delta_ts>2010-06-10 07:17:05 -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>Tools / Tests</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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Marcus Bulach">bulach</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bulach</cc>
    
    <cc>commit-queue</cc>
    
    <cc>dglazkov</cc>
    
    <cc>dpranke</cc>
    
    <cc>jorlow</cc>
    
    <cc>victorw</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>230649</commentid>
    <comment_count>0</comment_count>
    <who name="Marcus Bulach">bulach</who>
    <bug_when>2010-05-26 03:50:19 -0700</bug_when>
    <thetext>Geolocation is now supported on both DumpRenderTree and test_shell, need to upstream the test expectations.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>230703</commentid>
    <comment_count>1</comment_count>
      <attachid>57093</attachid>
    <who name="Marcus Bulach">bulach</who>
    <bug_when>2010-05-26 07:30:24 -0700</bug_when>
    <thetext>Created attachment 57093
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231645</commentid>
    <comment_count>2</comment_count>
      <attachid>57093</attachid>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-05-28 05:14:11 -0700</bug_when>
    <thetext>Comment on attachment 57093
Patch

LayoutTests/platform/chromium/test_expectations.txt:709
 +  BUG11246 DEFER SKIP : fast/dom/Geolocation/callback-exception.html = TEXT
Just rebaseline it in this patch.  Use the rebaseline tool.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231670</commentid>
    <comment_count>3</comment_count>
      <attachid>57324</attachid>
    <who name="Marcus Bulach">bulach</who>
    <bug_when>2010-05-28 06:40:07 -0700</bug_when>
    <thetext>Created attachment 57324
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231672</commentid>
    <comment_count>4</comment_count>
    <who name="Marcus Bulach">bulach</who>
    <bug_when>2010-05-28 06:41:11 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 57093 [details])
&gt; LayoutTests/platform/chromium/test_expectations.txt:709
&gt;  +  BUG11246 DEFER SKIP : fast/dom/Geolocation/callback-exception.html = TEXT
&gt; Just rebaseline it in this patch.  Use the rebaseline tool.

duh, of course! rebaselined. another look please?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231675</commentid>
    <comment_count>5</comment_count>
      <attachid>57324</attachid>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-05-28 06:49:28 -0700</bug_when>
    <thetext>Comment on attachment 57324
Patch

LayoutTests/platform/chromium/fast/dom/Geolocation/callback-exception-expected.txt:1
 +  CONSOLE MESSAGE: line 22: Uncaught Error: Exception in success callback
This worries me some.  Does it show up with JSC?  I feel like maybe you could catch the exception in the test.

LayoutTests/ChangeLog:9
 +          * platform/chromium/fast/dom/Geolocation/callback-exception-expected.txt: Added.
Does this work?  I believe you have to add it to -win and -mac.  Dimitri has confirmed this in the past.  But, now that I think about it, maybe that&apos;s silly?  Especially if this works.  (Are you sure it does?)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231676</commentid>
    <comment_count>6</comment_count>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-05-28 06:49:47 -0700</bug_when>
    <thetext>Dimitri, can you take a quick look?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231678</commentid>
    <comment_count>7</comment_count>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-05-28 06:52:25 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; (From update of attachment 57324 [details])
&gt; LayoutTests/platform/chromium/fast/dom/Geolocation/callback-exception-expected.txt:1
&gt;  +  CONSOLE MESSAGE: line 22: Uncaught Error: Exception in success callback
&gt; This worries me some.  Does it show up with JSC?  I feel like maybe you could catch the exception in the test.

Nm.  I&apos;m an idiot.  :-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231681</commentid>
    <comment_count>8</comment_count>
    <who name="Marcus Bulach">bulach</who>
    <bug_when>2010-05-28 06:55:07 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; (In reply to comment #5)
&gt; &gt; (From update of attachment 57324 [details] [details])
&gt; &gt; LayoutTests/platform/chromium/fast/dom/Geolocation/callback-exception-expected.txt:1
&gt; &gt;  +  CONSOLE MESSAGE: line 22: Uncaught Error: Exception in success callback
&gt; &gt; This worries me some.  Does it show up with JSC?  I feel like maybe you could catch the exception in the test.
&gt; 
&gt; Nm.  I&apos;m an idiot.  :-)

so the difference between V8 and JSC is the &quot;Uncaught &quot; word.. I could certainly change the test, but that would be a separate change..

&gt; 
&gt; LayoutTests/ChangeLog:9
&gt;  +          * platform/chromium/fast/dom/Geolocation/callback-exception-expected.txt: Added.
&gt; Does this work?  I believe you have to add it to -win and -mac.  Dimitri has confirmed this in the past.  But, now that I think about it, maybe that&apos;s silly?  Especially if this works.  (Are you sure it does?)

&quot;works for me!&quot; :)
dimitri, could you please confirm it&apos;ll work everywhere?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231684</commentid>
    <comment_count>9</comment_count>
    <who name="Dimitri Glazkov (Google)">dglazkov</who>
    <bug_when>2010-05-28 06:57:45 -0700</bug_when>
    <thetext>(In reply to comment #8)
&gt; (In reply to comment #7)
&gt; &gt; (In reply to comment #5)
&gt; &gt; &gt; (From update of attachment 57324 [details] [details] [details])
&gt; &gt; &gt; LayoutTests/platform/chromium/fast/dom/Geolocation/callback-exception-expected.txt:1
&gt; &gt; &gt;  +  CONSOLE MESSAGE: line 22: Uncaught Error: Exception in success callback
&gt; &gt; &gt; This worries me some.  Does it show up with JSC?  I feel like maybe you could catch the exception in the test.
&gt; &gt; 
&gt; &gt; Nm.  I&apos;m an idiot.  :-)
&gt; 
&gt; so the difference between V8 and JSC is the &quot;Uncaught &quot; word.. I could certainly change the test, but that would be a separate change..
&gt; 
&gt; &gt; 
&gt; &gt; LayoutTests/ChangeLog:9
&gt; &gt;  +          * platform/chromium/fast/dom/Geolocation/callback-exception-expected.txt: Added.
&gt; &gt; Does this work?  I believe you have to add it to -win and -mac.  Dimitri has confirmed this in the past.  But, now that I think about it, maybe that&apos;s silly?  Especially if this works.  (Are you sure it does?)
&gt; 
&gt; &quot;works for me!&quot; :)
&gt; dimitri, could you please confirm it&apos;ll work everywhere?

Not that I am a aware. I think you still need to put &apos;em in -win and -mac. Ask dpranke. He knows.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231688</commentid>
    <comment_count>10</comment_count>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-05-28 07:01:46 -0700</bug_when>
    <thetext>If there&apos;s a good reason platform/chromium can&apos;t be in the search path for Linux, Win, and Mac, then we need to delete everything from that directory.  (IIRC, there are a bunch of other expected results in there!)

If there&apos;s not, we should probably change the re-baselining tool, send a FYI email to chromium-dev, and optionally create a script to move all the duplicate files in -mac and -win to the shared folder.

I hope the answer is the latter.  :-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231919</commentid>
    <comment_count>11</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-05-28 13:57:40 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; If there&apos;s a good reason platform/chromium can&apos;t be in the search path for Linux, Win, and Mac, then we need to delete everything from that directory.  (IIRC, there are a bunch of other expected results in there!)
&gt; 
&gt; If there&apos;s not, we should probably change the re-baselining tool, send a FYI email to chromium-dev, and optionally create a script to move all the duplicate files in -mac and -win to the shared folder.
&gt; 
&gt; I hope the answer is the latter.  :-)

platform/chromium is in the search path on all three platforms. It&apos;s fine to put files that are the same on all three Chromium ports but different from upstream there.

I thought the rebaselining tool automatically dealt with this, but I could be wrong.

Copying victorw for thoughts on this ... (watch me pass the buck!)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231948</commentid>
    <comment_count>12</comment_count>
    <who name="Victor Wang">victorw</who>
    <bug_when>2010-05-28 14:37:45 -0700</bug_when>
    <thetext>Personally, I prefer not having baselines in platform/chromium. For now, since the fallback paths are chromium-linux-&gt;chromium-win-&gt;chromium-mac, if baselines for all three ports are same, then the baseline would be in chromium-mac and chromium-mac is serving the same purpose as platform/chromium.

I think having baselines in platfomr/chromium may add extra maintenance work.
For example, if a baseline for all ports are same but latter linux has a different baseline, then in addition to add new linux baseline, we also need to remove baseline from platform/chromium and add it to platform/chromium-mac.
It makes rebaseline tool little harder when it tries to rebaseline for one platform and the tool needs to figure out where to add new baseline. I agree these issues could be resolved by making rebaseline smarter, but don&apos;t feel it worth the effort to do that, at least for now. I prefer keeping it simple and just use the existing path, thoughts?

(In reply to comment #11)
&gt; (In reply to comment #10)
&gt; &gt; If there&apos;s a good reason platform/chromium can&apos;t be in the search path for Linux, Win, and Mac, then we need to delete everything from that directory.  (IIRC, there are a bunch of other expected results in there!)
&gt; &gt; 
&gt; &gt; If there&apos;s not, we should probably change the re-baselining tool, send a FYI email to chromium-dev, and optionally create a script to move all the duplicate files in -mac and -win to the shared folder.
&gt; &gt; 
&gt; &gt; I hope the answer is the latter.  :-)
&gt; 
&gt; platform/chromium is in the search path on all three platforms. It&apos;s fine to put files that are the same on all three Chromium ports but different from upstream there.
&gt; 
&gt; I thought the rebaselining tool automatically dealt with this, but I could be wrong.
&gt; 
&gt; Copying victorw for thoughts on this ... (watch me pass the buck!)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>231953</commentid>
    <comment_count>13</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-05-28 14:40:44 -0700</bug_when>
    <thetext>(In reply to comment #12)
&gt; Personally, I prefer not having baselines in platform/chromium. For now, since the fallback paths are chromium-linux-&gt;chromium-win-&gt;chromium-mac, if baselines for all three ports are same, then the baseline would be in chromium-mac and chromium-mac is serving the same purpose as platform/chromium.

Except that neither chromium-linux nor chromium-win fall back to chromium-mac.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235220</commentid>
    <comment_count>14</comment_count>
    <who name="Marcus Bulach">bulach</who>
    <bug_when>2010-06-08 05:35:18 -0700</bug_when>
    <thetext>(In reply to comment #13)
&gt; (In reply to comment #12)
&gt; &gt; Personally, I prefer not having baselines in platform/chromium. For now, since the fallback paths are chromium-linux-&gt;chromium-win-&gt;chromium-mac, if baselines for all three ports are same, then the baseline would be in chromium-mac and chromium-mac is serving the same purpose as platform/chromium.
&gt; 
&gt; Except that neither chromium-linux nor chromium-win fall back to chromium-mac.

thanks everyone for the feedback!
it seems that platform/chromium is indeed the fallback for all platforms?
should we go ahead with this patch or is it better to duplicate it and add to chromium-win and chromium-mac?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235221</commentid>
    <comment_count>15</comment_count>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-06-08 05:37:55 -0700</bug_when>
    <thetext>Unless there&apos;s a good reason to separate win/mac, I think we should go forward with this patch, change the default behavior of the rebaselining tool, and file a bug to unify all the tests which have identical mac/win baselines.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235271</commentid>
    <comment_count>16</comment_count>
    <who name="Victor Wang">victorw</who>
    <bug_when>2010-06-08 09:04:22 -0700</bug_when>
    <thetext>(In reply to comment #15)
&gt; Unless there&apos;s a good reason to separate win/mac, I think we should go forward with this patch, change the default behavior of the rebaselining tool, and file a bug to unify all the tests which have identical mac/win baselines.

I don&apos;t think we should put baselines in platform/chromium. We already have a baseline location for each platform and fallback logic works well through platform-linux-&gt;platform-&gt;win-&gt;platform-mac. I don&apos;t see much benefit if we add additional layer to this fallback path, but does see there is maintenance cost (either through rebaseline tool or manually update baselines) introduced by this extra location. Like I said, if we have this extra layer and baselines for all three platforms are same, the baseline is added to platform/chromium, then if the baseline for any platform changes, not only you need to add/update the baseline for that location, you may also need to update the other platforms and remove the file from platform/chromium. I don&apos;t mean we could not do this in rebaseline tool or manually rebaselining, but think this makes our rebaseline process more complicated and the benefit by adding this extra layer is not much.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235277</commentid>
    <comment_count>17</comment_count>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-06-08 09:18:31 -0700</bug_when>
    <thetext>(In reply to comment #16)
&gt; (In reply to comment #15)
&gt; &gt; Unless there&apos;s a good reason to separate win/mac, I think we should go forward with this patch, change the default behavior of the rebaselining tool, and file a bug to unify all the tests which have identical mac/win baselines.
&gt; 
&gt; I don&apos;t think we should put baselines in platform/chromium. We already have a baseline location for each platform and fallback logic works well through platform-linux-&gt;platform-&gt;win-&gt;platform-mac. I don&apos;t see much benefit if we add additional layer to this fallback path, but does see there is maintenance cost (either through rebaseline tool or manually update baselines) introduced by this extra location. Like I said, if we have this extra layer and baselines for all three platforms are same, the baseline is added to platform/chromium, then if the baseline for any platform changes, not only you need to add/update the baseline for that location, you may also need to update the other platforms and remove the file from platform/chromium. I don&apos;t mean we could not do this in rebaseline tool or manually rebaselining, but think this makes our rebaseline process more complicated and the benefit by adding this extra layer is not much.

By that logic, we should not have linux fall back on the win baselines for the same reason, right?

Also, if that&apos;s the case, we need to move all the baselines out of platform/chromium to avoid further confusion.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235283</commentid>
    <comment_count>18</comment_count>
    <who name="Victor Wang">victorw</who>
    <bug_when>2010-06-08 09:36:32 -0700</bug_when>
    <thetext>(In reply to comment #17)
&gt; (In reply to comment #16)
&gt; &gt; (In reply to comment #15)
&gt; &gt; &gt; Unless there&apos;s a good reason to separate win/mac, I think we should go forward with this patch, change the default behavior of the rebaselining tool, and file a bug to unify all the tests which have identical mac/win baselines.
&gt; &gt; 
&gt; &gt; I don&apos;t think we should put baselines in platform/chromium. We already have a baseline location for each platform and fallback logic works well through platform-linux-&gt;platform-&gt;win-&gt;platform-mac. I don&apos;t see much benefit if we add additional layer to this fallback path, but does see there is maintenance cost (either through rebaseline tool or manually update baselines) introduced by this extra location. Like I said, if we have this extra layer and baselines for all three platforms are same, the baseline is added to platform/chromium, then if the baseline for any platform changes, not only you need to add/update the baseline for that location, you may also need to update the other platforms and remove the file from platform/chromium. I don&apos;t mean we could not do this in rebaseline tool or manually rebaselining, but think this makes our rebaseline process more complicated and the benefit by adding this extra layer is not much.
&gt; 
&gt; By that logic, we should not have linux fall back on the win baselines for the same reason, right?
Jeremy, maybe I didn&apos;t make my comments clearer. I do think we should have linux fallback to win (lots of baselines for linux and win are same). The point I am trying to make here is each baseline location is associated with one platform (mac, win7, vista, winxp, linux etc). The new location platform/chromium is not associate with one platform, it is for hosting files if baselines for all platforms are same. This case has already been handled now by simply adding the baseline to platform-mac. Adding this new location will make the process MORE complicated and seems to me it is not necessary.

&gt; 
&gt; Also, if that&apos;s the case, we need to move all the baselines out of platform/chromium to avoid further confusion.
Yes, I agree.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235286</commentid>
    <comment_count>19</comment_count>
    <who name="Victor Wang">victorw</who>
    <bug_when>2010-06-08 09:50:01 -0700</bug_when>
    <thetext>(In reply to comment #18)
&gt; (In reply to comment #17)
&gt; &gt; (In reply to comment #16)
&gt; &gt; &gt; (In reply to comment #15)
&gt; &gt; &gt; &gt; Unless there&apos;s a good reason to separate win/mac, I think we should go forward with this patch, change the default behavior of the rebaselining tool, and file a bug to unify all the tests which have identical mac/win baselines.
&gt; &gt; &gt; 
&gt; &gt; &gt; I don&apos;t think we should put baselines in platform/chromium. We already have a baseline location for each platform and fallback logic works well through platform-linux-&gt;platform-&gt;win-&gt;platform-mac. I don&apos;t see much benefit if we add additional layer to this fallback path, but does see there is maintenance cost (either through rebaseline tool or manually update baselines) introduced by this extra location. Like I said, if we have this extra layer and baselines for all three platforms are same, the baseline is added to platform/chromium, then if the baseline for any platform changes, not only you need to add/update the baseline for that location, you may also need to update the other platforms and remove the file from platform/chromium. I don&apos;t mean we could not do this in rebaseline tool or manually rebaselining, but think this makes our rebaseline process more complicated and the benefit by adding this extra layer is not much.
&gt; &gt; 
&gt; &gt; By that logic, we should not have linux fall back on the win baselines for the same reason, right?
&gt; Jeremy, maybe I didn&apos;t make my comments clearer. I do think we should have linux fallback to win (lots of baselines for linux and win are same). The point I am trying to make here is each baseline location is associated with one platform (mac, win7, vista, winxp, linux etc). The new location platform/chromium is not associate with one platform, it is for hosting files if baselines for all platforms are same. This case has already been handled now by simply adding the baseline to platform-mac. Adding this new location will make the process MORE complicated and seems to me it is not necessary.
&gt; 
&gt; &gt; 
&gt; &gt; Also, if that&apos;s the case, we need to move all the baselines out of platform/chromium to avoid further confusion.
&gt; Yes, I agree.

I think it is more logically clear and simple if baseline for each platform are added to it own location and no dupe check if there is no storage concern, but lots of baselines are same so we need the fallback logic to save storage. Adding platform/chromium does not save us any space but does make the process more complicated.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235367</commentid>
    <comment_count>20</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-06-08 11:03:37 -0700</bug_when>
    <thetext>(In reply to comment #16)
&gt; (In reply to comment #15)
&gt; &gt; Unless there&apos;s a good reason to separate win/mac, I think we should go forward with this patch, change the default behavior of the rebaselining tool, and file a bug to unify all the tests which have identical mac/win baselines.
&gt; 
&gt; I don&apos;t think we should put baselines in platform/chromium. We already have a baseline location for each platform and fallback logic works well through platform-linux-&gt;platform-&gt;win-&gt;platform-mac. 

To repeat what I said in comment #13, neither platform/chromium-win nor platform/chromium-linux  falls back to chromium-mac, which means there is no place we can put a single file that has the same expectation on all chromium ports but is different from upstream webkit.

I think it makes sense to have such a place, because that is easy to understand for people writing new tests, and that way people don&apos;t get confused if we tell them to put a result in chromium-mac even if it is the same on all three platforms.

In addition, we know that it is something of a hassle to rebaseline all three platforms (although the rebaselining tool does make it easier than it would be by hand), and it does introduce a source of error for both committers and reviewers.

So, if I put a file in platform/chromium, and the mac baseline changes, then I don&apos;t have to remove the file from platform/chromium (linux and win will still pick it up). This is no different from how the rest of the fallback code works.

Things would be a little clearer if we removed the platform/mac (not chromium-mac) baseline, so that it was clear that the win (and linux) platforms never depended on mac-specific *anything*. I think most of the mac-specific baselines picked up by the win bots are for mac-specific tests, so arguably we could stop running those as well.

However, you&apos;re right that storage isn&apos;t a concern.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235396</commentid>
    <comment_count>21</comment_count>
    <who name="Victor Wang">victorw</who>
    <bug_when>2010-06-08 11:29:57 -0700</bug_when>
    <thetext>(In reply to comment #20)
&gt; (In reply to comment #16)
&gt; &gt; (In reply to comment #15)
&gt; &gt; &gt; Unless there&apos;s a good reason to separate win/mac, I think we should go forward with this patch, change the default behavior of the rebaselining tool, and file a bug to unify all the tests which have identical mac/win baselines.
&gt; &gt; 
&gt; &gt; I don&apos;t think we should put baselines in platform/chromium. We already have a baseline location for each platform and fallback logic works well through platform-linux-&gt;platform-&gt;win-&gt;platform-mac. 
&gt; 
&gt; To repeat what I said in comment #13, neither platform/chromium-win nor platform/chromium-linux  falls back to chromium-mac, which means there is no place we can put a single file that has the same expectation on all chromium ports but is different from upstream webkit.
&gt; 
&gt; I think it makes sense to have such a place, because that is easy to understand for people writing new tests, and that way people don&apos;t get confused if we tell them to put a result in chromium-mac even if it is the same on all three platforms.
&gt; 
&gt; In addition, we know that it is something of a hassle to rebaseline all three platforms (although the rebaselining tool does make it easier than it would be by hand), and it does introduce a source of error for both committers and reviewers.
&gt; 
&gt; So, if I put a file in platform/chromium, and the mac baseline changes, then I don&apos;t have to remove the file from platform/chromium (linux and win will still pick it up). This is no different from how the rest of the fallback code works.
&gt; 
&gt; Things would be a little clearer if we removed the platform/mac (not chromium-mac) baseline, so that it was clear that the win (and linux) platforms never depended on mac-specific *anything*. I think most of the mac-specific baselines picked up by the win bots are for mac-specific tests, so arguably we could stop running those as well.
&gt; 
&gt; However, you&apos;re right that storage isn&apos;t a concern.

I see. Somehow, I thought the chromium-win falls back to chromium-mac as we have in downstream code. This is why I thought the new platform/chromium is not that useful. If this is not the case, I agree adding platform/chromium to fallback path is fine.

Since platform/chromium is already in search path and chromium-win or chromium-linux does not fall back to chromium-mac, the existing rebaseline tool should work as you said in your comments.

Sorry for the misunderstanding.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235726</commentid>
    <comment_count>22</comment_count>
    <who name="Marcus Bulach">bulach</who>
    <bug_when>2010-06-09 03:07:16 -0700</bug_when>
    <thetext>(In reply to comment #21)
&gt; (In reply to comment #20)
&gt; &gt; (In reply to comment #16)
&gt; &gt; &gt; (In reply to comment #15)
&gt; &gt; &gt; &gt; Unless there&apos;s a good reason to separate win/mac, I think we should go forward with this patch, change the default behavior of the rebaselining tool, and file a bug to unify all the tests which have identical mac/win baselines.
&gt; &gt; &gt; 
&gt; &gt; &gt; I don&apos;t think we should put baselines in platform/chromium. We already have a baseline location for each platform and fallback logic works well through platform-linux-&gt;platform-&gt;win-&gt;platform-mac. 
&gt; &gt; 
&gt; &gt; To repeat what I said in comment #13, neither platform/chromium-win nor platform/chromium-linux  falls back to chromium-mac, which means there is no place we can put a single file that has the same expectation on all chromium ports but is different from upstream webkit.
&gt; &gt; 
&gt; &gt; I think it makes sense to have such a place, because that is easy to understand for people writing new tests, and that way people don&apos;t get confused if we tell them to put a result in chromium-mac even if it is the same on all three platforms.
&gt; &gt; 
&gt; &gt; In addition, we know that it is something of a hassle to rebaseline all three platforms (although the rebaselining tool does make it easier than it would be by hand), and it does introduce a source of error for both committers and reviewers.
&gt; &gt; 
&gt; &gt; So, if I put a file in platform/chromium, and the mac baseline changes, then I don&apos;t have to remove the file from platform/chromium (linux and win will still pick it up). This is no different from how the rest of the fallback code works.
&gt; &gt; 
&gt; &gt; Things would be a little clearer if we removed the platform/mac (not chromium-mac) baseline, so that it was clear that the win (and linux) platforms never depended on mac-specific *anything*. I think most of the mac-specific baselines picked up by the win bots are for mac-specific tests, so arguably we could stop running those as well.
&gt; &gt; 
&gt; &gt; However, you&apos;re right that storage isn&apos;t a concern.
&gt; 
&gt; I see. Somehow, I thought the chromium-win falls back to chromium-mac as we have in downstream code. This is why I thought the new platform/chromium is not that useful. If this is not the case, I agree adding platform/chromium to fallback path is fine.
&gt; 
&gt; Since platform/chromium is already in search path and chromium-win or chromium-linux does not fall back to chromium-mac, the existing rebaseline tool should work as you said in your comments.
&gt; 
&gt; Sorry for the misunderstanding.

thanks all for the clarifications!
so should we go with this patch as is then?! :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>235729</commentid>
    <comment_count>23</comment_count>
      <attachid>57324</attachid>
    <who name="Jeremy Orlow">jorlow</who>
    <bug_when>2010-06-09 03:10:28 -0700</bug_when>
    <thetext>Comment on attachment 57324
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>236354</commentid>
    <comment_count>24</comment_count>
      <attachid>57324</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-06-10 07:16:58 -0700</bug_when>
    <thetext>Comment on attachment 57324
Patch

Clearing flags on attachment: 57324

Committed r60956: &lt;http://trac.webkit.org/changeset/60956&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>236355</commentid>
    <comment_count>25</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-06-10 07:17:05 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>57093</attachid>
            <date>2010-05-26 07:30:24 -0700</date>
            <delta_ts>2010-05-28 06:40:01 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-39721-20100526153022.patch</filename>
            <type>text/plain</type>
            <size>1518</size>
            <attacher name="Marcus Bulach">bulach</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCA1MDk3ODdjMDc4NmQ4MmIxMzQ5ZGQ5NWEwMTVkNDQzNjIzNmFiNTVhLi4xNGEyYWIy
YWU2ZjkyNWZjNjg4Y2U2ZmQ0NTgwZGJiMTNkMGU4MGViIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAK
KzIwMTAtMDUtMjYgIE1hcmN1cyBCdWxhY2ggIDxidWxhY2hAZ29vZ2xlLmNvbT4KKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbY2hyb21pdW1dIFVwc3Ry
ZWFtIGxheW91dCB0ZXN0cyBleHBlY3RhdGlvbnMgZm9yIEdlb2xvY2F0aW9uLgorICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mzk3MjEKKworICAgICAgICAq
IHBsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0YXRpb25zLnR4dDoKKwogMjAxMC0wNS0yNiAg
QWxleGFuZGVyIFBhdmxvdiAgPGFwYXZsb3ZAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmll
d2VkIGJ5IFBhdmVsIEZlbGRtYW4uCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9j
aHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJv
bWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQKaW5kZXggYzQwYmIzMTVkMDZhZmZhMTM5NmM4YjBl
M2ZmMGY5MmYzMDY2MmVlOS4uMDYwMzY0OGRmYWM0NGM2MjRiMjM1MGIyZjgzZjVjYTEwNTdiN2Zj
MiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vY2hyb21pdW0vdGVzdF9leHBlY3Rh
dGlvbnMudHh0CisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0
YXRpb25zLnR4dApAQCAtNzA0LDkgKzcwNCw5IEBAIEJVRzI3Njk3IFNLSVAgOiBzdG9yYWdlL2hh
c2gtY2hhbmdlLXdpdGgteGhyLmh0bWwgPSBQQVNTCiAvLyBXZSBkbyBub3QgdXNlIFNhZmFyaSdz
IHByaXZhdGUgYnJvd3NpbmcKIFdPTlRGSVggU0tJUCA6IHN0b3JhZ2UvcHJpdmF0ZS1icm93c2lu
Zy1yZWFkb25seS5odG1sID0gUEFTUwogCi0vLyBJbXBsZW1lbnQgSFRNTDUgR2VvbG9jYXRpb24g
QVBJLgotQlVHMTEyNDYgREVGRVIgU0tJUCA6IGZhc3QvZG9tL0dlb2xvY2F0aW9uID0gRkFJTAot
QlVHMTEyNDYgREVGRVIgU0tJUCA6IGZhc3QvZG9tL1dpbmRvdy93aW5kb3ctcHJvcGVydGllcy1n
ZW9sb2NhdGlvbi5odG1sID0gVEVYVAorLy8gSFRNTDUgR2VvbG9jYXRpb24gQVBJLgorLy8gTWlz
bWF0Y2ggYmV0d2VlbiBWOCBhbmQgSlNDIG1lc3NhZ2UuCitCVUcxMTI0NiBERUZFUiBTS0lQIDog
ZmFzdC9kb20vR2VvbG9jYXRpb24vY2FsbGJhY2stZXhjZXB0aW9uLmh0bWwgPSBURVhUCiAKIC8v
IEhUTUw1IGRhdGFsaXN0IGVsZW1lbnQuIFdlIGRvbid0IGVuYWJsZSBpdCBiZWNhdXNlIFdlYktp
dCBpbXBsZW1lbnRhdGlvbgogLy8gaXMgaW5jb21wbGV0ZS4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>57324</attachid>
            <date>2010-05-28 06:40:07 -0700</date>
            <delta_ts>2010-06-10 07:16:58 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-39721-20100528144005.patch</filename>
            <type>text/plain</type>
            <size>2554</size>
            <attacher name="Marcus Bulach">bulach</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCA0NDRkNzQ4NjA2ZDMwZTQ5YTk5YjEwNzMzMzgyOGRhNTgxYWRiZGVhLi44ZDE3ZGJm
MmVlMTRjNDAwNDQwMzkwYThlMTA1MTc4OGMxNWIyN2M3IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAK
KzIwMTAtMDUtMjggIE1hcmN1cyBCdWxhY2ggIDxidWxhY2hAZ29vZ2xlLmNvbT4KKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbY2hyb21pdW1dIFVwc3Ry
ZWFtIGxheW91dCB0ZXN0cyBleHBlY3RhdGlvbnMgZm9yIEdlb2xvY2F0aW9uLgorICAgICAgICBS
ZWJhc2VsaW5lIGZhc3QvZG9tL0dlb2xvY2F0aW9uL2NhbGxiYWNrLWV4Y2VwdGlvbi1leHBlY3Rl
ZC5odG1sIGR1ZSB0byBKU0MgeCBWOCBtZXNzYWdlcy4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndl
YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM5NzIxCisKKyAgICAgICAgKiBwbGF0Zm9ybS9jaHJv
bWl1bS9mYXN0L2RvbS9HZW9sb2NhdGlvbi9jYWxsYmFjay1leGNlcHRpb24tZXhwZWN0ZWQudHh0
OiBBZGRlZC4KKyAgICAgICAgKiBwbGF0Zm9ybS9jaHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50
eHQ6CisKIDIwMTAtMDUtMjcgIE1PUklUQSBIYWppbWUgIDxtb3JyaXRhQGdvb2dsZS5jb20+CiAK
ICAgICAgICAgUmV2aWV3ZWQgYnkgT2phbiBWYWZhaS4KZGlmZiAtLWdpdCBhL0xheW91dFRlc3Rz
L3BsYXRmb3JtL2Nocm9taXVtL2Zhc3QvZG9tL0dlb2xvY2F0aW9uL2NhbGxiYWNrLWV4Y2VwdGlv
bi1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJvbWl1bS9mYXN0L2RvbS9H
ZW9sb2NhdGlvbi9jYWxsYmFjay1leGNlcHRpb24tZXhwZWN0ZWQudHh0Cm5ldyBmaWxlIG1vZGUg
MTAwNjQ0CmluZGV4IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLmVl
Y2UxNGZmMjg0Mzg4ZjU2NDlhOWMwZGY0OWU4MjhhNTE5OWM2YmIKLS0tIC9kZXYvbnVsbAorKysg
Yi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9jaHJvbWl1bS9mYXN0L2RvbS9HZW9sb2NhdGlvbi9jYWxs
YmFjay1leGNlcHRpb24tZXhwZWN0ZWQudHh0CkBAIC0wLDAgKzEsMTMgQEAKK0NPTlNPTEUgTUVT
U0FHRTogbGluZSAyMjogVW5jYXVnaHQgRXJyb3I6IEV4Y2VwdGlvbiBpbiBzdWNjZXNzIGNhbGxi
YWNrCitUZXN0cyB0aGF0IHdoZW4gYW4gZXhjZXB0aW9uIGlzIHRocm93biBpbiB0aGUgc3VjY2Vz
cyBjYWxsYmFjaywgdGhlIGVycm9yIGNhbGxiYWNrIGlzIG5vdCBpbnZva2VkLiBOb3RlIHRoYXQg
dGhpcyB0ZXN0IHRocm93cyBhbiBleGNlcHRpb24gd2hpY2ggaXMgbm90IGNhdWdodC4KKworT24g
c3VjY2VzcywgeW91IHdpbGwgc2VlIGEgc2VyaWVzIG9mICJQQVNTIiBtZXNzYWdlcywgZm9sbG93
ZWQgYnkgIlRFU1QgQ09NUExFVEUiLgorCisKK1BBU1MgcG9zaXRpb24uY29vcmRzLmxhdGl0dWRl
IGlzIG1vY2tMYXRpdHVkZQorUEFTUyBwb3NpdGlvbi5jb29yZHMubG9uZ2l0dWRlIGlzIG1vY2tM
b25naXR1ZGUKK1BBU1MgcG9zaXRpb24uY29vcmRzLmFjY3VyYWN5IGlzIG1vY2tBY2N1cmFjeQor
UEFTUyBzdWNjZXNzZnVsbHlQYXJzZWQgaXMgdHJ1ZQorCitURVNUIENPTVBMRVRFCisKZGlmZiAt
LWdpdCBhL0xheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0YXRpb25zLnR4
dCBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0YXRpb25zLnR4dApp
bmRleCA5MmJhODY3M2U5MDE0OWY2MzEzMjBiYzVhZmMzM2QwN2FmNmVhYzcwLi41MzZiOGVkMjk4
Y2Q3OTZmYzdiMzk1ZDFjMmUzZGEyYjY0NjczMWFiIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9w
bGF0Zm9ybS9jaHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQKKysrIGIvTGF5b3V0VGVzdHMv
cGxhdGZvcm0vY2hyb21pdW0vdGVzdF9leHBlY3RhdGlvbnMudHh0CkBAIC03MDQsMTAgKzcwNCw2
IEBAIEJVRzI3Njk3IFNLSVAgOiBzdG9yYWdlL2hhc2gtY2hhbmdlLXdpdGgteGhyLmh0bWwgPSBQ
QVNTCiAvLyBXZSBkbyBub3QgdXNlIFNhZmFyaSdzIHByaXZhdGUgYnJvd3NpbmcKIFdPTlRGSVgg
U0tJUCA6IHN0b3JhZ2UvcHJpdmF0ZS1icm93c2luZy1yZWFkb25seS5odG1sID0gUEFTUwogCi0v
LyBJbXBsZW1lbnQgSFRNTDUgR2VvbG9jYXRpb24gQVBJLgotQlVHMTEyNDYgREVGRVIgU0tJUCA6
IGZhc3QvZG9tL0dlb2xvY2F0aW9uID0gRkFJTAotQlVHMTEyNDYgREVGRVIgU0tJUCA6IGZhc3Qv
ZG9tL1dpbmRvdy93aW5kb3ctcHJvcGVydGllcy1nZW9sb2NhdGlvbi5odG1sID0gVEVYVAotCiAv
LyBIVE1MNSBkYXRhbGlzdCBlbGVtZW50LiBXZSBkb24ndCBlbmFibGUgaXQgYmVjYXVzZSBXZWJL
aXQgaW1wbGVtZW50YXRpb24KIC8vIGlzIGluY29tcGxldGUuCiBCVUcyMDIyNiBERUZFUiA6IGZh
c3QvZm9ybXMvZGF0YWxpc3Qtbm9ub3B0aW9uLWNoaWxkLmh0bWwgPSBURVhUCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>