Ticket #2057 (closed bug)

Opened 10 years ago

Last modified 9 years ago

[natives]Tweak: Writing a not existing address should act as in a real browser

Reported by: milen Owned by: sveto
Priority: major Milestone: M12_RELEASE
Component: uncategorized Version: 2.0
Keywords: Cc:
Category: unknown Effort:
Importance: 0 Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: milen Design_owners: sveto
Imp._owners: sveto Test_owners:
Analysis_reviewers: sveto Changelog:
Design_reviewers: meddle Imp._reviewers: meddle
Test_reviewers: Analysis_score: 0
Design_score: 0 Imp._score: 0
Test_score: 0

Description (last modified by deyan) (diff)

  1. Create a new book.
  2. Open the HUD for writing the address for the frame and write a not existing one.(for example www.affgwge.com)
  3. Open preview.
  4. Notice the browser frame is empty. It should have a content as in a usual browser (for example "Server not found").

Change History

comment:1 Changed 10 years ago by milen

  • Summary changed from Tweak: Writing a not existing address should act as in a real browser to [natives]Tweak: Writing a not existing address should act as in a real browser

comment:2 Changed 10 years ago by milen

  • Status changed from new to s1b_analysis_finished

comment:3 Changed 10 years ago by sveto

  • Design_owners set to sveto
  • Status changed from s1b_analysis_finished to s2a_design_started

comment:4 Changed 10 years ago by sveto

The native browser needs to be modified to handle HTTP request and responses. The status code from the response and the reason phrase will be forwarded to the Java part of the browser as part of the GetInfo command/response.

comment:5 Changed 10 years ago by sveto

  • Status changed from s2a_design_started to s2b_design_finished

comment:6 Changed 10 years ago by sveto

  • Status changed from s2b_design_finished to s2c_design_ok

comment:7 Changed 10 years ago by sveto

  • Owner set to sveto
  • Status changed from s2c_design_ok to s3a_implementation_started

comment:8 Changed 10 years ago by sveto

As the native browser does not return an HTTP status if establishing an HTTP connection fails, an error status has been added to GetInfo. If it is > 0 then an error has occurred.
Also as the loading is asynchronous a loadFinished status has been added to GetInfo. GetInfo has relevant information about a URL load attempt if loadFinished is true.

comment:9 Changed 10 years ago by sveto

  • Status changed from s3a_implementation_started to s3b_implementation_finished
  • Imp._owners set to sveto

comment:10 Changed 10 years ago by sveto

  • Status changed from s3b_implementation_finished to s2c_design_ok

comment:11 Changed 10 years ago by sveto

  • Status changed from s2c_design_ok to s3a_implementation_started

comment:12 Changed 10 years ago by sveto

Forgot to make changes in Sophie... The behavior should be - when entering an web address the address is checked and if it can't be displayed for some reason the bound control should not accept the address, i.e. fail to validate it.

comment:13 Changed 10 years ago by sveto

  • Status changed from s3a_implementation_started to s3b_implementation_finished

The URL check should be delayed to avoid sending to much request on each keystroke and freezing the user input. Now the address is submitted to the native browser after 2 seconds.
The URL check is now synchronous, if no result comes in 5 seconds the address control fails to validate the URL and a relevant message is displayed.

comment:14 Changed 10 years ago by meddle

  • Status changed from s3b_implementation_finished to s2c_design_ok
  • Design_reviewers set to meddle
  • Analysis_reviewers set to sveto
  • Imp._reviewers set to meddle

You should create a new branch from the trunk in your directory branches/private/sveto, and not delete it and branch it again, now it is not mergeable...

just write something like svn copy svn://sophie2.org/sophie2/trunk svn://sophie2.org/sophie2/branches/private/sveto/bug-2057 and put your code there...

At this state I can not add your code to the trunk.

0p (10m)

comment:15 Changed 10 years ago by deyan

  • Importance set to 20

Batch update from file 0911261.csv

comment:16 Changed 9 years ago by deyan

  • Status changed from s2c_design_ok to new
  • Description modified (diff)

Batch update from file active_tickets.csv

comment:17 Changed 9 years ago by deyan

  • Importance changed from 20 to 0
  • Status changed from new to closed

Batch update from file priorities8.csv

Note: See TracTickets for help on using tickets.