<?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>225763</bug_id>
          
          <creation_ts>2021-05-13 04:59:12 -0700</creation_ts>
          <short_desc>[GTK] MiniBrowser: add support for saving multiple tabs in session state</short_desc>
          <delta_ts>2021-06-04 03:43:50 -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>WebKitGTK</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Garcia Campos">cgarcia</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>mcatanzaro</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1759647</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2021-05-13 04:59:12 -0700</bug_when>
    <thetext>We currently save the active tab when the window is closed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1759648</commentid>
    <comment_count>1</comment_count>
      <attachid>428489</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2021-05-13 05:02:07 -0700</bug_when>
    <thetext>Created attachment 428489
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1759683</commentid>
    <comment_count>2</comment_count>
      <attachid>428489</attachid>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2021-05-13 07:14:58 -0700</bug_when>
    <thetext>Comment on attachment 428489
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=428489&amp;action=review

&gt; Tools/MiniBrowser/gtk/BrowserWindow.c:1463
&gt; +    if (!g_key_file_load_from_file(session, sessionFile, G_KEY_FILE_NONE, NULL)) {

I would check and print a warning if !g_error_matches(error, G_FILE_ERROR, G_FILE_ERROR_NOENT). I think that&apos;s the only case where failure to restore the session is expected. If the session state file contains data and we fail to restore it, then that&apos;s probably a bug and warning developers is good to do.

&gt; Tools/MiniBrowser/gtk/BrowserWindow.c:1476
&gt; +
&gt; +        return;

This extra blank line doesn&apos;t seem like your usual code style?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1766727</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2021-06-04 03:43:50 -0700</bug_when>
    <thetext>Committed r278451 (238471@main): &lt;https://commits.webkit.org/238471@main&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>428489</attachid>
            <date>2021-05-13 05:02:07 -0700</date>
            <delta_ts>2021-05-13 07:14:58 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>mb-session.diff</filename>
            <type>text/plain</type>
            <size>7737</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCAxZDBj
NzIzMDk3YzguLmQ0ZTI0MjBiOTU5MCAxMDA2NDQKLS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBi
L1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDIxLTA1LTEzICBDYXJsb3MgR2Fy
Y2lhIENhbXBvcyAgPGNnYXJjaWFAaWdhbGlhLmNvbT4KKworICAgICAgICBbR1RLXSBNaW5pQnJv
d3NlcjogYWRkIHN1cHBvcnQgZm9yIHNhdmluZyBtdWx0aXBsZSB0YWJzIGluIHNlc3Npb24gc3Rh
dGUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIyNTc2
MworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogTWlu
aUJyb3dzZXIvZ3RrL0Jyb3dzZXJXaW5kb3cuYzoKKyAgICAgICAgKGJyb3dzZXJXaW5kb3dTYXZl
U2Vzc2lvbik6CisgICAgICAgIChicm93c2VyV2luZG93VHJ5Q2xvc2UpOgorICAgICAgICAoYnJv
d3NlcldpbmRvd0ZpbmFsaXplKToKKyAgICAgICAgKGJyb3dzZXJXaW5kb3dEaXNwb3NlKToKKyAg
ICAgICAgKGJyb3dzZXJfd2luZG93X2xvYWRfc2Vzc2lvbik6CisKIDIwMjEtMDUtMTIgIFNpbW9u
IEZyYXNlciAgPHNpbW9uLmZyYXNlckBhcHBsZS5jb20+CiAKICAgICAgICAgVXBkYXRlIHRoZSBN
b3Rpb25NYXJrIDEuMSBwbGFuIGZpbGUKZGlmZiAtLWdpdCBhL1Rvb2xzL01pbmlCcm93c2VyL2d0
ay9Ccm93c2VyV2luZG93LmMgYi9Ub29scy9NaW5pQnJvd3Nlci9ndGsvQnJvd3NlcldpbmRvdy5j
CmluZGV4IGQxZWI4OWFkOTlmZS4uNjgxZGIwOGFjNDJmIDEwMDY0NAotLS0gYS9Ub29scy9NaW5p
QnJvd3Nlci9ndGsvQnJvd3NlcldpbmRvdy5jCisrKyBiL1Rvb2xzL01pbmlCcm93c2VyL2d0ay9C
cm93c2VyV2luZG93LmMKQEAgLTMzMSw2ICszMzEsMzcgQEAgc3RhdGljIGdib29sZWFuIG5hdmln
YXRpb25CdXR0b25QcmVzc0NhbGxiYWNrKEd0a0J1dHRvbiAqYnV0dG9uLCBHZGtFdmVudCAqZXZl
bnQKIH0KICNlbmRpZgogCitzdGF0aWMgdm9pZCBicm93c2VyV2luZG93U2F2ZVNlc3Npb24oQnJv
d3NlcldpbmRvdyAqd2luZG93KQoreworICAgIGlmICghd2luZG93LT5zZXNzaW9uRmlsZSkKKyAg
ICAgICAgcmV0dXJuOworCisgICAgR0tleUZpbGUgKnNlc3Npb24gPSBnX2tleV9maWxlX25ldygp
OworICAgIGludCB0YWJzQ291bnQgPSBndGtfbm90ZWJvb2tfZ2V0X25fcGFnZXMoR1RLX05PVEVC
T09LKHdpbmRvdy0+bm90ZWJvb2spKTsKKyAgICBpbnQgaTsKKyAgICBmb3IgKGkgPSAwOyBpIDwg
dGFic0NvdW50OyArK2kpIHsKKyAgICAgICAgQnJvd3NlclRhYiAqdGFiID0gKEJyb3dzZXJUYWIg
KilndGtfbm90ZWJvb2tfZ2V0X250aF9wYWdlKEdUS19OT1RFQk9PSyh3aW5kb3ctPm5vdGVib29r
KSwgaSk7CisgICAgICAgIFdlYktpdFdlYlZpZXcgKndlYlZpZXcgPSBicm93c2VyX3RhYl9nZXRf
d2ViX3ZpZXcodGFiKTsKKyAgICAgICAgV2ViS2l0V2ViVmlld1Nlc3Npb25TdGF0ZSAqc3RhdGUg
PSB3ZWJraXRfd2ViX3ZpZXdfZ2V0X3Nlc3Npb25fc3RhdGUod2ViVmlldyk7CisgICAgICAgIEdC
eXRlcyAqYnl0ZXMgPSB3ZWJraXRfd2ViX3ZpZXdfc2Vzc2lvbl9zdGF0ZV9zZXJpYWxpemUoc3Rh
dGUpOworICAgICAgICBpZiAoYnl0ZXMpIHsKKyAgICAgICAgICAgIGdzaXplIGRhdGFMZW5ndGg7
CisgICAgICAgICAgICBnY29uc3Rwb2ludGVyIGRhdGE7CisgICAgICAgICAgICBkYXRhID0gZ19i
eXRlc19nZXRfZGF0YShieXRlcywgJmRhdGFMZW5ndGgpOworCisgICAgICAgICAgICBnY2hhciAq
Z3JvdXBOYW1lID0gZ19zdHJkdXBfcHJpbnRmKCJUYWItJWQiLCBpKTsKKyAgICAgICAgICAgIGdj
aGFyICpiYXNlNjQgPSBnX2Jhc2U2NF9lbmNvZGUoZGF0YSwgZGF0YUxlbmd0aCk7CisgICAgICAg
ICAgICBnX2tleV9maWxlX3NldF9zdHJpbmcoc2Vzc2lvbiwgZ3JvdXBOYW1lLCAic3RhdGUiLCBi
YXNlNjQpOworICAgICAgICAgICAgZ19mcmVlKGJhc2U2NCk7CisgICAgICAgICAgICBnX2ZyZWUo
Z3JvdXBOYW1lKTsKKyAgICAgICAgICAgIGdfYnl0ZXNfdW5yZWYoYnl0ZXMpOworICAgICAgICB9
CisgICAgICAgIHdlYmtpdF93ZWJfdmlld19zZXNzaW9uX3N0YXRlX3VucmVmKHN0YXRlKTsKKyAg
ICB9CisgICAgZ19rZXlfZmlsZV9zYXZlX3RvX2ZpbGUoc2Vzc2lvbiwgd2luZG93LT5zZXNzaW9u
RmlsZSwgTlVMTCk7CisgICAgZ19rZXlfZmlsZV9mcmVlKHNlc3Npb24pOworfQorCiBzdGF0aWMg
dm9pZCBicm93c2VyV2luZG93VHJ5Q2xvc2VDdXJyZW50V2ViVmlldyhHU2ltcGxlQWN0aW9uICph
Y3Rpb24sIEdWYXJpYW50ICpwYXJhbWV0ZXIsIGdwb2ludGVyIHVzZXJEYXRhKQogewogICAgIEJy
b3dzZXJXaW5kb3cgKndpbmRvdyA9IEJST1dTRVJfV0lORE9XKHVzZXJEYXRhKTsKQEAgLTM0Miw2
ICszNzMsOCBAQCBzdGF0aWMgdm9pZCBicm93c2VyV2luZG93VHJ5Q2xvc2VDdXJyZW50V2ViVmll
dyhHU2ltcGxlQWN0aW9uICphY3Rpb24sIEdWYXJpYW50CiBzdGF0aWMgdm9pZCBicm93c2VyV2lu
ZG93VHJ5Q2xvc2UoR1NpbXBsZUFjdGlvbiAqYWN0aW9uLCBHVmFyaWFudCAqcGFyYW1ldGVyLCBn
cG9pbnRlciB1c2VyRGF0YSkKIHsKICAgICBCcm93c2VyV2luZG93ICp3aW5kb3cgPSBCUk9XU0VS
X1dJTkRPVyh1c2VyRGF0YSk7CisgICAgYnJvd3NlcldpbmRvd1NhdmVTZXNzaW9uKHdpbmRvdyk7
CisKICAgICBHU0xpc3QgKndlYlZpZXdzID0gTlVMTDsKICAgICBpbnQgbiA9IGd0a19ub3RlYm9v
a19nZXRfbl9wYWdlcyhHVEtfTk9URUJPT0sod2luZG93LT5ub3RlYm9vaykpOwogICAgIGludCBp
OwpAQCAtODU2LDIwICs4ODksNiBAQCBzdGF0aWMgdm9pZCB0eXBpbmdBdHRyaWJ1dGVzQ2hhbmdl
ZChXZWJLaXRFZGl0b3JTdGF0ZSAqZWRpdG9yU3RhdGUsIEdQYXJhbVNwZWMgKgogICAgIGdfc2lt
cGxlX2FjdGlvbl9zZXRfc3RhdGUoR19TSU1QTEVfQUNUSU9OKGFjdGlvbiksIGdfdmFyaWFudF9u
ZXdfYm9vbGVhbih0eXBpbmdBdHRyaWJ1dGVzICYgV0VCS0lUX0VESVRPUl9UWVBJTkdfQVRUUklC
VVRFX1NUUklLRVRIUk9VR0gpKTsKIH0KIAotc3RhdGljIHZvaWQgYnJvd3NlcldpbmRvd1NhdmVT
ZXNzaW9uKEJyb3dzZXJXaW5kb3cgKndpbmRvdykKLXsKLSAgICBpZiAoIXdpbmRvdy0+c2Vzc2lv
bkZpbGUpCi0gICAgICAgIHJldHVybjsKLQotICAgIFdlYktpdFdlYlZpZXcgKndlYlZpZXcgPSBi
cm93c2VyX3RhYl9nZXRfd2ViX3ZpZXcod2luZG93LT5hY3RpdmVUYWIpOwotICAgIFdlYktpdFdl
YlZpZXdTZXNzaW9uU3RhdGUgKnN0YXRlID0gd2Via2l0X3dlYl92aWV3X2dldF9zZXNzaW9uX3N0
YXRlKHdlYlZpZXcpOwotICAgIEdCeXRlcyAqYnl0ZXMgPSB3ZWJraXRfd2ViX3ZpZXdfc2Vzc2lv
bl9zdGF0ZV9zZXJpYWxpemUoc3RhdGUpOwotICAgIHdlYmtpdF93ZWJfdmlld19zZXNzaW9uX3N0
YXRlX3VucmVmKHN0YXRlKTsKLSAgICBnX2ZpbGVfc2V0X2NvbnRlbnRzKHdpbmRvdy0+c2Vzc2lv
bkZpbGUsIGdfYnl0ZXNfZ2V0X2RhdGEoYnl0ZXMsIE5VTEwpLCBnX2J5dGVzX2dldF9zaXplKGJ5
dGVzKSwgTlVMTCk7Ci0gICAgZ19ieXRlc191bnJlZihieXRlcyk7Ci0gICAgZ19jbGVhcl9wb2lu
dGVyKCZ3aW5kb3ctPnNlc3Npb25GaWxlLCBnX2ZyZWUpOwotfQotCiBzdGF0aWMgdm9pZCBicm93
c2VyV2luZG93RmluYWxpemUoR09iamVjdCAqZ09iamVjdCkKIHsKICAgICBCcm93c2VyV2luZG93
ICp3aW5kb3cgPSBCUk9XU0VSX1dJTkRPVyhnT2JqZWN0KTsKQEAgLTg4Niw2ICs5MDUsNyBAQCBz
dGF0aWMgdm9pZCBicm93c2VyV2luZG93RmluYWxpemUoR09iamVjdCAqZ09iamVjdCkKICAgICAg
ICAgZ19zb3VyY2VfcmVtb3ZlKHdpbmRvdy0+cmVzZXRFbnRyeVByb2dyZXNzVGltZW91dElkKTsK
IAogICAgIGdfY2xlYXJfb2JqZWN0KCZ3aW5kb3ctPmVkaXRBY3Rpb25Hcm91cCk7CisgICAgZ19j
bGVhcl9wb2ludGVyKCZ3aW5kb3ctPnNlc3Npb25GaWxlLCBnX2ZyZWUpOwogCiAgICAgR19PQkpF
Q1RfQ0xBU1MoYnJvd3Nlcl93aW5kb3dfcGFyZW50X2NsYXNzKS0+ZmluYWxpemUoZ09iamVjdCk7
CiB9CkBAIC04OTQsOCArOTE0LDYgQEAgc3RhdGljIHZvaWQgYnJvd3NlcldpbmRvd0Rpc3Bvc2Uo
R09iamVjdCAqZ09iamVjdCkKIHsKICAgICBCcm93c2VyV2luZG93ICp3aW5kb3cgPSBCUk9XU0VS
X1dJTkRPVyhnT2JqZWN0KTsKIAotICAgIGJyb3dzZXJXaW5kb3dTYXZlU2Vzc2lvbih3aW5kb3cp
OwotCiAgICAgaWYgKHdpbmRvdy0+cGFyZW50V2luZG93KSB7CiAgICAgICAgIGdfb2JqZWN0X3Jl
bW92ZV93ZWFrX3BvaW50ZXIoR19PQkpFQ1Qod2luZG93LT5wYXJlbnRXaW5kb3cpLCAoZ3BvaW50
ZXIgKikmd2luZG93LT5wYXJlbnRXaW5kb3cpOwogICAgICAgICB3aW5kb3ctPnBhcmVudFdpbmRv
dyA9IE5VTEw7CkBAIC0xNDM5LDI3ICsxNDU3LDY3IEBAIHZvaWQgYnJvd3Nlcl93aW5kb3dfbG9h
ZF9zZXNzaW9uKEJyb3dzZXJXaW5kb3cgKndpbmRvdywgY29uc3QgY2hhciAqc2Vzc2lvbkZpbGUp
CiAgICAgZ19yZXR1cm5faWZfZmFpbChzZXNzaW9uRmlsZSk7CiAKICAgICBXZWJLaXRXZWJWaWV3
ICp3ZWJWaWV3ID0gYnJvd3Nlcl90YWJfZ2V0X3dlYl92aWV3KHdpbmRvdy0+YWN0aXZlVGFiKTsK
KwogICAgIHdpbmRvdy0+c2Vzc2lvbkZpbGUgPSBnX3N0cmR1cChzZXNzaW9uRmlsZSk7Ci0gICAg
Z2NoYXIgKmRhdGEgPSBOVUxMOwotICAgIGdzaXplIGRhdGFMZW5ndGg7Ci0gICAgaWYgKGdfZmls
ZV9nZXRfY29udGVudHMoc2Vzc2lvbkZpbGUsICZkYXRhLCAmZGF0YUxlbmd0aCwgTlVMTCkpIHsK
LSAgICAgICAgR0J5dGVzICpieXRlcyA9IGdfYnl0ZXNfbmV3X3Rha2UoZGF0YSwgZGF0YUxlbmd0
aCk7Ci0gICAgICAgIFdlYktpdFdlYlZpZXdTZXNzaW9uU3RhdGUgKnN0YXRlID0gd2Via2l0X3dl
Yl92aWV3X3Nlc3Npb25fc3RhdGVfbmV3KGJ5dGVzKTsKLSAgICAgICAgZ19ieXRlc191bnJlZihi
eXRlcyk7CisgICAgR0tleUZpbGUgKnNlc3Npb24gPSBnX2tleV9maWxlX25ldygpOworICAgIGlm
ICghZ19rZXlfZmlsZV9sb2FkX2Zyb21fZmlsZShzZXNzaW9uLCBzZXNzaW9uRmlsZSwgR19LRVlf
RklMRV9OT05FLCBOVUxMKSkgeworICAgICAgICB3ZWJraXRfd2ViX3ZpZXdfbG9hZF91cmkod2Vi
VmlldywgQlJPV1NFUl9ERUZBVUxUX1VSTCk7CisgICAgICAgIGdfa2V5X2ZpbGVfZnJlZShzZXNz
aW9uKTsKKyAgICAgICAgcmV0dXJuOworICAgIH0KKworICAgIGdzaXplIGdyb3VwQ291bnQ7Cisg
ICAgZ2NoYXIgKipncm91cHMgPSBnX2tleV9maWxlX2dldF9ncm91cHMoc2Vzc2lvbiwgJmdyb3Vw
Q291bnQpOworICAgIGlmICghZ3JvdXBDb3VudCkgeworICAgICAgICB3ZWJraXRfd2ViX3ZpZXdf
bG9hZF91cmkod2ViVmlldywgQlJPV1NFUl9ERUZBVUxUX1VSTCk7CisgICAgICAgIGdfc3RyZnJl
ZXYoZ3JvdXBzKTsKKyAgICAgICAgZ19rZXlfZmlsZV9mcmVlKHNlc3Npb24pOworCisgICAgICAg
IHJldHVybjsKKyAgICB9CisKKyAgICBXZWJLaXRXZWJWaWV3ICpwcmV2aW91c1dlYlZpZXcgPSBO
VUxMOworICAgIGdzaXplIGk7CisgICAgZm9yIChpID0gMDsgaSA8IGdyb3VwQ291bnQ7ICsraSkg
eworICAgICAgICBXZWJLaXRXZWJWaWV3U2Vzc2lvblN0YXRlICpzdGF0ZSA9IE5VTEw7CisgICAg
ICAgIGdjaGFyICpiYXNlNjQgPSBnX2tleV9maWxlX2dldF9zdHJpbmcoc2Vzc2lvbiwgZ3JvdXBz
W2ldLCAic3RhdGUiLCBOVUxMKTsKKyAgICAgICAgaWYgKGJhc2U2NCkgeworICAgICAgICAgICAg
Z3NpemUgc3RhdGVEYXRhTGVuZ3RoOworICAgICAgICAgICAgZ3VjaGFyICpzdGF0ZURhdGEgPSBn
X2Jhc2U2NF9kZWNvZGUoYmFzZTY0LCAmc3RhdGVEYXRhTGVuZ3RoKTsKKyAgICAgICAgICAgIEdC
eXRlcyAqYnl0ZXMgPSBnX2J5dGVzX25ld190YWtlKHN0YXRlRGF0YSwgc3RhdGVEYXRhTGVuZ3Ro
KTsKKyAgICAgICAgICAgIHN0YXRlID0gd2Via2l0X3dlYl92aWV3X3Nlc3Npb25fc3RhdGVfbmV3
KGJ5dGVzKTsKKyAgICAgICAgICAgIGdfYnl0ZXNfdW5yZWYoYnl0ZXMpOworICAgICAgICAgICAg
Z19mcmVlKGJhc2U2NCk7CisgICAgICAgIH0KKworICAgICAgICBpZiAoIXdlYlZpZXcpIHsKKyAg
ICAgICAgICAgIHdlYlZpZXcgPSBXRUJLSVRfV0VCX1ZJRVcoZ19vYmplY3RfbmV3KFdFQktJVF9U
WVBFX1dFQl9WSUVXLAorICAgICAgICAgICAgICAgICJ3ZWItY29udGV4dCIsIHdlYmtpdF93ZWJf
dmlld19nZXRfY29udGV4dChwcmV2aW91c1dlYlZpZXcpLAorICAgICAgICAgICAgICAgICJzZXR0
aW5ncyIsIHdlYmtpdF93ZWJfdmlld19nZXRfc2V0dGluZ3MocHJldmlvdXNXZWJWaWV3KSwKKyAg
ICAgICAgICAgICAgICAidXNlci1jb250ZW50LW1hbmFnZXIiLCB3ZWJraXRfd2ViX3ZpZXdfZ2V0
X3VzZXJfY29udGVudF9tYW5hZ2VyKHByZXZpb3VzV2ViVmlldyksCisgICAgICAgICAgICAgICAg
IndlYnNpdGUtcG9saWNpZXMiLCB3ZWJraXRfd2ViX3ZpZXdfZ2V0X3dlYnNpdGVfcG9saWNpZXMo
cHJldmlvdXNXZWJWaWV3KSwKKyAgICAgICAgICAgICAgICBOVUxMKSk7CisgICAgICAgICAgICBi
cm93c2VyX3dpbmRvd19hcHBlbmRfdmlldyh3aW5kb3csIHdlYlZpZXcpOworICAgICAgICB9CiAK
ICAgICAgICAgaWYgKHN0YXRlKSB7CiAgICAgICAgICAgICB3ZWJraXRfd2ViX3ZpZXdfcmVzdG9y
ZV9zZXNzaW9uX3N0YXRlKHdlYlZpZXcsIHN0YXRlKTsKICAgICAgICAgICAgIHdlYmtpdF93ZWJf
dmlld19zZXNzaW9uX3N0YXRlX3VucmVmKHN0YXRlKTsKICAgICAgICAgfQotICAgIH0KIAotICAg
IFdlYktpdEJhY2tGb3J3YXJkTGlzdCAqYmZMaXN0ID0gd2Via2l0X3dlYl92aWV3X2dldF9iYWNr
X2ZvcndhcmRfbGlzdCh3ZWJWaWV3KTsKLSAgICBXZWJLaXRCYWNrRm9yd2FyZExpc3RJdGVtICpp
dGVtID0gd2Via2l0X2JhY2tfZm9yd2FyZF9saXN0X2dldF9jdXJyZW50X2l0ZW0oYmZMaXN0KTsK
LSAgICBpZiAoaXRlbSkKLSAgICAgICAgd2Via2l0X3dlYl92aWV3X2dvX3RvX2JhY2tfZm9yd2Fy
ZF9saXN0X2l0ZW0od2ViVmlldywgaXRlbSk7Ci0gICAgZWxzZQotICAgICAgICB3ZWJraXRfd2Vi
X3ZpZXdfbG9hZF91cmkod2ViVmlldywgQlJPV1NFUl9ERUZBVUxUX1VSTCk7CisgICAgICAgIFdl
YktpdEJhY2tGb3J3YXJkTGlzdCAqYmZMaXN0ID0gd2Via2l0X3dlYl92aWV3X2dldF9iYWNrX2Zv
cndhcmRfbGlzdCh3ZWJWaWV3KTsKKyAgICAgICAgV2ViS2l0QmFja0ZvcndhcmRMaXN0SXRlbSAq
aXRlbSA9IHdlYmtpdF9iYWNrX2ZvcndhcmRfbGlzdF9nZXRfY3VycmVudF9pdGVtKGJmTGlzdCk7
CisgICAgICAgIGlmIChpdGVtKQorICAgICAgICAgICAgd2Via2l0X3dlYl92aWV3X2dvX3RvX2Jh
Y2tfZm9yd2FyZF9saXN0X2l0ZW0od2ViVmlldywgaXRlbSk7CisgICAgICAgIGVsc2UKKyAgICAg
ICAgICAgIHdlYmtpdF93ZWJfdmlld19sb2FkX3VyaSh3ZWJWaWV3LCAiYWJvdXQ6YmxhbmsiKTsK
KworICAgICAgICBwcmV2aW91c1dlYlZpZXcgPSB3ZWJWaWV3OworICAgICAgICB3ZWJWaWV3ID0g
TlVMTDsKKyAgICB9CiAKKyAgICBnX3N0cmZyZWV2KGdyb3Vwcyk7CisgICAgZ19rZXlfZmlsZV9m
cmVlKHNlc3Npb24pOwogfQogCiB2b2lkIGJyb3dzZXJfd2luZG93X3NldF9iYWNrZ3JvdW5kX2Nv
bG9yKEJyb3dzZXJXaW5kb3cgKndpbmRvdywgR2RrUkdCQSAqcmdiYSkK
</data>
<flag name="review"
          id="449426"
          type_id="1"
          status="+"
          setter="mcatanzaro"
    />
          </attachment>
      

    </bug>

</bugzilla>